Github user osma commented on the issue:
https://github.com/apache/jena/pull/227
@anujgandharv That was what I meant - get rid of all the currently written
ES test classes completely by moving all the existing unit tests to the new
integration tests.
I've suggested two sets of test scenarios in these review comments above:
https://github.com/apache/jena/pull/227#discussion_r106405506
https://github.com/apache/jena/pull/227#discussion_r106406514
If you can implement those as integration tests then at least from my
perspective that would be enough to consider this ready for merging :) I still
want to take another pass at the code but perhaps I'll wait for your
integration tests first as they may affect the code too.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
