test: fix uses of deprecated assert.fail with multiple args

PR-URL: #23673 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Refael Ackermann <refack@gmail.com>
nodejs · Nov 13, 2018 · 9af7ad5 · 9af7ad5
1 parent 2b0410a
commit 9af7ad5
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 6 deletions.
diff --git a/test/async-hooks/init-hooks.js b/test/async-hooks/init-hooks.js
@@ -120,8 +120,7 @@ class ActivityCollector {
     }
     if (violations.length) {
       console.error(violations.join('\n\n') + '\n');
-      assert.fail(violations.length, 0,
-                  `${violations.length} failed sanity checks`);
+      assert.fail(`${violations.length} failed sanity checks`);
     }
   }
 

diff --git a/test/parallel/test-net-connect-options-fd.js b/test/parallel/test-net-connect-options-fd.js
@@ -70,7 +70,7 @@ const forAllClients = (cb) => common.mustCall(cb, CLIENT_VARIANTS);
   })
   .on('error', function(err) {
     console.error(err);
-    assert.fail(null, null, `[Pipe server]${err}`);
+    assert.fail(`[Pipe server]${err}`);
   })
   .listen({ path: serverPath }, common.mustCall(function serverOnListen() {
     const getSocketOpt = (index) => {
@@ -94,7 +94,7 @@ const forAllClients = (cb) => common.mustCall(cb, CLIENT_VARIANTS);
       console.error(`[Pipe]Sending data through fd ${oldHandle.fd}`);
       this.on('error', function(err) {
         console.error(err);
-        assert.fail(null, null, `[Pipe Client]${err}`);
+        assert.fail(`[Pipe Client]${err}`);
       });
     });
 

diff --git a/test/parallel/test-string-decoder.js b/test/parallel/test-string-decoder.js
@@ -228,7 +228,7 @@ function test(encoding, input, expected, singleSequence) {
         `input: ${input.toString('hex').match(hexNumberRE)}\n` +
         `Write sequence: ${JSON.stringify(sequence)}\n` +
         `Full Decoder State: ${inspect(decoder)}`;
-      assert.fail(output, expected, message);
+      assert.fail(message);
     }
   });
 }

diff --git a/test/pseudo-tty/test-tty-get-color-depth.js b/test/pseudo-tty/test-tty-get-color-depth.js
@@ -2,7 +2,6 @@
 
 const common = require('../common');
 const assert = require('assert').strict;
-/* eslint-disable no-restricted-properties */
 const { WriteStream } = require('tty');
 
 const fd = common.getTTYfd();