Github user FlorianHockmann commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/842#discussion_r188686243
--- Diff:
gremlin-dotnet/test/Gremlin.Net.UnitTest/Structure/IO/GraphSON/GraphSONReaderTests.cs
---
@@ -39,7 +39,13 @@ public class GraphSONReaderTests
private GraphSONReader CreateStandardGraphSONReader()
{
return new GraphSONReader();
- }
+ }
+
+ [Fact]
+ public void Assert_loaded_Newtonsoft_Json_version()
+ {
+ Assert.Equal(new Version(11, 0, 0, 0),
typeof(JToken).Assembly.GetName().Version);
--- End diff --
Sorry but I completely missed this test in my last review. I thought that
you just added the test to find out what was causing the other test to fail,
but I don't think that we should add a test like that as we would have to
change it every time we update Newtonsoft.Json.
---
