test: s/assert.fail/common.fail as appropriate

Many tests use assert.fail(null, null, msg) where it would be
simpler to use common.fail(msg). This is largely because
common.fail() is fairly new. This commit makes the replacement
when applicable.

PR-URL: #7735
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>

Author: cjihrig

test/internet/test-dgram-send-cb-quelches-error.js

@@ -28,7 +28,7 @@ function callbackOnly(err) {
 }
 
 function onEvent(err) {
-  assert.fail(null, null, 'Error should not be emitted if there is callback');
+  common.fail('Error should not be emitted if there is callback');
 }
 
 function onError(err) {

test/parallel/test-event-emitter-listeners-side-effects.js

@@ -1,6 +1,6 @@
 'use strict';
 
-require('../common');
+const common = require('../common');
 var assert = require('assert');
 
 var EventEmitter = require('events').EventEmitter;
@@ -13,12 +13,12 @@ assert(Array.isArray(fl));
 assert(fl.length === 0);
 assert.deepEqual(e._events, {});
 
-e.on('foo', assert.fail);
+e.on('foo', common.fail);
 fl = e.listeners('foo');
-assert(e._events.foo === assert.fail);
+assert(e._events.foo === common.fail);
 assert(Array.isArray(fl));
 assert(fl.length === 1);
-assert(fl[0] === assert.fail);
+assert(fl[0] === common.fail);
 
 e.listeners('bar');
 assert(!e._events.hasOwnProperty('bar'));
@@ -28,12 +28,12 @@ fl = e.listeners('foo');
 
 assert(Array.isArray(e._events.foo));
 assert(e._events.foo.length === 2);
-assert(e._events.foo[0] === assert.fail);
+assert(e._events.foo[0] === common.fail);
 assert(e._events.foo[1] === assert.ok);
 
 assert(Array.isArray(fl));
 assert(fl.length === 2);
-assert(fl[0] === assert.fail);
+assert(fl[0] === common.fail);
 assert(fl[1] === assert.ok);
 
 console.log('ok');

test/parallel/test-event-emitter-once.js

@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+const common = require('../common');
 var assert = require('assert');
 var events = require('events');
 
@@ -16,7 +16,7 @@ e.emit('hello', 'a', 'b');
 e.emit('hello', 'a', 'b');
 
 var remove = function() {
-  assert.fail(1, 0, 'once->foo should not be emitted', '!');
+  common.fail('once->foo should not be emitted');
 };
 
 e.once('foo', remove);

test/parallel/test-http-client-reject-chunked-with-content-length.js

@@ -18,7 +18,7 @@ server.listen(0, () => {
   // both a Content-Length header and a Transfer-Encoding: chunked
   // header, which is a violation of the HTTP spec.
   const req = http.get({port: server.address().port}, (res) => {
-    assert.fail(null, null, 'callback should not be called');
+    common.fail('callback should not be called');
   });
   req.on('error', common.mustCall((err) => {
     assert(/^Parse Error/.test(err.message));

test/parallel/test-http-client-reject-cr-no-lf.js

@@ -17,7 +17,7 @@ server.listen(0, () => {
   // The callback should not be called because the server is sending a
   // header field that ends only in \r with no following \n
   const req = http.get({port: server.address().port}, (res) => {
-    assert.fail(null, null, 'callback should not be called');
+    common.fail('callback should not be called');
   });
   req.on('error', common.mustCall((err) => {
     assert(/^Parse Error/.test(err.message));

test/parallel/test-http-double-content-length.js

@@ -23,8 +23,7 @@ server.listen(0, () => {
     // Send two content-length header values.
     headers: {'Content-Length': [1, 2]}},
     (res) => {
-      assert.fail(null, null, 'an error should have occurred');
-      server.close();
+      common.fail('an error should have occurred');
     }
   );
   req.on('error', common.mustCall(() => {

test/parallel/test-http-localaddress-bind-error.js

@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+const common = require('../common');
 var assert = require('assert');
 var http = require('http');
 
@@ -24,7 +24,7 @@ server.listen(0, '127.0.0.1', function() {
     method: 'GET',
     localAddress: invalidLocalAddress
   }, function(res) {
-    assert.fail(null, null, 'unexpectedly got response from server');
+    common.fail('unexpectedly got response from server');
   }).on('error', function(e) {
     console.log('client got error: ' + e.message);
     gotError = true;

test/parallel/test-http-response-multi-content-length.js

@@ -19,7 +19,7 @@ const server = http.createServer((req, res) => {
       res.writeHead(200, {'content-length': [1, 2]});
       break;
     default:
-      assert.fail(null, null, 'should never get here');
+      common.fail('should never get here');
   }
   res.end('ok');
 });

test/parallel/test-http-response-splitting.js

@@ -38,7 +38,7 @@ const server = http.createServer((req, res) => {
       }));
       break;
     default:
-      assert.fail(null, null, 'should not get to here.');
+      common.fail('should not get to here.');
   }
   if (count === 3)
     server.close();

test/parallel/test-http-server-reject-chunked-with-content-length.js

@@ -10,7 +10,7 @@ const reqstr = 'POST / HTTP/1.1\r\n' +
                'Transfer-Encoding: chunked\r\n\r\n';
 
 const server = http.createServer((req, res) => {
-  assert.fail(null, null, 'callback should not be invoked');
+  common.fail('callback should not be invoked');
 });
 server.on('clientError', common.mustCall((err) => {
   assert(/^Parse Error/.test(err.message));
@@ -25,7 +25,7 @@ server.listen(0, () => {
   client.on('data', (data) => {
     // Should not get to this point because the server should simply
     // close the connection without returning any data.
-    assert.fail(null, null, 'no data should be returned by the server');
+    common.fail('no data should be returned by the server');
   });
   client.on('end', common.mustCall(() => {}));
 });