TINKERPOP-1612 Configure tests to allow them to pass under new plugins
Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/d282d0e9 Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/d282d0e9 Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/d282d0e9 Branch: refs/heads/TINKERPOP-1612 Commit: d282d0e9d1e5ec938fcdd0e5f46a6396fd6d1c9e Parents: 1fc5293 Author: Stephen Mallette <sp...@genoprime.com> Authored: Wed Jan 25 06:52:10 2017 -0500 Committer: Stephen Mallette <sp...@genoprime.com> Committed: Mon Jan 30 15:16:06 2017 -0500 ---------------------------------------------------------------------- .../gremlin/groovy/engine/GremlinExecutor.java | 4 +--- .../gremlin/server/GremlinServerIntegrateTest.java | 14 +++++--------- 2 files changed, 6 insertions(+), 12 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/d282d0e9/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java ---------------------------------------------------------------------- diff --git a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java index 8fdb86e..80e72fd 100644 --- a/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java +++ b/gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/engine/GremlinExecutor.java @@ -128,10 +128,8 @@ public class GremlinExecutor implements AutoCloseable { // deprecated code around this is removed. if (!useGremlinScriptEngineManager) this.scriptEngines = createScriptEngines(); - else { + else this.scriptEngines = null; - gremlinScriptEngineManager.getEngineByName("gremlin-groovy"); - } this.suppliedExecutor = suppliedExecutor; this.suppliedScheduledExecutor = suppliedScheduledExecutor; http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/d282d0e9/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 88b0339..16476c3 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 @@ -43,8 +43,8 @@ import org.apache.tinkerpop.gremlin.driver.ser.Serializers; import org.apache.tinkerpop.gremlin.driver.simple.SimpleClient; import org.apache.tinkerpop.gremlin.groovy.jsr223.GremlinGroovyScriptEngine; import org.apache.tinkerpop.gremlin.groovy.jsr223.GroovyCompilerGremlinPlugin; -import org.apache.tinkerpop.gremlin.groovy.jsr223.customizer.ConfigurationCustomizerProvider; import org.apache.tinkerpop.gremlin.groovy.jsr223.customizer.SimpleSandboxExtension; +import org.apache.tinkerpop.gremlin.jsr223.ScriptFileGremlinPlugin; import org.apache.tinkerpop.gremlin.process.remote.RemoteGraph; import org.apache.tinkerpop.gremlin.process.traversal.Traversal; import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversal; @@ -65,7 +65,6 @@ import org.junit.Test; import java.lang.reflect.Field; import java.nio.channels.ClosedChannelException; import java.util.ArrayList; -import java.util.Collections; import java.util.HashMap; import java.util.List; import java.util.Map; @@ -205,6 +204,8 @@ public class GremlinServerIntegrateTest extends AbstractGremlinServerIntegration return settings; case "shouldUseSimpleSandbox": settings.scriptEngines.get("gremlin-groovy").plugins.put(GroovyCompilerGremlinPlugin.class.getName(), getScriptEngineConfForSimpleSandbox()); + // remove the script because it isn't used in the test but also because it's not CompileStatic ready + settings.scriptEngines.get("gremlin-groovy").plugins.remove(ScriptFileGremlinPlugin.class.getName()); break; case "shouldUseInterpreterMode": settings.scriptEngines.get("gremlin-groovy").plugins.put(GroovyCompilerGremlinPlugin.class.getName(), getScriptEngineConfForInterpreterMode()); @@ -213,6 +214,7 @@ public class GremlinServerIntegrateTest extends AbstractGremlinServerIntegration settings.scriptEngines.get("gremlin-groovy").plugins.put(GroovyCompilerGremlinPlugin.class.getName(), getScriptEngineConfForTimedInterrupt()); break; case "shouldUseBaseScript": + settings.scriptEngines.get("gremlin-groovy").plugins.put(GroovyCompilerGremlinPlugin.class.getName(), getScriptEngineConfForBaseScript()); settings.scriptEngines.get("gremlin-groovy").config = getScriptEngineConfForBaseScript(); break; } @@ -253,15 +255,9 @@ public class GremlinServerIntegrateTest extends AbstractGremlinServerIntegration private static Map<String, Object> getScriptEngineConfForBaseScript() { final Map<String,Object> scriptEngineConf = new HashMap<>(); - final Map<String,Object> compilerCustomizerProviderConf = new HashMap<>(); - final List<Object> keyValues = new ArrayList<>(); - final Map<String,Object> properties = new HashMap<>(); properties.put("ScriptBaseClass", BaseScriptForTesting.class.getName()); - keyValues.add(properties); - - compilerCustomizerProviderConf.put(ConfigurationCustomizerProvider.class.getName(), keyValues); - scriptEngineConf.put("compilerCustomizerProviders", compilerCustomizerProviderConf); + scriptEngineConf.put("compilerConfigurationOptions", properties); return scriptEngineConf; }