Merge pull request #120 from nolanlawson/refactor-tests

Switch WPT tests to `node:test` runner

Author: dumbmatter

package.json

@@ -94,7 +94,7 @@
         "test-mocha": "mocha --timeout 10000 build/esm/test/fakeIndexedDB",
         "build-qunit": "mkdir build/esm/test/indexedDBmock && cp src/test/indexedDBmock/* build/esm/test/indexedDBmock && browserify src/test/indexedDBmock/exports-qunit-bundle.js -o build/esm/test/indexedDBmock/exports-qunit-bundle.js -t [ babelify --presets [ @babel/preset-env ] ]",
         "test-qunit": "pnpm run build-qunit && OPENSSL_CONF=/dev/null node-qunit-phantomjs build/esm/test/indexedDBmock/index.html",
-        "test-w3c": "node src/test/web-platform-tests/run-all.js",
+        "test-w3c": "node --test src/test/web-platform-tests/run-all.js",
         "test": "pnpm run lint && pnpm run build && pnpm run test-jest && node test/test.js && node test/dexie.js && pnpm run test-w3c && pnpm run test-mocha && pnpm run test-qunit",
         "prepare": "husky"
     },

src/test/web-platform-tests/README.md

@@ -134,6 +134,19 @@ const outFolder = path.posix.join(__dirname, "converted");
             codeChunks.push(declareGlobalVars);
         }
 
+        if (testScript.includes("title=")) {
+            console.log("found!");
+        }
+        const titleMatches = [
+            ...testScript.matchAll(/\/\/\s*META:\s*title=(.+)$/gm),
+        ];
+        if (titleMatches.length) {
+            // some tests use `self.title` to create the test name
+            codeChunks.push(
+                `globalThis.title = ${JSON.stringify(titleMatches[0][1])};\n`,
+            );
+        }
+
         const importMatches = testScript
             .matchAll(/^\/\/\s*META:\s*script=(.+)$/gm)
             .filter(

src/test/web-platform-tests/convert.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "IndexedDB";
+
 /* Delete created databases
  *
  * Go through each finished test, see if it has an associated database. Close

src/test/web-platform-tests/converted/abort-in-initial-upgradeneeded.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "IndexedDB: ES bindings - Inject a key into a value - Keys bypass setters";
+
 'use strict';
 
 // Returns an IndexedDB database name that is unique to the test case.

src/test/web-platform-tests/converted/bindings-inject-keys-bypass.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "IndexedDB: ES bindings - Inject a key into a value - Values bypass chain and setters";
+
 'use strict';
 
 // Returns an IndexedDB database name that is unique to the test case.

src/test/web-platform-tests/converted/bindings-inject-values-bypass.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "IDB-backed composite blobs maintain coherency";
+
 'use strict';
 
 // Returns an IndexedDB database name that is unique to the test case.

src/test/web-platform-tests/converted/blob-composite-blob-reads.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "Blob Content Type";
+
 /* Delete created databases
  *
  * Go through each finished test, see if it has an associated database. Close

src/test/web-platform-tests/converted/blob-contenttype.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "Blob Delete Object Store";
+
 /* Delete created databases
  *
  * Go through each finished test, see if it has an associated database. Close

src/test/web-platform-tests/converted/blob-delete-objectstore-db.any.js

@@ -2,6 +2,8 @@ import "../wpt-env.js";
 
 let attrs,cursor,db,store,store2;
 
+globalThis.title = "Blob Valid After Abort";
+
 /* Delete created databases
  *
  * Go through each finished test, see if it has an associated database. Close