test: skip url WPT subtests instead of modifying test script

panva · aduh95 · commit e04e2c9ac145 · 2026-05-07T15:41:36.000+02:00
Signed-off-by: Filip Skokan <panva.ip@gmail.com> PR-URL: #62635 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Matthew Aitken <maitken033380023@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/wpt/status/url.cjs b/test/wpt/status/url.cjs
@@ -0,0 +1,25 @@
+'use strict';
+
+module.exports = {
+  'historical.any.js': {
+    'fail': {
+      'expected': [
+        'searchParams on location object',
+      ],
+    },
+  },
+  'javascript-urls.window.js': {
+    'skip': 'requires document.body reference',
+  },
+  'percent-encoding.window.js': {
+    'skip': 'requires document.body reference',
+  },
+  'toascii.window.js': {
+    'skipTests': [
+      /\(using <a(rea)?>/,
+    ],
+  },
+  'url-setters-a-area.window.js': {
+    'skip': 'already tested in url-setters.any.js',
+  },
+};
diff --git a/test/wpt/status/url.json b/test/wpt/status/url.json
diff --git a/test/wpt/test-url.js b/test/wpt/test-url.js
@@ -10,13 +10,5 @@ const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('url');
 
-runner.setScriptModifier((obj) => {
-  if (obj.filename.includes('toascii.window.js')) {
-    // `a` and `area` in `toascii.window.js` is for testing `Element` that
-    // created via `document.createElement`. So we need to ignore them and just
-    // test `URL`.
-    obj.code = obj.code.replace(/\["url", "a", "area"\]/, '[ "url" ]');
-  }
-});
 runner.pretendGlobalThisAs('Window');
 runner.runJsTests();
