Github user FlorianHockmann commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/817#discussion_r175086849
  
    --- Diff: 
gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs ---
    @@ -38,11 +38,7 @@ namespace Gremlin.Net.IntegrationTest.Gherkin
         public class GherkinTestRunner
         {
             private static readonly IDictionary<string, IgnoreReason> 
IgnoredScenarios =
    -            new Dictionary<string, IgnoreReason>
    -            {
    -                {"g_V_hasIdXwithinXemptyXX_count", 
IgnoreReason.PWithinWrapsArgumentsInArray},
    -                {"g_VX1X_out_aggregateXxX_out_whereXnotXwithinXaXXX", 
IgnoreReason.PWithinWrapsArgumentsInArray}
    --- End diff --
    
    Please also remove the enum value here: 
https://github.com/apache/tinkerpop/blob/tp32/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/IgnoreException.cs#L60


---

Reply via email to