You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by sp...@apache.org on 2021/09/13 19:16:25 UTC

[tinkerpop] 15/17: TINKERPOP-2601 Fixed failing gherkin test configurations

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

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

commit daff9ab23d6d4a40df78bcf9db831f75725360de
Author: Stephen Mallette <st...@amazon.com>
AuthorDate: Thu Sep 9 13:32:20 2021 -0400

    TINKERPOP-2601 Fixed failing gherkin test configurations
---
 .../process/ProcessLimitedComputerSuite.java       | 58 ---------------------
 .../process/ProcessLimitedStandardSuite.java       | 59 ----------------------
 .../src/test/resources/cucumber.properties         |  2 +-
 .../SparkGraphComputerProcessIntegrateTest.java    |  3 +-
 .../src/test/resources/cucumber.properties         |  2 +-
 5 files changed, 4 insertions(+), 120 deletions(-)

diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedComputerSuite.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedComputerSuite.java
index ee77c91..bf3f357 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedComputerSuite.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedComputerSuite.java
@@ -151,68 +151,10 @@ public class ProcessLimitedComputerSuite extends AbstractGremlinSuite {
      * A list of the minimum set of base tests that Gremlin flavors should implement to be compliant with Gremlin.
      */
     private static final Class<?>[] testsToEnforce = new Class<?>[]{
-            // branch
-            BranchTest.class,
-            ChooseTest.class,
-            OptionalTest.class,
-            LocalTest.class,
-            RepeatTest.class,
-            UnionTest.class,
-
-            // filter
-            AndTest.class,
-            CoinTest.class,
-            CyclicPathTest.class,
-            DedupTest.class,
-            FilterTest.class,
-            HasTest.class,
-            IsTest.class,
-            OrTest.class,
-            RangeTest.class,
-            SampleTest.class,
-            SimplePathTest.class,
-            TailTest.class,
-            WhereTest.class,
-
-            // map
-            CoalesceTest.class,
-            ConstantTest.class,
-            CountTest.class,
-            FlatMapTest.class,
-            ElementMapTest.class,
-            FoldTest.class,
             MatchTest.class,
-            MathTest.class,
-            MapTest.class,
-            MaxTest.class,
-            MeanTest.class,
-            MinTest.class,
-            SumTest.class,
-            MatchTest.class,
-            OrderTest.class,
-            PageRankTest.class,
-            // PeerPressureTest.class,
-            PathTest.class,
             ProfileTest.class,
-            ProjectTest.class,
             ProgramTest.class,
-            PropertiesTest.class,
-            ShortestPathTest.class,
-            SelectTest.class,
-            UnfoldTest.class,
-            ValueMapTest.class,
-            VertexTest.class,
-
-            // sideEffect
-            AddEdgeTest.class,
-            AggregateTest.class,
-            GroupTest.class,
-            GroupCountTest.class,
-            InjectTest.class,
-            SackTest.class,
-            SideEffectCapTest.class,
             SideEffectTest.class,
-            StoreTest.class,
             SubgraphTest.class,
             TreeTest.class
     };
diff --git a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedStandardSuite.java b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedStandardSuite.java
index 1f5d6aa..ddf3280 100644
--- a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedStandardSuite.java
+++ b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/ProcessLimitedStandardSuite.java
@@ -146,67 +146,8 @@ public class ProcessLimitedStandardSuite extends AbstractGremlinSuite {
      * A list of the minimum set of base tests that Gremlin flavors should implement to be compliant with Gremlin.
      */
     private static final Class<?>[] testsToEnforce = new Class<?>[]{
-            // branch
-            BranchTest.class,
-            ChooseTest.class,
-            OptionalTest.class,
-            LocalTest.class,
-            RepeatTest.class,
-            UnionTest.class,
-
-            // filter
-            AndTest.class,
-            CoinTest.class,
-            CyclicPathTest.class,
-            DedupTest.class,
-            DropTest.class,
-            FilterTest.class,
-            HasTest.class,
-            IsTest.class,
-            OrTest.class,
-            RangeTest.class,
-            SampleTest.class,
-            SimplePathTest.class,
-            TailTest.class,
-            WhereTest.class,
-
-            // map
-            AddEdgeTest.class,
-            AddVertexTest.class,
-            CoalesceTest.class,
-            ConstantTest.class,
-            CountTest.class,
-            ElementMapTest.class,
-            FlatMapTest.class,
-            FoldTest.class,
-            LoopsTest.class,
-            IndexTest.class,
-            MapTest.class,
-            MatchTest.class,
-            MathTest.class,
-            MaxTest.class,
-            MeanTest.class,
-            MinTest.class,
-            SumTest.class,
-            OrderTest.class,
-            PathTest.class,
-            PropertiesTest.class,
             ProfileTest.class,
-            ProjectTest.class,
-            SelectTest.class,
-            VertexTest.class,
-            UnfoldTest.class,
-            ValueMapTest.class,
-
-            // sideEffect
-            AggregateTest.class,
-            GroupTest.class,
-            GroupCountTest.class,
-            InjectTest.class,
-            SackTest.class,
-            SideEffectCapTest.class,
             SideEffectTest.class,
-            StoreTest.class,
             SubgraphTest.class,
             TreeTest.class,
     };
diff --git a/hadoop-gremlin/src/test/resources/cucumber.properties b/hadoop-gremlin/src/test/resources/cucumber.properties
index 66a963b..b30d19d 100644
--- a/hadoop-gremlin/src/test/resources/cucumber.properties
+++ b/hadoop-gremlin/src/test/resources/cucumber.properties
@@ -1 +1 @@
-guice.injector-source=org.apache.tinkerpop.gremlin.hadoop.HadoopGraphFeatureTest$WorldInjectorSource
\ No newline at end of file
+guice.injector-source=org.apache.tinkerpop.gremlin.hadoop.HadoopGraphFeatureIntegrateTest$WorldInjectorSource
\ No newline at end of file
diff --git a/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/process/computer/SparkGraphComputerProcessIntegrateTest.java b/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/process/computer/SparkGraphComputerProcessIntegrateTest.java
index f38dcf5..c909b5b 100644
--- a/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/process/computer/SparkGraphComputerProcessIntegrateTest.java
+++ b/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/process/computer/SparkGraphComputerProcessIntegrateTest.java
@@ -21,12 +21,13 @@ package org.apache.tinkerpop.gremlin.spark.process.computer;
 import org.apache.tinkerpop.gremlin.GraphProviderClass;
 import org.apache.tinkerpop.gremlin.hadoop.structure.HadoopGraph;
 import org.apache.tinkerpop.gremlin.process.ProcessComputerSuite;
+import org.apache.tinkerpop.gremlin.process.ProcessLimitedStandardSuite;
 import org.junit.runner.RunWith;
 
 /**
  * @author Marko A. Rodriguez (http://markorodriguez.com)
  */
-@RunWith(ProcessComputerSuite.class)
+@RunWith(ProcessLimitedStandardSuite.class)
 @GraphProviderClass(provider = SparkHadoopGraphProvider.class, graph = HadoopGraph.class)
 public class SparkGraphComputerProcessIntegrateTest {
 }
diff --git a/spark-gremlin/src/test/resources/cucumber.properties b/spark-gremlin/src/test/resources/cucumber.properties
index 0180176..21cdec8 100644
--- a/spark-gremlin/src/test/resources/cucumber.properties
+++ b/spark-gremlin/src/test/resources/cucumber.properties
@@ -1 +1 @@
-guice.injector-source=org.apache.tinkerpop.gremlin.spark.SparkGraphFeatureTest$WorldInjectorSource
\ No newline at end of file
+guice.injector-source=org.apache.tinkerpop.gremlin.spark.SparkGraphFeatureIntegrateTest$WorldInjectorSource
\ No newline at end of file