diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml index 63d2127d738597..12be376357a81a 100644 --- a/test/.eslintrc.yaml +++ b/test/.eslintrc.yaml @@ -48,9 +48,8 @@ rules: message: "The first argument should be the `actual`, not the `expected` value." - selector: "CallExpression[callee.object.name='assert'][callee.property.name='deepStrictEqual'][arguments.0.type='Literal']:not([arguments.1.type='Literal']):not([arguments.1.type='ObjectExpression']):not([arguments.1.type='ArrayExpression'])" message: "The first argument should be the `actual`, not the `expected` value." - # TODO: Activate the `strictEqual` rule as soon as it produces less churn. - # - selector: "CallExpression[callee.object.name='assert'][callee.property.name='strictEqual'][arguments.0.type='Literal']:not([arguments.1.type='Literal']):not([arguments.1.type='ObjectExpression']):not([arguments.1.type='ArrayExpression'])" - # message: "The first argument should be the `actual`, not the `expected` value." + - selector: "CallExpression[callee.object.name='assert'][callee.property.name='strictEqual'][arguments.0.type='Literal']:not([arguments.1.type='Literal']):not([arguments.1.type='ObjectExpression']):not([arguments.1.type='ArrayExpression'])" + message: "The first argument should be the `actual`, not the `expected` value." # Global scoped methods and vars globals: WebAssembly: false diff --git a/test/parallel/test-http-blank-header.js b/test/parallel/test-http-blank-header.js index 3f2b512fc85567..fd447b46d213a9 100644 --- a/test/parallel/test-http-blank-header.js +++ b/test/parallel/test-http-blank-header.js @@ -28,11 +28,11 @@ const net = require('net'); const server = http.createServer(common.mustCall((req, res) => { assert.strictEqual(req.method, 'GET'); assert.strictEqual(req.url, '/blah'); - assert.deepStrictEqual({ + assert.deepStrictEqual(req.headers, { host: 'example.org:443', origin: 'http://example.org', cookie: '' - }, req.headers); + }); })); @@ -52,7 +52,7 @@ server.listen(0, common.mustCall(() => { received += data.toString(); })); c.on('end', common.mustCall(() => { - assert.strictEqual('HTTP/1.1 400 Bad Request\r\n\r\n', received); + assert.strictEqual(received, 'HTTP/1.1 400 Bad Request\r\n\r\n'); c.end(); })); c.on('close', common.mustCall(() => server.close())); diff --git a/test/parallel/test-process-env.js b/test/parallel/test-process-env.js index 51cc637d06e5c4..81651efbea024d 100644 --- a/test/parallel/test-process-env.js +++ b/test/parallel/test-process-env.js @@ -68,7 +68,7 @@ if (process.argv[2] === 'you-are-the-child') { // delete should return true except for non-configurable properties // https://github.com/nodejs/node/issues/7960 delete process.env.NON_EXISTING_VARIABLE; -assert.strictEqual(true, delete process.env.NON_EXISTING_VARIABLE); +assert(delete process.env.NON_EXISTING_VARIABLE); /* For the moment we are not going to support setting the timezone via the * environment variables. The problem is that various V8 platform backends