fix: removed blank node filter from internal changelog to allow thing…

…s with blank nodes
inrupt · Apr 12, 2022 · 580fd39 · 580fd39
1 parent a2e1f01
commit 580fd39
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 17 deletions.
diff --git a/src/thing/thing.internal.ts b/src/thing/thing.internal.ts
@@ -117,23 +117,21 @@ export function internal_addAdditionsToChangeLog<Dataset extends SolidDataset>(
     : /* istanbul ignore next: This function always gets called after addDeletionsToChangeLog, so the ChangeLog always already exists in tests: */
       { additions: [], deletions: [] };
 
-  const [newAdditions, newDeletions] = additions
-    .filter((addition) => !containsBlankNode(addition))
-    .reduce(
-      ([additionsAcc, deletionsAcc], addition) => {
-        const existingDeletion = deletionsAcc.find((deletion) =>
-          deletion.equals(addition)
-        );
-        if (typeof existingDeletion !== "undefined") {
-          return [
-            additionsAcc,
-            deletionsAcc.filter((deletion) => !deletion.equals(addition)),
-          ];
-        }
-        return [additionsAcc.concat(addition), deletionsAcc];
-      },
-      [changeLog.additions, changeLog.deletions]
-    );
+  const [newAdditions, newDeletions] = additions.reduce(
+    ([additionsAcc, deletionsAcc], addition) => {
+      const existingDeletion = deletionsAcc.find((deletion) =>
+        deletion.equals(addition)
+      );
+      if (typeof existingDeletion !== "undefined") {
+        return [
+          additionsAcc,
+          deletionsAcc.filter((deletion) => !deletion.equals(addition)),
+        ];
+      }
+      return [additionsAcc.concat(addition), deletionsAcc];
+    },
+    [changeLog.additions, changeLog.deletions]
+  );
 
   return freeze({
     ...solidDataset,

diff --git a/src/thing/thing.test.ts b/src/thing/thing.test.ts
@@ -407,6 +407,7 @@ describe("getThingAll", () => {
 describe("setThing", () => {
   const mockThing1Iri = "https://some.vocab/subject1";
   const mockThing2Iri = "https://some.vocab/subject2";
+  const mockThing3Iri = "https://some.vocab/subject3";
   const mockThing1: ThingPersisted = {
     type: "Subject",
     url: mockThing1Iri,
@@ -425,6 +426,22 @@ describe("setThing", () => {
       },
     },
   };
+  const mockThing3: ThingPersisted = {
+    type: "Subject",
+    url: mockThing3Iri,
+    predicates: {
+      ["https://arbitrary.vocab/predicate"]: {
+        namedNodes: ["https://arbitrary.vocab/object"],
+        blankNodes: [
+          {
+            ["https://arbitrary.vocab/blanknode/predicate"]: {
+              namedNodes: ["https://arbitrary.vocab/blanknode/object"],
+            },
+          },
+        ],
+      },
+    },
+  };
   function getMockDataset(things = [mockThing1, mockThing2]): SolidDataset {
     const solidDataset: SolidDataset = {
       type: "Dataset",
@@ -576,6 +593,12 @@ describe("setThing", () => {
       getThing(updatedDataset, "https://some.pod/resource#subjectName")
     ).toStrictEqual(originalThing);
   });
+
+  it("will create blank nodes including in a Thing", () => {
+    const dataset = getMockDataset([]);
+    const updatedDataset = setThing(dataset, mockThing3);
+    expect(updatedDataset.internal_changeLog.additions).not.toHaveLength(1);
+  });
 });
 
 describe("removeThing", () => {