[ 
https://issues.apache.org/jira/browse/TINKERPOP-1320?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15334082#comment-15334082
 ] 

ASF GitHub Bot commented on TINKERPOP-1320:
-------------------------------------------

Github user spmallette commented on the issue:

    https://github.com/apache/tinkerpop/pull/336
  
    `gremlin-core` just have tests that need the some of the same exact 
functions that are in `gremlin-test`, but the dependencies are such that 
`gremlin-core` can't use the `TestHelper` in `gremlin-test` and `gremlin-test` 
can't use the `TestHelper` in `gremlin-core` so there are two of them. 
    
    I suppose we could move `TestHelper` out of `gremlin-core/test` to the main 
source code as a utility??? then there could be one. Or we publish 
`gremlin-core/test` as an artifiact which i kinda hate. Or we create a new 
dependency all together for `gremlin-core` and `gremlin-test` to depend on. All 
of the answers sorta stink i guess. Up until now, copying one file seemed to 
smell the best of all of them....


> GremlinGroovyScriptEngineFileSandboxTest throws error: URI is not hierarchical
> ------------------------------------------------------------------------------
>
>                 Key: TINKERPOP-1320
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1320
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: test-suite
>    Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>            Reporter: Jason Plurad
>            Assignee: Jason Plurad
>            Priority: Minor
>             Fix For: 3.1.3, 3.2.1
>
>
> This is similar to TINKERPOP-1317. The differences here are
> * The {{TestHelper.generateTempFileFromResource()}} call to load the resource 
> is happening from the {{public static void init()}} method before a graph 
> instance is available.
> * A reference to {{GremlinGroovyScriptEngineFileSandboxTest.class}} is still 
> required to located the {{sandbox.yaml}} found in the {{gremlin-test.jar}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to