DRIVER-201: revert LBP query plan to use original contact points

Before-init query plan now uses getContactPoints() (original unresolved
hostname nodes) instead of getResolvedContactPoints(). The DNS expansion
to all IPs happens at the ChannelFactory level (PR scylladb#890), so expanding
here was redundant and broke should_connect_with_mocked_hostname by
replacing hostname endpoints with resolved-IP endpoints.

Also remove the should_connect_when_first_dns_entry_is_non_responsive
integration test from this PR; it belongs in PR scylladb#890 where ChannelFactory
expansion actually enables it to pass.

Author: nikagra

core/src/main/java/com/datastax/oss/driver/internal/core/metadata/LoadBalancingPolicyWrapper.java

@@ -147,9 +147,7 @@ public Queue<Node> newQueryPlan(
     switch (stateRef.get()) {
       case BEFORE_INIT:
       case DURING_INIT:
-        // The contact points are not stored in the metadata yet; use the expanded list so that
-        // all DNS-mapped IPs are tried (not just the first one per hostname).
-        List<Node> nodes = new ArrayList<>(context.getMetadataManager().getResolvedContactPoints());
+        List<Node> nodes = new ArrayList<>(context.getMetadataManager().getContactPoints());
         Collections.shuffle(nodes);
         return new ConcurrentLinkedQueue<>(nodes);
       case RUNNING:

core/src/test/java/com/datastax/oss/driver/internal/core/metadata/LoadBalancingPolicyWrapperTest.java

@@ -101,8 +101,6 @@ public void setup() {
     when(metadataManager.getMetadata()).thenReturn(metadata);
     when(metadata.getNodes()).thenReturn(allNodes);
     when(metadataManager.getContactPoints()).thenReturn(contactPoints);
-    when(metadataManager.getResolvedContactPoints())
-        .thenReturn(ImmutableList.copyOf(contactPoints));
     when(context.getMetadataManager()).thenReturn(metadataManager);
 
     when(context.getConfig()).thenReturn(config);

integration-tests/src/test/java/com/datastax/oss/driver/core/resolver/MockResolverIT.java

@@ -201,46 +201,6 @@ public void run_replace_test_20_times() {
     }
   }
 
-  /**
-   * Verifies that the driver can connect to a cluster when the first DNS entry for the contact
-   * point hostname resolves to a non-responsive IP address (DRIVER-201).
-   *
-   * <p>Contact points are always kept as unresolved hostnames and expanded to all their DNS IPs at
-   * connection time. When the first IP is unreachable the driver tries the remaining ones, so the
-   * session opens successfully even if the first DNS entry is non-responsive.
-   */
-  @Test
-  public void should_connect_when_first_dns_entry_is_non_responsive() {
-    // Use a 2-node cluster on 127.0.1.x.  Node 11 (127.0.1.11) does not exist and is therefore
-    // non-responsive; nodes 1 and 2 are real.
-    try (CcmBridge ccmBridge = CcmBridge.builder().withNodes(2).withIpPrefix("127.0.1.").build()) {
-      MultimapHostResolverProvider.removeResolverEntries("test.cluster.fake");
-      // First entry intentionally points to a non-existent/non-responsive address.
-      MultimapHostResolverProvider.addResolverEntry("test.cluster.fake", "127.0.1.11");
-      MultimapHostResolverProvider.addResolverEntry(
-          "test.cluster.fake", ccmBridge.getNodeIpAddress(1));
-      MultimapHostResolverProvider.addResolverEntry(
-          "test.cluster.fake", ccmBridge.getNodeIpAddress(2));
-      ccmBridge.create();
-      ccmBridge.start();
-
-      DriverConfigLoader loader =
-          new DefaultProgrammaticDriverConfigLoaderBuilder()
-              .withBoolean(TypedDriverOption.RECONNECT_ON_INIT.getRawOption(), false)
-              .withStringList(
-                  TypedDriverOption.CONTACT_POINTS.getRawOption(),
-                  Collections.singletonList("test.cluster.fake:9042"))
-              .build();
-
-      // The session must open successfully despite the first DNS entry being unreachable.
-      try (CqlSession session = new CqlSessionBuilder().withConfigLoader(loader).build()) {
-        ResultSet rs = session.execute("select * from system.local where key='local'");
-        assertThat(rs.one()).isNotNull();
-        waitForAllNodesUp(session, 2);
-      }
-    }
-  }
-
   // This is too long to run during CI, but is useful for manual investigations.
   @SuppressWarnings("unused")
   public void cannot_reconnect_with_resolved_socket() {