diff options
author | cjihrig <cjihrig@gmail.com> | 2016-07-14 11:26:35 -0400 |
---|---|---|
committer | cjihrig <cjihrig@gmail.com> | 2016-07-15 15:50:01 -0400 |
commit | 6510eb5ddc802dcd25713c2b3cbb82711c94d075 (patch) | |
tree | 9f22bb1adff8ef2474267297873f03f1c3704d49 /test/parallel/test-http-client-reject-cr-no-lf.js | |
parent | 28d9485c25a0ddeb8817525e25d4c7bb1ee5013b (diff) | |
download | android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.tar.gz android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.tar.bz2 android-node-v8-6510eb5ddc802dcd25713c2b3cbb82711c94d075.zip |
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: https://github.com/nodejs/node/pull/7735
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Diffstat (limited to 'test/parallel/test-http-client-reject-cr-no-lf.js')
-rw-r--r-- | test/parallel/test-http-client-reject-cr-no-lf.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-http-client-reject-cr-no-lf.js b/test/parallel/test-http-client-reject-cr-no-lf.js index e396637aaa..4cabfa0107 100644 --- a/test/parallel/test-http-client-reject-cr-no-lf.js +++ b/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)); |