diff options
author | Rich Trott <rtrott@gmail.com> | 2017-12-23 14:40:49 -0800 |
---|---|---|
committer | Myles Borins <mylesborins@google.com> | 2018-02-12 19:28:26 -0500 |
commit | 038e52627fddd331361fa73f848bd3ad3f9c7cf7 (patch) | |
tree | 042791e6f0ab19c6414916e174b2b0745fac3c00 | |
parent | 653c02657845b170f8888f892ec45f25ea534766 (diff) | |
download | node-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.js | 4 |
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(); |