test: remove undefined function

`common.fail()` no longer exists as its functionality is now in `assert.fail()`. Replace only two instances in the code base with `assert.fail()`. PR-URL: #17845 Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Weijia Wang <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]>
nodejs · Jan 9, 2018 · 9110654 · 9110654
1 parent ca35d08
commit 9110654
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/test/parallel/test-timers-max-duration-warning.js b/test/parallel/test-timers-max-duration-warning.js
@@ -7,7 +7,7 @@ const timers = require('timers');
 const OVERFLOW = Math.pow(2, 31); // TIMEOUT_MAX is 2^31-1
 
 function timerNotCanceled() {
-  common.fail('Timer should be canceled');
+  assert.fail('Timer should be canceled');
 }
 
 process.on('warning', common.mustCall((warning) => {

diff --git a/test/sequential/test-http-server-consumed-timeout.js b/test/sequential/test-http-server-consumed-timeout.js
@@ -1,6 +1,8 @@
 'use strict';
 
 const common = require('../common');
+
+const assert = require('assert');
 const http = require('http');
 
 let time = Date.now();
@@ -16,7 +18,7 @@ const server = http.createServer((req, res) => {
   req.setTimeout(TIMEOUT, () => {
     if (!intervalWasInvoked)
       return common.skip('interval was not invoked quickly enough for test');
-    common.fail('Request timeout should not fire');
+    assert.fail('Request timeout should not fire');
   });
 
   req.resume();