spmallette commented on code in PR #3379: URL: https://github.com/apache/tinkerpop/pull/3379#discussion_r3057308305
########## gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/test/features/map/Fold.feature: ########## Review Comment: I didn't want to use gherkin to test all possible combinations of this. The gherkin tests are for testing Gremlin not the test infrastructure itself. I think my intention was to go back to tests that we had rewritten because we lacked this functionality and bring them back to the form we once had in the Java tests. I wasn't sure how to go about doing that though and wanted this to move forward. I wonder if our tests are reaching significant enough complexity to warrant their own unit testing to some degree. I'll think about this a bit more I guess. -- 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]
