Skip to content

Commit b441573

Browse files
jonask-wixcjihrig
authored andcommitted
test: use default assertion message
In test-child-process-spawnsync, the assert.strictEqual() custom message was hiding information about why the test has failed. It just showed what value is expected and in case of failure we want to know which value has caused test to fail. PR-URL: #16819 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Rich Trott <[email protected]>
1 parent 978aa84 commit b441573

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/parallel/test-child-process-spawnsync.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ const spawnSync = require('child_process').spawnSync;
2828
// Echo does different things on Windows and Unix, but in both cases, it does
2929
// more-or-less nothing if there are no parameters
3030
const ret = spawnSync('sleep', ['0']);
31-
assert.strictEqual(ret.status, 0, 'exit status should be zero');
31+
assert.strictEqual(ret.status, 0);
3232

3333
// Error test when command does not exist
3434
const ret_err = spawnSync('command_does_not_exist', ['bar']).error;

0 commit comments

Comments
 (0)