From 4054c24cac5cad258ec9ef2d1703d51bbbf9c84a Mon Sep 17 00:00:00 2001
From: "ivan.filenko" <filenko@uchi.ru>
Date: Mon, 15 Oct 2018 20:32:30 +0300
Subject: [PATCH] test: fix uses of deprecated assert.fail with multiple args

PR-URL: https://github.com/nodejs/node/pull/23673
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Refael Ackermann <refack@gmail.com>
---
 test/async-hooks/init-hooks.js               | 3 +--
 test/parallel/test-net-connect-options-fd.js | 4 ++--
 test/parallel/test-string-decoder.js         | 2 +-
 test/pseudo-tty/test-tty-get-color-depth.js  | 1 -
 4 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/test/async-hooks/init-hooks.js b/test/async-hooks/init-hooks.js
index 14969d8e753b68..817b2db0c781b4 100644
--- 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
index 76a5e30755b15c..8a53c3df928b25 100644
--- 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
index c4607672b0420c..37902c5ccbc5fd 100644
--- 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
index e735f4c286aeb3..d4062f5fdb6d39 100644
--- 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();