(tinkerpop) branch master updated: CTR: Fix gremlin-driver integration tests

[colegreer]

This is an automated email from the ASF dual-hosted git repository.

colegreer pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/master by this push:
     new b1bda549b3 CTR: Fix gremlin-driver integration tests
b1bda549b3 is described below

commit b1bda549b3899f27de35aef3606b0ecf8fd57708
Author: Cole-Greer <cole.gr...@improving.com>
AuthorDate: Tue May 6 09:12:18 2025 -0700

    CTR: Fix gremlin-driver integration tests
    
    Test shouldFailOnInitiallyDeadHostForSessionedClient was mistakenly
    reintroduced to the master branch upon merging from 3.8-dev. This
    test does not belong in TP4 as the sessioned client has been removed.
---
 .../gremlin/server/GremlinDriverIntegrateTest.java       | 16 +++-------------
 1 file changed, 3 insertions(+), 13 deletions(-)

diff --git 
a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java
 
b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java
index d7f7bc5192..54e70af717 100644
--- 
a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java
+++ 
b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java
@@ -1185,14 +1185,14 @@ public class GremlinDriverIntegrateTest extends 
AbstractGremlinServerIntegration
 
     /**
      * Client created on an initially dead host should fail initially, and 
recover after the dead host has restarted
-     * @param testClusterClient - boolean flag set to test clustered client if 
true and sessioned client if false.
      */
-    private void testShouldFailOnInitiallyDeadHost(final boolean 
testClusterClient) throws Exception {
+    @Test
+    public void shouldFailOnInitiallyDeadHostForClusterClient() throws 
Exception {
         logger.info("Stopping server.");
         this.stopServer();
 
         final Cluster cluster = TestClientFactory.build().create();
-        final Client client = testClusterClient? cluster.connect() : 
cluster.connect("sessionClient");
+        final Client client = cluster.connect();
 
         try {
             // try to re-issue a request now that the server is down
@@ -1242,16 +1242,6 @@ public class GremlinDriverIntegrateTest extends 
AbstractGremlinServerIntegration
         }
     }
 
-    @Test
-    public void shouldFailOnInitiallyDeadHostForClusterClient() throws 
Exception {
-        testShouldFailOnInitiallyDeadHost(true);
-    }
-
-    @Test
-    public void shouldFailOnInitiallyDeadHostForSessionClient() throws 
Exception {
-        testShouldFailOnInitiallyDeadHost(false);
-    }
-
     @Test
     public void shouldReturnUuid() throws Exception {
         final Cluster cluster = TestClientFactory.build().create();