JCBC-1207: Dont do reverse address lookup for DNS SRV bootstrap

Motivation
----------
Changes for JVMCBC-513 made sure the addresses can be resolved to fix
unhandled NPE. This check is not necessary for DNS SRV bootstrap.

Changes
-------
Use the unresolved hosts from ConnectionString in core. Changed the bad
ips to x.y.z on the tests as they resolve now.

Change-Id: I0821c247923144c103416d0a43d8ddf6ec2c9b73
Reviewed-on: http://review.couchbase.org/95228
Reviewed-by: Michael Nitschinger <[email protected]>
Tested-by: Michael Nitschinger <[email protected]>

Author: bsubhashni

src/integration/java/com/couchbase/client/java/ConnectionTest.java

@@ -83,7 +83,14 @@ public void shouldBootstrapWithBadHost() {
 
     @Test
     public void shouldProvideClusterInfoWithBadHostInBootstrapList() {
-        Cluster cluster = CouchbaseCluster.create("1.1.1.1", "2.2.2.2", "3.3.3.3", "4.4.4.4", TestProperties.seedNode());
+        Cluster cluster = CouchbaseCluster.create("x.y.z", TestProperties.seedNode());
+        cluster.authenticate(TestProperties.adminName(), TestProperties.adminPassword());
+        cluster.clusterManager().info();
+    }
+
+    @Test(expected = NullPointerException.class)
+    public void shouldNotCheckReverseLookupWhenDNSSRVEnabled() throws Exception {
+        Cluster cluster = CouchbaseCluster.create(DefaultCouchbaseEnvironment.builder().dnsSrvEnabled(true).build(), "x.y.z");
         cluster.authenticate(TestProperties.adminName(), TestProperties.adminPassword());
         cluster.clusterManager().info();
     }

src/main/java/com/couchbase/client/java/CouchbaseAsyncCluster.java

@@ -325,11 +325,11 @@ private static List<String> assembleSeedNodes(ConnectionString connectionString,
      */
     private static void seedNodesViaDnsSrv(ConnectionString connectionString,
        CouchbaseEnvironment environment, List<String> seedNodes) {
-        if (connectionString.hosts().size() == 1) {
-            InetSocketAddress lookupNode = connectionString.hosts().get(0);
+        if (connectionString.allHosts().size() == 1) {
+            InetSocketAddress lookupNode = connectionString.allHosts().get(0);
             LOGGER.debug(
                 "Attempting to load DNS SRV records from {}.",
-                connectionString.hosts().get(0)
+                connectionString.allHosts().get(0)
             );
 
             try {