Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1814

Some process tests require the graph API

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.2.6
    • 3.2.7, 3.3.1
    • process
    • None

    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

      Attachments

        Issue Links

          Activity

            People

              okram Marko A. Rodriguez
              bryncooke Bryn Cooke
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: