Github user ajs6f commented on a diff in the pull request:
https://github.com/apache/jena/pull/287#discussion_r144314830
--- Diff:
jena-arq/src/test/java/org/apache/jena/sparql/core/AbstractTestDataset.java ---
@@ -108,4 +108,12 @@
assertFalse(model1.isIsomorphicWith(ds.getNamedModel(graphName))) ;
assertTrue(model2.isIsomorphicWith(ds.getNamedModel(graphName))) ;
}
+
+ @Test public void dataset_06()
+ {
+ String graphName = "http://example/"; ;
+ Dataset ds = createDataset() ;
+ ds.addNamedModel(graphName, model1) ;
+ assertFalse("Dataset should not be empty after a named graph has
been added!", ds.isEmpty());
+ }
--- End diff --
See above-- so do we need a different impl of `isEmpty` for every kind of
`DatasetGraph`?
---
