Github user afs commented on the pull request:

    https://github.com/apache/jena/pull/94#issuecomment-156132377
  
    Great. 
    
    You don't need `/testing`. The material under there is for testing query 
execution, not the dataset storage.
    
    For functional tests, `AbstractDatasetGraphTests` and others in that 
package should be enough.  It's all programmatic, not script driven.
    
    There are other sets around that might be worth at least checking they 
work, and if not, put more tests in the datasetgraph programmatic tests 
("testing the tests"). e.g. We can try to feed it through the SPARQL test suite 
as a one-off but they aren't parameterized to implementation.  They test ARQ, 
not the dataset storage and if theer are failures, they should be captured in 
dataset tests.
    
    I have some more tests (coverage of code paths in `DatasetGraphBaseFind`) 
which I'll put in but after this is merged so we have a stable base to get the 
merge done.


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

Reply via email to