Skip to content

Commit 82b59a0

Browse files
Trottrefack
authored andcommitted
test: remove unnecessary eslint-disable comments
PR-URL: nodejs#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 7414928 commit 82b59a0

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
@@ -22,8 +22,6 @@
2222

2323
'use strict';
2424

25-
/* eslint-disable node-core/prefer-common-expectserror */
26-
2725
const common = require('../common');
2826
const assert = require('assert');
2927
const { inspect } = require('util');
@@ -664,7 +662,6 @@ common.expectsError(
664662
const tmp = fs.close;
665663
fs.close = common.mustCall(tmp, 1);
666664
function throwErr() {
667-
// eslint-disable-next-line prefer-assert-methods
668665
assert(
669666
(Buffer.from('test') instanceof Error)
670667
);

0 commit comments

Comments
 (0)