diff options
author | Rich Trott <rtrott@gmail.com> | 2018-09-01 22:47:37 -0700 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2018-09-16 20:37:28 -0700 |
commit | bb898ca39de0c4f1c2ff62ce59c5315078c0856a (patch) | |
tree | 9cbb6dda051fa9bfe0b3a00cf620f89e101acfdb /test/sequential | |
parent | 2b5320130bf3aeb2f57e79eb95cb69e39d2d6da8 (diff) | |
download | node-new-bb898ca39de0c4f1c2ff62ce59c5315078c0856a.tar.gz |
test: improve assertion in test-inspector.js
Remove an unecessary string literal from assert.strictEqual() call in
test-inspector.js. The string literal is printed instead of the value
that causes an error. Removing the string literal allows the value that
caused the error to be printed. This improves the troubleshooting
experience when the test fails due to that assertion.
PR-URL: https://github.com/nodejs/node/pull/22849
Reviewed-By: Teddy Katz <teddy.katz@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'test/sequential')
-rw-r--r-- | test/sequential/test-inspector.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/test/sequential/test-inspector.js b/test/sequential/test-inspector.js index 5b8921b4ee..2a7a2ef4bf 100644 --- a/test/sequential/test-inspector.js +++ b/test/sequential/test-inspector.js @@ -33,8 +33,7 @@ function checkBadPath(err) { } function checkException(message) { - assert.strictEqual(message.exceptionDetails, undefined, - 'An exception occurred during execution'); + assert.strictEqual(message.exceptionDetails, undefined); } function assertScopeValues({ result }, expected) { |