[tinkerpop] branch TINKERPOP-2338 updated: TINKERPOP-2338 Fix gherkin test runner for .NET

[jorgebg]

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

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


The following commit(s) were added to refs/heads/TINKERPOP-2338 by this push:
     new 3775925  TINKERPOP-2338 Fix gherkin test runner for .NET
3775925 is described below

commit 377592519f3d8cbb0c959769c490ddc38f97c5ee
Author: Jorge Bay Gondra <jorgebaygon...@gmail.com>
AuthorDate: Thu Feb 13 12:34:18 2020 +0100

    TINKERPOP-2338 Fix gherkin test runner for .NET
    
    Default to object for Properties() method generic param
---
 .../test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs       | 2 +-
 .../Gherkin/TraversalEvaluation/ModernGraphTypeInformation.cs           | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs 
b/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs
index 621ec8e..59fd11e 100644
--- 
a/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs
+++ 
b/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/GherkinTestRunner.cs
@@ -41,7 +41,7 @@ namespace Gremlin.Net.IntegrationTest.Gherkin
             new Dictionary<string, IgnoreReason>
             {
                 // Add here the name of scenarios to ignore and the reason, 
e.g.:
-                { "g_V_properties_propertiesXstartTimeX_drop", 
IgnoreReason.NoReason },
+                //{ "g_V_properties_propertiesXstartTimeX_drop", 
IgnoreReason.NoReason },
             };
         
         private static class Keywords
diff --git 
a/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/TraversalEvaluation/ModernGraphTypeInformation.cs
 
b/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/TraversalEvaluation/ModernGraphTypeInformation.cs
index 77ed4eb..d77ec5e 100644
--- 
a/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/TraversalEvaluation/ModernGraphTypeInformation.cs
+++ 
b/gremlin-dotnet/test/Gremlin.Net.IntegrationTest/Gherkin/TraversalEvaluation/ModernGraphTypeInformation.cs
@@ -61,6 +61,7 @@ namespace 
Gremlin.Net.IntegrationTest.Gherkin.TraversalEvaluation
             switch (method.Name)
             {
                 case nameof(GraphTraversal<object,object>.Properties):
+                    return typeof(object);
                 case nameof(GraphTraversal<object,object>.Values) when 
parameterValues.Length == 1:
                     // The parameter contains the element property names
                     var properties = ((IEnumerable) 
parameterValues[parameterValues.Length - 1]).Cast<string>();