[tinkerpop] branch master updated: Fixed some feature test ignores to prevent random test failures

[spmallette]

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

spmallette 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 4b6713b  Fixed some feature test ignores to prevent random test 
failures
4b6713b is described below

commit 4b6713ba206825d09ec43650eaf131c478bf5061
Author: Stephen Mallette <stepm...@amazon.com>
AuthorDate: Tue Dec 21 12:24:16 2021 -0500

    Fixed some feature test ignores to prevent random test failures
    
    Really need some improvements to the assertion framework to better handle 
these tests. CTR
---
 .../main/java/org/apache/tinkerpop/gremlin/features/StepDefinition.java | 1 -
 .../apache/tinkerpop/gremlin/spark/SparkGraphFeatureIntegrateTest.java  | 2 +-
 .../java/org/apache/tinkerpop/gremlin/tinkergraph/TinkerGraphWorld.java | 2 +-
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/features/StepDefinition.java
 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/features/StepDefinition.java
index daeff68..e4d5e37 100644
--- 
a/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/features/StepDefinition.java
+++ 
b/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/features/StepDefinition.java
@@ -294,7 +294,6 @@ public final class StepDefinition {
     public void theResultShouldBeOf(final DataTable dataTable) {
         assertThatNoErrorWasThrown();
 
-
         final List<Object> actual = translateResultsToActual();
 
         // skip the header in the dataTable
diff --git 
a/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/SparkGraphFeatureIntegrateTest.java
 
b/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/SparkGraphFeatureIntegrateTest.java
index 3b15904..b6e877d 100644
--- 
a/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/SparkGraphFeatureIntegrateTest.java
+++ 
b/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/SparkGraphFeatureIntegrateTest.java
@@ -92,7 +92,7 @@ public class SparkGraphFeatureIntegrateTest {
         add(Pair.with("g_V_group_byXoutE_countX_byXnameX", 
skipReasonOrdering));
         
add(Pair.with("g_V_asXvX_mapXbothE_weight_foldX_sumXlocalX_asXsX_selectXv_sX_order_byXselectXsX_descX",
 skipReasonOrdering));
         add(Pair.with("g_V_hasXlangX_groupXaX_byXlangX_byXnameX_out_capXaX", 
skipReasonOrdering));
-        add(Pair.with("g_V_group_byXageX", skipReasonOrdering));
+        
add(Pair.with("g_withStrategiesXProductiveByStrategyX_V_group_byXageX", 
skipReasonOrdering));
         add(Pair.with("g_V_order_byXoutE_count_descX", skipReasonOrdering));
         add(Pair.with("g_V_both_both_dedup_byXoutE_countX_name", 
skipReasonOrdering));
         add(Pair.with("g_V_mapXbothE_weight_foldX_order_byXsumXlocalX_descX", 
skipReasonOrdering));
diff --git 
a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/TinkerGraphWorld.java
 
b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/TinkerGraphWorld.java
index cda080f..7d6f820 100644
--- 
a/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/TinkerGraphWorld.java
+++ 
b/tinkergraph-gremlin/src/test/java/org/apache/tinkerpop/gremlin/tinkergraph/TinkerGraphWorld.java
@@ -118,7 +118,7 @@ public class TinkerGraphWorld implements World {
                 "g_V_group_byXoutE_countX_byXnameX",
                 
"g_V_asXvX_mapXbothE_weight_foldX_sumXlocalX_asXsX_selectXv_sX_order_byXselectXsX_descX",
                 "g_V_hasXlangX_groupXaX_byXlangX_byXnameX_out_capXaX",
-                "g_V_group_byXageX",
+                "g_withStrategiesXProductiveByStrategyX_V_group_byXageX",
                 "g_V_order_byXoutE_count_descX",
                 "g_V_both_both_dedup_byXoutE_countX_name",
                 "g_V_mapXbothE_weight_foldX_order_byXsumXlocalX_descX",