Squash a few warnings from IDEA

Author: michaelklishin

src/test/java/com/rabbitmq/client/test/functional/ConnectionRecovery.java

@@ -21,8 +21,9 @@
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.concurrent.atomic.AtomicReference;
 
+@SuppressWarnings("ThrowFromFinallyBlock")
 public class ConnectionRecovery extends BrokerTestCase {
-    public static final long RECOVERY_INTERVAL = 2000;
+    private static final long RECOVERY_INTERVAL = 2000;
 
     public void testConnectionRecovery() throws IOException, InterruptedException {
         assertTrue(connection.isOpen());
@@ -211,7 +212,7 @@ public void testClientNamedQueueRecoveryWithNoWait() throws IOException, Interru
         testClientNamedQueueRecoveryWith("java-client.test.recovery.q1-nowait", true);
     }
 
-    protected void testClientNamedQueueRecoveryWith(String q, boolean noWait) throws IOException, InterruptedException, TimeoutException {
+    private void testClientNamedQueueRecoveryWith(String q, boolean noWait) throws IOException, InterruptedException, TimeoutException {
         Channel ch = connection.createChannel();
         if(noWait) {
             declareClientNamedQueueNoWait(ch, q);
@@ -759,11 +760,11 @@ private void waitForConfirms(Channel ch) throws InterruptedException, TimeoutExc
         ch.waitForConfirms(30 * 60 * 1000);
     }
 
-    protected void assertRecordedQueues(Connection conn, int size) {
+    private void assertRecordedQueues(Connection conn, int size) {
         assertEquals(size, ((AutorecoveringConnection)conn).getRecordedQueues().size());
     }
 
-    protected void assertRecordedExchanges(Connection conn, int size) {
+    private void assertRecordedExchanges(Connection conn, int size) {
         assertEquals(size, ((AutorecoveringConnection)conn).getRecordedExchanges().size());
     }
 }