test: remove eslint-disable comments from fixtures

We do not lint the fixtures code so eslint-disable comments are superfluous. PR-URL: nodejs#41859 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Mestery <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
danielleadams · Mar 4, 2022 · 7077e5b · 7077e5b
1 parent 50d6ff3
commit 7077e5b
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 8 deletions.
diff --git a/test/fixtures/es-module-loaders/loader-invalid-url.mjs b/test/fixtures/es-module-loaders/loader-invalid-url.mjs
@@ -1,4 +1,3 @@
-/* eslint-disable node-core/required-modules */
 export async function resolve(specifier, { parentURL, importAssertions }, defaultResolve) {
   if (parentURL && specifier === '../fixtures/es-modules/test-esm-ok.mjs') {
     return {

diff --git a/test/fixtures/es-module-loaders/throw-undefined.mjs b/test/fixtures/es-module-loaders/throw-undefined.mjs
@@ -1,4 +1,3 @@
 'use strict';
-/* eslint-disable node-core/required-modules */
 
 throw undefined;
diff --git a/test/fixtures/es-modules/import-json-named-export.mjs b/test/fixtures/es-modules/import-json-named-export.mjs
@@ -1,2 +1 @@
-/* eslint-disable no-unused-vars */
 import { ofLife } from '../experimental.json' assert { type: 'json' };
diff --git a/test/fixtures/wpt/streams/readable-streams/general.any.js b/test/fixtures/wpt/streams/readable-streams/general.any.js
@@ -630,7 +630,6 @@ promise_test(() => {
   let pullCalled = 0;
   let cancelCalled = 0;
 
-  /* eslint-disable no-use-before-define */
   class Source {
     start(c) {
       startCalled++;

diff --git a/test/fixtures/wpt/streams/readable-streams/patched-global.any.js b/test/fixtures/wpt/streams/readable-streams/patched-global.any.js
@@ -22,7 +22,6 @@ test(t => {
 
   const trappedProperties = ['highWaterMark', 'size', 'start', 'type', 'mode'];
   for (const property of trappedProperties) {
-    // eslint-disable-next-line no-extend-native, accessor-pairs
     Object.defineProperty(Object.prototype, property, {
       get() { throw new Error(`${property} getter called`); },
       configurable: true

diff --git a/test/fixtures/wpt/streams/resources/test-utils.js b/test/fixtures/wpt/streams/resources/test-utils.js
@@ -57,7 +57,6 @@ self.garbageCollect = () => {
     // Present in some WebKit development environments
     GCController.collect();
   } else {
-    /* eslint-disable no-console */
     console.warn('Tests are running without the ability to do manual garbage collection. They will still work, but ' +
       'coverage will be suboptimal.');
     /* eslint-enable no-console */

diff --git a/test/fixtures/wpt/streams/transform-streams/patched-global.any.js b/test/fixtures/wpt/streams/transform-streams/patched-global.any.js
@@ -5,13 +5,11 @@
 // interfering with other tests.
 
 test(t => {
-  // eslint-disable-next-line no-extend-native, accessor-pairs
   Object.defineProperty(Object.prototype, 'highWaterMark', {
     set() { throw new Error('highWaterMark setter called'); },
     configurable: true
   });
 
-  // eslint-disable-next-line no-extend-native, accessor-pairs
   Object.defineProperty(Object.prototype, 'size', {
     set() { throw new Error('size setter called'); },
     configurable: true