Increased timeout a bit for test. Wasn't giving enough time (suddenly) to handle the follow-on request that ensures calls work after a timeout has triggered. CTR
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/a5d60c6b Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/a5d60c6b Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/a5d60c6b Branch: refs/heads/TINKERPOP-1489 Commit: a5d60c6b844aee5169238f000b5fd0b66d850e61 Parents: 2ba4104 Author: Stephen Mallette <[email protected]> Authored: Mon Jul 17 11:36:14 2017 -0400 Committer: Stephen Mallette <[email protected]> Committed: Mon Jul 17 12:24:44 2017 -0400 ---------------------------------------------------------------------- .../tinkerpop/gremlin/server/GremlinServerIntegrateTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/a5d60c6b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java ---------------------------------------------------------------------- diff --git a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java index aad8131..8b0c280 100644 --- a/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java +++ b/gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinServerIntegrateTest.java @@ -157,7 +157,7 @@ public class GremlinServerIntegrateTest extends AbstractGremlinServerIntegration settings.writeBufferLowWaterMark = 32; break; case "shouldReceiveFailureTimeOutOnScriptEval": - settings.scriptEvaluationTimeout = 200; + settings.scriptEvaluationTimeout = 1000; break; case "shouldReceiveFailureTimeOutOnTotalSerialization": settings.serializedResponseTimeout = 1; @@ -699,7 +699,7 @@ public class GremlinServerIntegrateTest extends AbstractGremlinServerIntegration public void shouldReceiveFailureTimeOutOnScriptEval() throws Exception { try (SimpleClient client = TestClientFactory.createWebSocketClient()){ final List<ResponseMessage> responses = client.submit("Thread.sleep(3000);'some-stuff-that-should not return'"); - assertThat(responses.get(0).getStatus().getMessage(), startsWith("Script evaluation exceeded the configured 'scriptEvaluationTimeout' threshold of 200 ms")); + assertThat(responses.get(0).getStatus().getMessage(), startsWith("Script evaluation exceeded the configured 'scriptEvaluationTimeout' threshold of 1000 ms")); // validate that we can still send messages to the server assertEquals(2, ((List<Integer>) client.submit("1+1").get(0).getResult().getData()).get(0).intValue());
