test: remove useless require('../common') from WPTs

panva · targos · commit 323415535ba8 · 2023-03-13T15:15:21.000+01:00
PR-URL: #46796 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
diff --git a/test/wpt/.eslintrc.yaml b/test/wpt/.eslintrc.yaml
@@ -0,0 +1,3 @@
+rules:
+  node-core/required-modules: off
+  node-core/require-common-first: off
diff --git a/test/wpt/README.md b/test/wpt/README.md
@@ -46,7 +46,6 @@ For example, for the URL tests, add a file `test/wpt/test-url.js`:
 ```js
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('url');
diff --git a/test/wpt/test-abort.js b/test/wpt/test-abort.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('dom/abort');
diff --git a/test/wpt/test-atob.js b/test/wpt/test-atob.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('html/webappapis/atob');
diff --git a/test/wpt/test-blob.js b/test/wpt/test-blob.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('FileAPI/blob');
diff --git a/test/wpt/test-broadcastchannel.js b/test/wpt/test-broadcastchannel.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('webmessaging/broadcastchannel');
diff --git a/test/wpt/test-console.js b/test/wpt/test-console.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('console');
diff --git a/test/wpt/test-domexception.js b/test/wpt/test-domexception.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('webidl/ecmascript-binding/es-exceptions');
diff --git a/test/wpt/test-encoding.js b/test/wpt/test-encoding.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 const runner = new WPTRunner('encoding');
 
diff --git a/test/wpt/test-events.js b/test/wpt/test-events.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('dom/events');
diff --git a/test/wpt/test-file.js b/test/wpt/test-file.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('FileAPI/file');
diff --git a/test/wpt/test-hr-time.js b/test/wpt/test-hr-time.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('hr-time');
diff --git a/test/wpt/test-microtask-queuing.js b/test/wpt/test-microtask-queuing.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('html/webappapis/microtask-queuing');
diff --git a/test/wpt/test-performance-timeline.js b/test/wpt/test-performance-timeline.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('performance-timeline');
diff --git a/test/wpt/test-resource-timing.js b/test/wpt/test-resource-timing.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('resource-timing');
diff --git a/test/wpt/test-streams.js b/test/wpt/test-streams.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('streams');
diff --git a/test/wpt/test-structured-clone.js b/test/wpt/test-structured-clone.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('html/webappapis/structured-clone');
diff --git a/test/wpt/test-timers.js b/test/wpt/test-timers.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('html/webappapis/timers');
diff --git a/test/wpt/test-url.js b/test/wpt/test-url.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('url');
diff --git a/test/wpt/test-user-timing.js b/test/wpt/test-user-timing.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('user-timing');
diff --git a/test/wpt/test-wasm-webapi.js b/test/wpt/test-wasm-webapi.js
@@ -1,6 +1,5 @@
 'use strict';
 
-require('../common');
 const { WPTRunner } = require('../common/wpt');
 
 const runner = new WPTRunner('wasm/webapi');

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+rules:`
	`2`	`+ node-core/required-modules: off`
	`3`	`+ node-core/require-common-first: off`