fail ci if we fall short of 100% coverage, and rename some npm tasks.

Author: botandrose-machine

.github/workflows/ci.yml

@@ -31,7 +31,21 @@ jobs:
       - name: Install dependencies
         run: npm install
       - name: Run tests
-        run: npm run test-move-before
+        run: npm run test:move-before
+
+  coverage:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      - name: Use Node.js
+        uses: actions/setup-node@v4
+        with:
+          cache: 'npm'
+      - name: Install dependencies
+        run: npm install
+      - name: Check code coverage
+        run: npm run test:coverage
 
   typecheck:
     runs-on: ubuntu-latest

TESTING.md

@@ -27,7 +27,7 @@ This will run the tests using Playwright’s headless browser setup across Chrom
 
 To run all tests against Chrome with experimental `moveBefore` support added, execute:
 ```bash
-npm run test-move-before
+npm run test:move-before
 ```
 This will start headless Chrome in a new profile with the `atomic-move` experimental flag set. This runs in a separate job in CI.
 
@@ -50,7 +50,7 @@ This runs all the tests in the browser using Mocha instead of web-test-runner fo
 
 If you really want to open web-test-runner in headed mode, you can run:
 ```bash
-npm run debug
+npm run test:debug
 ```
 This will start the server, and open the test runner in a browser. From there you can choose a test file to run.
 

package-lock.json

@@ -20,8 +20,9 @@
   "unpkg": "dist/idiomorph.min.js",
   "scripts": {
     "test": "web-test-runner",
-    "debug": "web-test-runner --manual --open",
-    "test-move-before": "USE_MOVE_BEFORE=1 web-test-runner",
+    "test:coverage": "web-test-runner && node test/lib/ensure-full-coverage.js",
+    "test:debug": "web-test-runner --manual --open",
+    "test:move-before": "USE_MOVE_BEFORE=1 web-test-runner",
     "ci": "web-test-runner --fail-only --playwright --browsers chromium firefox webkit",
     "perf": "node perf/runner.js",
     "amd": "(echo \"define(() => {\n\" && cat src/idiomorph.js && echo \"\nreturn Idiomorph});\") > dist/idiomorph.amd.js",
@@ -46,6 +47,7 @@
     "chromedriver": "^131.0.5",
     "fs-extra": "^9.1.0",
     "htmx.org": "1.9.9",
+    "lcov-parse": "^1.0.0",
     "mocha": "^11.0.1",
     "prettier": "^3.4.2",
     "sinon": "^9.2.4",

package.json

@@ -0,0 +1,16 @@
+const lcovParse = require("lcov-parse");
+
+lcovParse("coverage/lcov.info", (err, data) => {
+  if (err) {
+    console.error("Error parsing lcov file:", err);
+    process.exit(1);
+  }
+
+  data.forEach((record) => {
+    ["lines", "functions", "branches"].forEach((type) => {
+      if(record[type].hit !== record[type].found) {
+        process.exit(1);
+      }
+    });
+  });
+});