One more test case for channel.open

It's not currently possible to work around
ChannelManager's channel number availability
checks without making ConsumerWorkPool public
so that ChannelN can be constructed directly.

Other clients are not as strict about channel
number validation.

Author: michaelklishin

test/src/com/rabbitmq/client/test/server/ChannelLimitNegotiation.java

@@ -3,9 +3,7 @@
 import com.rabbitmq.client.AMQP;
 import com.rabbitmq.client.Connection;
 import com.rabbitmq.client.ConnectionFactory;
-import com.rabbitmq.client.impl.AMQConnection;
-import com.rabbitmq.client.impl.DefaultExceptionHandler;
-import com.rabbitmq.client.impl.SocketFrameHandler;
+import com.rabbitmq.client.impl.*;
 import com.rabbitmq.client.test.BrokerTestCase;
 import com.rabbitmq.tools.Host;
 
@@ -46,7 +44,7 @@ protected int negotiateChannelMax(int requestedChannelMax, int serverMax) {
     }
   }
 
-  public void testChannelMaxLowerThanServerValue() throws Exception {
+  public void testChannelMaxLowerThanServerMinimum() throws Exception {
     int n = 64;
     ConnectionFactory cf = new ConnectionFactory();
     cf.setRequestedChannelMax(n);
@@ -70,4 +68,18 @@ public void testChannelMaxGreaterThanServerValue() throws Exception {
         Host.rabbitmqctl("eval 'application:set_env(rabbit, channel_max, 0).'");
       }
   }
+
+  public void testOpeningChannelWithIdGreaterThanChannelMaxFails() throws Exception {
+    int n = 48;
+
+    ConnectionFactory cf = new ConnectionFactory();
+    Connection conn = cf.newConnection();
+    assertEquals(n, conn.getChannelMax());
+
+    for(int i = 1; i <= n; i++) {
+      assertNotNull(conn.createChannel(i));
+    }
+    // ChannelManager guards against channel.open being sent
+    assertNull(conn.createChannel(n + 1));
+  }
 }