Skip to content

Commit

Permalink
test: fix common.mustNotCall() usage in HTTP test
Browse files Browse the repository at this point in the history
The argument to `common.mustNotCall()` is a message, not a function.

PR-URL: nodejs#24750
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
Anna Henningsen authored and refack committed Jan 10, 2019
1 parent 654a0c9 commit 3a13b4a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/sequential/test-http-max-http-headers.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ function test1() {

server.listen(0, common.mustCall(() => {
const port = server.address().port;
const client = http.get({ port: port }, common.mustNotCall(() => {}));
const client = http.get({ port: port }, common.mustNotCall());

client.on('error', common.mustCall((err) => {
assert.strictEqual(err.code, 'HPE_HEADER_OVERFLOW');
Expand Down

0 comments on commit 3a13b4a

Please sign in to comment.