mirror of
https://github.com/nodejs/node.git
synced 2025-12-28 16:07:39 +00:00
test: remove eslint-disable comments from fixtures
We do not lint the fixtures code so eslint-disable comments are superfluous. PR-URL: https://github.com/nodejs/node/pull/41859 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Mestery <mestery@protonmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
parent
4e18f472c7
commit
fa05f805ab
@ -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 {
|
||||
|
||||
@ -1,4 +1,3 @@
|
||||
'use strict';
|
||||
/* eslint-disable node-core/required-modules */
|
||||
|
||||
throw undefined;
|
||||
|
||||
@ -1,2 +1 @@
|
||||
/* eslint-disable no-unused-vars */
|
||||
import { ofLife } from '../experimental.json' assert { type: 'json' };
|
||||
|
||||
@ -630,7 +630,6 @@ promise_test(() => {
|
||||
let pullCalled = 0;
|
||||
let cancelCalled = 0;
|
||||
|
||||
/* eslint-disable no-use-before-define */
|
||||
class Source {
|
||||
start(c) {
|
||||
startCalled++;
|
||||
|
||||
@ -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
|
||||
|
||||
@ -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 */
|
||||
|
||||
@ -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
|
||||
|
||||
Loading…
Reference in New Issue
Block a user