@7365admin1/module-hygiene 4.29.1-staging.13 → 4.29.1-staging.15

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@7365admin1/module-hygiene",
   "license": "MIT",
-  "version": "4.29.1-staging.13",
+  "version": "4.29.1-staging.15",
   "author": "7365admin1",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -13,7 +13,7 @@
     "build": "tsup src/index.ts --format cjs,esm --dts",
     "release": "yarn run build && changeset publish",
     "lint": "tsc",
-    "test": "tsup src/utils/hygiene-dashboard-metrics.util.ts src/utils/hygiene-checkout-decision.util.ts src/utils/completion-photo.util.ts --format esm --out-dir test/.build --no-dts --silent && node --test \"test/*.test.mjs\""
+    "test": "tsup src/utils/hygiene-dashboard-metrics.util.ts src/utils/hygiene-checkout-decision.util.ts src/utils/completion-photo.util.ts src/utils/area-unit.util.ts src/utils/escape-regex.util.ts --format esm --out-dir test/.build --no-dts --silent && node --test \"test/*.test.mjs\""
   },
   "devDependencies": {
     "@changesets/cli": "^2.26.0",

package/test/area-import-unit-id.test.mjs

@@ -0,0 +1,18 @@
+import { test } from "node:test";
+import assert from "node:assert/strict";
+import { ObjectId } from "mongodb";
+import { areaUnitSchema, toAreaUnit } from "./.build/area-unit.util.mjs";
+
+// The area import found (or created) each unit and passed its ObjectId to the
+// area schema, which only takes a hex string - so every imported area that
+// listed units was refused. toAreaUnit gives the schema what it takes, and the
+// area model validates units with this same areaUnitSchema.
+test("an imported area's units pass the area's unit schema", () => {
+  const found = [{ _id: new ObjectId(), name: "Toilet Bowl" }, { _id: new ObjectId().toString(), name: "Sink" }];
+  for (const unit of found) assert.equal(areaUnitSchema.validate(toAreaUnit(unit)).error, undefined);
+  assert.deepEqual(found.map(toAreaUnit).map((u) => u.unit), found.map((f) => String(f._id)));
+});
+
+test("the ObjectId the import used to pass is what the schema refused", () => {
+  assert.match(String(areaUnitSchema.validate({ unit: new ObjectId(), name: "Floor" }).error?.message), /must be a string/);
+});

package/test/escape-regex.util.test.mjs

@@ -0,0 +1,26 @@
+import { test } from "node:test";
+import assert from "node:assert/strict";
+import { readFileSync, readdirSync } from "node:fs";
+import { escapeRegex } from "./.build/escape-regex.util.mjs";
+
+test("search text is matched literally, not run as a pattern", () => {
+  for (const typed of [".*", "(a+)+$", "[", "a|b", "^x$", "\\", "50% off (new)"]) {
+    const rx = new RegExp(escapeRegex(typed), "i");
+    assert.ok(rx.test(`xx${typed}yy`), `${typed} should match itself`);
+  }
+  assert.equal(new RegExp(escapeRegex(".*"), "i").test("anything"), false);
+});
+
+test("missing or non-string search stays harmless", () => {
+  assert.equal(escapeRegex(undefined), "");
+  assert.equal(escapeRegex(null), "");
+  assert.equal(escapeRegex(42), "42");
+});
+
+test("every repository $regex on a search box goes through escapeRegex", () => {
+  const dir = new URL("../src/repositories/", import.meta.url);
+  for (const f of readdirSync(dir)) {
+    const src = readFileSync(new URL(f, dir), "utf8");
+    assert.doesNotMatch(src, /\$regex:\s*search\b/, `${f} passes raw search into $regex`);
+  }
+});