kenhuuu commented on PR #3714:
URL: https://github.com/apache/tinkerpop/pull/3714#issuecomment-6093425780

   The added unit tests are good and directly test the implemented fix however 
it doesn't actually test `option()`. What's needed to cover this are feature 
tests. In the `3.8-dev` branch you can assert for errors during traversal 
construction due a small change in the test harness. I think you should 
backport that harness fix and add feature tests for this change.


-- 
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