summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRich Trott <rtrott@gmail.com>2017-12-23 14:40:49 -0800
committerMyles Borins <mylesborins@google.com>2018-02-12 19:28:26 -0500
commit038e52627fddd331361fa73f848bd3ad3f9c7cf7 (patch)
tree042791e6f0ab19c6414916e174b2b0745fac3c00
parent653c02657845b170f8888f892ec45f25ea534766 (diff)
downloadnode-new-038e52627fddd331361fa73f848bd3ad3f9c7cf7.tar.gz
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: https://github.com/nodejs/node/pull/17845 Reviewed-By: Timothy Gu <timothygu99@gmail.com> Reviewed-By: Anatoli Papirovski <apapirovski@mac.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Weijia Wang <starkwang@126.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
-rw-r--r--test/sequential/test-http-server-consumed-timeout.js4
1 files changed, 3 insertions, 1 deletions
diff --git a/test/sequential/test-http-server-consumed-timeout.js b/test/sequential/test-http-server-consumed-timeout.js
index abe100c190..5621722650 100644
--- 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();