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

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

Github user dkuppitz commented on the pull request:

    
https://github.com/apache/incubator-tinkerpop/pull/178#issuecomment-164510250
  
    * `mvn clean install`: worked
    * integration tests: worked
    
    VOTE: +1


> Consistent test directory usage
> -------------------------------
>
>                 Key: TINKERPOP-954
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-954
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: test-suite
>    Affects Versions: 3.0.2-incubating
>            Reporter: stephen mallette
>            Assignee: stephen mallette
>            Priority: Minor
>             Fix For: 3.1.1-incubating
>
>
> There are a number of tests that need to use the file system to read/write 
> input/output.  Determining that directory to read/write from/to is currently 
> done in a number of ways in different contexts in the test suite.
> The appropriate method is to use {{TestHelper}} to construct that path as it 
> accounts for a number of environmental situations.  {{TestHelper}} could use 
> some additional work to improve it's usage - maybe it needs its own unit 
> tests as well.



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

Reply via email to