Skip to content

Commit

Permalink
test: remove message from strictEqual assertions
Browse files Browse the repository at this point in the history
When an AssertionError happens, the value of headers['set-cookie']
is not reported. That information is useful for debugging.
Hence removed the value passed as the message in deepStrictEqual
assertions of test/parallel/test-http2-cookies.js

PR-URL: nodejs#20174
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
  • Loading branch information
bryancr89 authored and kjin committed Aug 30, 2018
1 parent 07dbef4 commit 2d260ad
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions test/parallel/test-http2-cookies.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,7 @@ server.on('listening', common.mustCall(() => {

req.on('response', common.mustCall((headers) => {
assert(Array.isArray(headers['set-cookie']));
assert.deepStrictEqual(headers['set-cookie'], setCookie,
'set-cookie header does not match');
assert.deepStrictEqual(headers['set-cookie'], setCookie);
}));

req.on('end', common.mustCall(() => {
Expand Down

0 comments on commit 2d260ad

Please sign in to comment.