kenhuuu commented on code in PR #3298:
URL: https://github.com/apache/tinkerpop/pull/3298#discussion_r2729595953


##########
gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs:
##########
@@ -122,6 +122,12 @@ public void RunGherkinBasedTests(IMessageSerializer 
messageSerializer)
                         continue;
                     }
 
+                    if (scenario.Tags.Any(t => t.Name == "@StepWrite"))
+                    {
+                        failedSteps.Add(scenario.Steps.First(), new 
IgnoreException(IgnoreReason.TreeStepNotSupported));

Review Comment:
   Nit: Should probably make a new IgnoreReason instead of using 
TreeStepNotSupported for these write tests.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to