[
https://issues.apache.org/jira/browse/TINKERPOP3-984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15022255#comment-15022255
]
ASF GitHub Bot commented on TINKERPOP3-984:
-------------------------------------------
Github user spmallette commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/150#issuecomment-158966996
I merged this locally and did a `mvn clean install -DincludeNeo4j` and `mvn
clean install -DskipIntegrationTests=false -pl tinkergraph-gremlin` to at least
exercise the integration tests for groovy. Of course, neither of these is a
perfect tests here because neither Neo4j nor TinkerGraph use complex objects
for identifiers. If OrientDB is good with this change then this should be good
to go.
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)
