Return this on Tag add/delete methods

[Etiquette] / model.test.js

diff --git a/model.test.js b/model.test.js
index 7944238493a1e5c4da4c398252171b6487737c17..7190830c3a043dd200bfecf4ed7d68d924e6b32c 100644 (file)
--- a/model.test.js
+++ b/model.test.js
@@ -306,6 +306,11 @@ describe("TagSystem", () => {
           ],
         );
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.addAltLabel(), tag);
+      });
     });
 
     describe("::addBroaderTag", () => {
@@ -328,6 +333,11 @@ describe("TagSystem", () => {
         );
       });
 
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.addBroaderTag(), tag);
+      });
+
       it("[[Call]] throws when adding a non‐persisted tag", () => {
         const tag = new Tag();
         assertThrows(() => {
@@ -372,6 +382,11 @@ describe("TagSystem", () => {
           ],
         );
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.addHiddenLabel(), tag);
+      });
     });
 
     describe("::addInCanonTag", () => {
@@ -394,6 +409,11 @@ describe("TagSystem", () => {
         );
       });
 
+      it("[[Call]] returns this", () => {
+        const tag = new Tag("EntityTag");
+        assertStrictEquals(tag.addInCanonTag(), tag);
+      });
+
       it("[[Call]] throws when this is not a tag which can be placed in canon", () => {
         assertThrows(() => {
           new Tag().addInCanonTag();
@@ -447,6 +467,11 @@ describe("TagSystem", () => {
         );
       });
 
+      it("[[Call]] returns this", () => {
+        const tag = new Tag("ConceptualTag");
+        assertStrictEquals(tag.addInvolvesTag(), tag);
+      });
+
       it("[[Call]] throws when this is not a conceptual tag", () => {
         assertThrows(() => {
           new Tag().addInvolvesTag();
@@ -547,6 +572,11 @@ describe("TagSystem", () => {
         );
         assertEquals([...tag.altLabels()], ["four"]);
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.deleteAltLabel(), tag);
+      });
     });
 
     describe("::deleteBroaderTag", () => {
@@ -579,6 +609,11 @@ describe("TagSystem", () => {
           [broader2.identifier],
         );
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.deleteBroaderTag(), tag);
+      });
     });
 
     describe("::deleteHiddenLabel", () => {
@@ -605,6 +640,11 @@ describe("TagSystem", () => {
         );
         assertEquals([...tag.hiddenLabels()], ["four"]);
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag();
+        assertStrictEquals(tag.deleteHiddenLabel(), tag);
+      });
     });
 
     describe("::deleteInCanonTag", () => {
@@ -633,6 +673,11 @@ describe("TagSystem", () => {
           [canon2.identifier],
         );
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag("EntityTag");
+        assertStrictEquals(tag.deleteInCanonTag(), tag);
+      });
     });
 
     describe("::deleteInvolvesTag", () => {
@@ -665,6 +710,11 @@ describe("TagSystem", () => {
           [involved2.identifier],
         );
       });
+
+      it("[[Call]] returns this", () => {
+        const tag = new Tag("ConceptualTag");
+        assertStrictEquals(tag.deleteInvolvesTag(), tag);
+      });
     });
 
     describe("::hasInCanonTags", () => {
@@ -849,6 +899,15 @@ describe("TagSystem", () => {
         const activity = tag.persist();
         assertStrictEquals(activity, null);
       });
+
+      it("[[Call]] returns undefined for a silent persist", () => {
+        const broader = new Tag();
+        broader.persist();
+        const tag = new Tag();
+        tag.prefLabel = "etaoin";
+        tag.addBroaderTag(broader);
+        assertStrictEquals(tag.persist(true), undefined);
+      });
     });
 
     describe("::prefLabel", () => {