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 · Oct 15, 2018 · 4054c24cac5cad258ec9ef2d1703d51bbbf9c84a · 4054c24
1 parent c417c7a
commit 4054c24cac5cad258ec9ef2d1703d51bbbf9c84a
Unified Split

Showing with 4 additions and 6 deletions.

+1 −2 test/async-hooks/init-hooks.js

+2 −2 test/parallel/test-net-connect-options-fd.js

+1 −1 test/parallel/test-string-decoder.js

+0 −1 test/pseudo-tty/test-tty-get-color-depth.js
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
@@ -68,7 +68,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) => {
@@ -92,7 +92,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
@@ -217,7 +217,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();