[
https://issues.apache.org/jira/browse/TINKERPOP3-984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15022224#comment-15022224
]
ASF GitHub Bot commented on TINKERPOP3-984:
-------------------------------------------
Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/150#issuecomment-158962321
I reviewed the code and ran `mvn clean install -DincludeNeo4j` and `BUILD
SUCCESSFUL`. Yes, @velo , we simply had a set of cases that we not using
bindings in an intelligent way. Thanks for fixing them up.
When we merge this, I will make a TinkerGraph provider/test that uses an
object id that does not have a nice `toString()` so we make sure this doesn't
pop up again.
VOTE +1.
> Use GraphProvider for id conversion in Groovy Environment test suite
> --------------------------------------------------------------------
>
> Key: TINKERPOP3-984
> URL: https://issues.apache.org/jira/browse/TINKERPOP3-984
> Project: TinkerPop 3
> Issue Type: Improvement
> Components: test-suite
> Affects Versions: 3.0.2-incubating
> Reporter: Marvin Froeder
> Assignee: stephen mallette
> Fix For: 3.1.1-incubating
>
>
> Groovy related tests rely on ID.toString(), which can cause problems with any
> implementation that uses non-primitive ids:
> {code}
> Script1.groovy: 1: unexpected char: '#' @ line 1, column 5.
> g.V(#11:0).next()
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
