KangweiZhu opened a new pull request, #3221:
URL: https://github.com/apache/tinkerpop/pull/3221

   Hi! We detected flakiness in the following tests using the Nondex tool, The 
reason for the flakiness is similar to 
[#2943](https://github.com/apache/tinkerpop/pull/2943), which is due to the use 
of unordered collections.
   ```
   
org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest.shouldHandleTraversalExplanation[v1]
   
org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest.shouldHandleTraversalExplanation[v2-default]
   
org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest.shouldHandleTraversalExplanation[v2]
   
org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest.shouldHandleTraversalExplanation[v3]
   ```
   
   And correspondingly, the flakiness could be reproduced by running the 
following commands:
   ```
     mvn clean -pl gremlin-core \
       edu.illinois:nondex-maven-plugin:2.2.1:nondex \
       
-Dtest="org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest#shouldHandleTraversalExplanation[v1]"
 \
       -Drat.skip=true
   ```
   ```
     mvn clean -pl gremlin-core \
       edu.illinois:nondex-maven-plugin:2.2.1:nondex \
       
-Dtest="org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest#shouldHandleTraversalExplanation[v2]"
 \
       -Drat.skip=true
   ```
   ```
     mvn clean -pl gremlin-core \
       edu.illinois:nondex-maven-plugin:2.2.1:nondex \
       
-Dtest="org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest#shouldHandleTraversalExplanation[v2-default]"
 \
       -Drat.skip=true
   ```
   ```
     mvn clean -pl gremlin-core \
       edu.illinois:nondex-maven-plugin:2.2.1:nondex \
       
-Dtest="org.apache.tinkerpop.gremlin.structure.io.graphson.GraphSONMapperTest#shouldHandleTraversalExplanation[v3]"
 \
       -Drat.skip=true
   ```
   And the error might be as follows:
   ```
   [INFO]
   [INFO] Results:
   [INFO]
   [ERROR] Failures:
   [ERROR]   GraphSONMapperTest.shouldHandleTraversalExplanation:182 
expected:<...exStep(OUT,edge)"],"[intermediate":[],"final":["InjectStep([])","VertexStep(OUT,vertex)","EdgeVertexStep(OUT)","VertexStep(OUT,edge)"]]}>
 but 
was:<...exStep(OUT,edge)"],"[final":["InjectStep([])","VertexStep(OUT,vertex)","EdgeVertexStep(OUT)","VertexStep(OUT,edge)"],"intermediate":[]]}>
   [INFO]
   [ERROR] Tests run: 1, Failures: 1, Errors: 0, Skipped: 0
   ```
   Once fixed, the output should be `build success` without test errors.
   
   <!--
   Thanks for contributing! Reminders:
   + TARGET the earliest branch where you want the change
       3.7-dev -> 3.7.5 (non-breaking only)
       3.8-dev -> 3.8.0
       master  -> 4.0.0
   + Committers will MERGE the PR forward to newer versions
   + ADD entry to the CHANGELOG.asciidoc for the targeted version
       Do not reference a JIRA number there
   + ADD JIRA number to title and link in description
   + PRs requires 3 +1s from committers OR
                  1 +1 and 7 day wait to merge.
   + MORE details: https://s.apache.org/rtnal
   -->


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to