[
https://issues.apache.org/jira/browse/TINKERPOP-2055?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16638549#comment-16638549
]
ASF GitHub Bot commented on TINKERPOP-2055:
-------------------------------------------
Github user FlorianHockmann commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/947#discussion_r222751451
--- Diff:
gremlin-javascript/src/main/javascript/gremlin-javascript/test/unit/graphson-test.js
---
@@ -46,6 +46,30 @@ describe('GraphSONReader', function () {
assert.strictEqual(result, item[1]);
});
});
+ it('should parse GraphSON Nan from GraphSON', function () {
+ const reader = new GraphSONReader();
+ var result = reader.read({
+ "@type": "g:Double",
+ "@value": "NaN"
+ });
+ assert.ok(isNaN(result));
+ });
+ it('should parse GraphSON -Infinity and Nan from GraphSON', function () {
--- End diff --
Just a minor nit but why does it say _and Nan_ when only `-Infinity` is
tested here?
> Provide support for special number cases like Infinity in GraphSON
> ------------------------------------------------------------------
>
> Key: TINKERPOP-2055
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2055
> Project: TinkerPop
> Issue Type: Improvement
> Components: io
> Affects Versions: 3.2.9
> Reporter: stephen mallette
> Assignee: stephen mallette
> Priority: Minor
>
> GraphSON currently doesn't handle: {{Double.Nan}},
> {{Double.NEGATIVE_INFINITY}} or {{Double.POSITIVE_INFINITY}} because it has a
> custom {{Double}} serializer that ignores those cases in GraphSON 2.0 and
> 3.0. Not going to try to solve this for 1.0.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
