URL passed to connectionFactory should not be encoded

Author: MarcialRosales

selenium/amqp10-roundtriptest/src/main/java/com/rabbitmq/amqp1_0/RoundTripTest.java

@@ -46,11 +46,13 @@ public static void main(String args[]) throws Exception {
         connectionParams.add("transport.keyAlias=client-rabbitmq-tls");
       }
       if (!connectionParams.isEmpty()) {
-        String uri2 = uri + "?" + java.net.URLEncoder.encode(String.join("&", connectionParams));       
-        System.out.println("Using AMQP URI " + uri2);
+        uri = uri + "?" + String.join("&", connectionParams);       
+        System.out.println("Using AMQP URI " + uri);
       }
     }
 
+    assertNotNull(uri);
+
     Hashtable<Object, Object> env = new Hashtable<>();
     env.put(Context.INITIAL_CONTEXT_FACTORY, "org.apache.qpid.jms.jndi.JmsInitialContextFactory");
     env.put("connectionfactory.myFactoryLookup", uri);
@@ -59,8 +61,6 @@ public static void main(String args[]) throws Exception {
     env.put("jms.requestTimeout", 5);
     javax.naming.Context context = new javax.naming.InitialContext(env);
 
-    assertNotNull(uri);
-
     ConnectionFactory factory = (ConnectionFactory) context.lookup("myFactoryLookup");
     Destination queue = (Destination) context.lookup("myQueueLookup");