Skip to content

Commit 175f7b6

Browse files
TrottMylesBorins
authored andcommitted
test: remove unnecessary eslint-disable comments
PR-URL: #25119 Refs: https://eslint.org/docs/user-guide/command-line-interface#--report-unused-disable-directives Reviewed-By: Teddy Katz <teddy.katz@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
1 parent 6a690ee commit 175f7b6

File tree

4 files changed

+1
-6
lines changed

4 files changed

+1
-6
lines changed

test/common/wpt.js

-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
/* eslint-disable node-core/required-modules */
21
'use strict';
32

43
const assert = require('assert');

test/js-native-api/test_general/testNapiRun.js

-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ const common = require('../../common');
44
const assert = require('assert');
55

66
// `addon` is referenced through the eval expression in testFile
7-
// eslint-disable-next-line no-unused-vars
87
const addon = require(`./build/${common.buildType}/test_general`);
98

109
const testCase = '(41.92 + 0.08);';

test/parallel/test-assert-deep.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ function re(literals, ...values) {
4242
// That is why we discourage using deepEqual in our own tests.
4343

4444
// Turn off no-restricted-properties because we are testing deepEqual!
45-
/* eslint-disable no-restricted-properties, prefer-common-expectserror */
45+
/* eslint-disable no-restricted-properties */
4646

4747
const arr = new Uint8Array([120, 121, 122, 10]);
4848
const buf = Buffer.from(arr);

test/parallel/test-assert.js

-3
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,6 @@
2121

2222
'use strict';
2323

24-
/* eslint-disable node-core/prefer-common-expectserror */
25-
2624
const common = require('../common');
2725
const assert = require('assert');
2826
const { inspect } = require('util');
@@ -662,7 +660,6 @@ common.expectsError(
662660
const tmp = fs.close;
663661
fs.close = common.mustCall(tmp, 1);
664662
function throwErr() {
665-
// eslint-disable-next-line prefer-assert-methods
666663
assert(
667664
(Buffer.from('test') instanceof Error)
668665
);

0 commit comments

Comments
 (0)