diff options
author | Rich Trott <rtrott@gmail.com> | 2015-10-14 21:22:55 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2015-10-16 00:31:04 -0700 |
commit | 676e61872f54dd546e324599c7871c20b798386a (patch) | |
tree | 43b073ac234e3e66882f50cfa5337baea001b012 /test/parallel/test-stream2-base64-single-char-read-end.js | |
parent | 0140e1b5e39342f87133f7f42e9b49a702f69b39 (diff) | |
download | android-node-v8-676e61872f54dd546e324599c7871c20b798386a.tar.gz android-node-v8-676e61872f54dd546e324599c7871c20b798386a.tar.bz2 android-node-v8-676e61872f54dd546e324599c7871c20b798386a.zip |
test: apply correct assert.fail() arguments
The assert.fail function signature has the message as the third argument
but, understandably, it is often assumed that it is the first argument
(or at least the first argument if no other arguments are passed).
This corrects the assert.fail() invocations in the Node.js tests.
Before:
assert.fail('message');
// result: AssertionError: 'message' undefined undefined
After:
assert.fail(null, null, 'message');
// result: AssertionError: message
PR-URL: https://github.com/nodejs/node/pull/3378
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'test/parallel/test-stream2-base64-single-char-read-end.js')
-rw-r--r-- | test/parallel/test-stream2-base64-single-char-read-end.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-stream2-base64-single-char-read-end.js b/test/parallel/test-stream2-base64-single-char-read-end.js index e50ea5a0cc..2d60877de8 100644 --- a/test/parallel/test-stream2-base64-single-char-read-end.js +++ b/test/parallel/test-stream2-base64-single-char-read-end.js @@ -33,5 +33,5 @@ src.on('end', function() { src.pipe(dst); timeout = setTimeout(function() { - assert.fail('timed out waiting for _write'); + assert.fail(null, null, 'timed out waiting for _write'); }, 100); |