[TINKERPOP-1814] Some process tests require the graph API

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.2.6
• Fix Version/s: 3.2.7, 3.3.1
• Component/s: process
• Assignee: Marko A. Rodriguez
• Reporter: Bryn Cooke
• Created: 26/Oct/17 11:41
• Updated: 09/Nov/17 17:35
• Resolved: 09/Nov/17 17:35

Description

This test uses the graph API:
https://github.com/apache/tinkerpop/blob/master/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/AddEdgeTest.java#L137-L142

This test assumes that the reference to the vertex will be updated. It should at least fetch the vertex again after it has been updated, but ideally use a traversal for verification.
https://github.com/apache/tinkerpop/blob/master/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/PartitionStrategyProcessTest.java#L142-L143

These tests disable the strategies supplied by the implementing graph and therefore they will default to using the graph API under the hood.
https://github.com/apache/tinkerpop/blob/master/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/CoreTraversalTest.java#L222-L261

Issue Links

• links to: GitHub Pull Request #740