Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/175#issuecomment-163924998
I've found that with IO tests you can't assume too much. Asserting counts
isn't quite enough. I would assert the edge labels as well as direction of the
edges after the read. I like this set of tests you've added btw - still not
convinced there's a problem in the `GraphSONReader` that needs to be fixed, but
I'd like to merge in the tests regardless of the ultimate finding there.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
