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

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

Github user okram commented on the issue:

    https://github.com/apache/tinkerpop/pull/511
  
    This is great. We should really be diligent to keep growing the tests here. 
As our strategies get more complex (and are working in recursive fashions) we 
really need to have complex, nasty nested traversal test cases.
    
    I think that growing `ComplexTest.Traversals` should be CTR based to make 
it easy/convenient to expand as much as possible.
    
    VOTE +1.


> Create a ComplexTraversalTest with crazy nested gnarly traversals.
> ------------------------------------------------------------------
>
>                 Key: TINKERPOP-1539
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1539
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process, test-suite
>    Affects Versions: 3.2.3
>            Reporter: Marko A. Rodriguez
>            Assignee: Daniel Kuppitz
>
> Our {{ProcessSuite}} has numerous tests verifying the semantics of the 
> various steps. Unfortunately, these tests are on simple traversals focused on 
> exposing the step in question in isolation.
> It would be good to add {{ComplexTraversalTest}} to the {{ProcessSuite}} 
> which has traversals over the Grateful Dead graph (for complexity reasons) 
> doing:
> 1. Numerous nests.
> 2. Match/Select/Where complexities.
> 3. Global side-effect access and unrolling and injecting.
> 4. ... just a bunch of nasty stuff.
> This will give us much more confidence as we add more strategies and 
> potentially, mess up our algebra which isn't exposed by the simple 
> "flat'-traversals we current test with.
> [~dkuppitz] --- would you be interested in doing this?



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

Reply via email to