[tinkerpop] branch driver-35 updated: Catch an exception on close in tests

[sp...@apache.org]

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

spmallette pushed a commit to branch driver-35
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/driver-35 by this push:
     new c13cb36  Catch an exception on close in tests
c13cb36 is described below

commit c13cb36399c97ae351dffb9c4804ba899ff58cb7
Author: stephen <sp...@gmail.com>
AuthorDate: Tue Dec 10 09:28:34 2019 -0500

    Catch an exception on close in tests
    
    Travis shows a NullPointerException coming from here, but not sure how that is possible. Adding logging to see the error better and hopefully allow the test to continue to a safe failure that allows the test to complete successfully.
---
 .../apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

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 d9801e0..9dbe441 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
@@ -1234,7 +1234,11 @@ public class GremlinDriverIntegrateTest extends AbstractGremlinServerIntegration
         } catch (Exception ex) {
             throw ex;
         } finally {
-            cluster.close();
+            try {
+                cluster.close();
+            } catch (Exception ex) {
+                logger.error("Cluster closing failure", ex);
+            }
         }
     }