test: remove lint rule for setTimeout() arguments

With the introduction of the promises API for setTimeout(), the
requirement that it have two parameters may not be sensible anymore in
tests.

PR-URL: #41901
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: Trott

test/.eslintrc.yaml

@@ -31,8 +31,6 @@ rules:
       message: "Use an object as second argument of `assert.throws()`."
     - selector: "CallExpression:matches([callee.name='throws'], [callee.property.name='throws'])[arguments.length<2]"
       message: "`assert.throws()` must be invoked with at least two arguments."
-    - selector: "CallExpression[callee.name='setTimeout'][arguments.length<2]"
-      message: "`setTimeout()` must be invoked with at least two arguments."
     - selector: "CallExpression[callee.name='setInterval'][arguments.length<2]"
       message: "`setInterval()` must be invoked with at least two arguments."
     - selector: "ThrowStatement > CallExpression[callee.name=/Error$/]"

test/parallel/test-stream-construct-async-error.js

@@ -12,7 +12,6 @@ const assert = require('assert');
 {
   class Foo extends Duplex {
     async _destroy(err, cb) {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
       throw new Error('boom');
     }
@@ -31,7 +30,6 @@ const assert = require('assert');
 {
   class Foo extends Duplex {
     async _destroy(err, cb) {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
     }
   }
@@ -46,7 +44,6 @@ const assert = require('assert');
 {
   class Foo extends Duplex {
     async _construct() {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
     }
 
@@ -64,7 +61,6 @@ const assert = require('assert');
 {
   class Foo extends Duplex {
     async _construct(callback) {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
       callback();
     }
@@ -88,7 +84,6 @@ const assert = require('assert');
     });
 
     async _final() {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
     }
   }
@@ -105,7 +100,6 @@ const assert = require('assert');
     });
 
     async _final(callback) {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
       callback();
     }
@@ -123,7 +117,6 @@ const assert = require('assert');
     });
 
     async _final() {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
       throw new Error('boom');
     }

test/parallel/test-stream-writable-final-async.js

@@ -9,7 +9,6 @@ const { setTimeout } = require('timers/promises');
 {
   class Foo extends Duplex {
     async _final(callback) {
-      // eslint-disable-next-line no-restricted-syntax
       await setTimeout(common.platformTimeout(1));
       callback();
     }