fix: dont set _graphs = false

rdfjs · Sep 8, 2024 · 37c7380 · 37c7380
1 parent 78932b4
commit 37c7380
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/N3Store.js b/src/N3Store.js
@@ -921,8 +921,11 @@ export default class N3Store {
     }
     else if ((other instanceof N3Store) && this._entityIndex === other._entityIndex) {
       const store = new N3Store({ entityIndex: this._entityIndex });
-      store._graphs = intersect(other._graphs, this._graphs);
-      store._size = null;
+      const graphs = intersect(other._graphs, this._graphs);
+      if (graphs) {
+        store._graphs = graphs;
+        store._size = null;
+      }
       return store;
     }
 

diff --git a/test/N3Store-test.js b/test/N3Store-test.js
@@ -2145,6 +2145,13 @@ describe('Store', () => {
             expect(s1.intersection(s2).equals(s2.intersection(s1)));
             expect(s1.union(s2).intersection(s1).equals(s1));
             expect(s1.intersection(s2).union(s1).equals(s1));
+            expect(new Store([...s1.union(s2).intersection(s1)]).equals(new Store([...s1])));
+            expect(new Store([...s1.intersection(s2).union(s1)]).equals(new Store([...s2])));
+
+            const newStore = s1.intersection(s2);
+            const size = newStore.size;
+            newStore.add(new Quad(new NamedNode('mys1'), new NamedNode('myp1'), new NamedNode('myo1')));
+            expect(newStore.size).toBe(size + 1);
           }
         }