Skip to content

Commit c5bf861

Browse files
addaleaxrvagg
authored andcommitted
test: fix common.mustNotCall() usage in HTTP test
The argument to `common.mustNotCall()` is a message, not a function. PR-URL: #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>
1 parent daaf3a4 commit c5bf861

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/sequential/test-http-max-http-headers.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ function test1() {
9696

9797
server.listen(0, common.mustCall(() => {
9898
const port = server.address().port;
99-
const client = http.get({ port: port }, common.mustNotCall(() => {}));
99+
const client = http.get({ port: port }, common.mustNotCall());
100100

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

0 commit comments

Comments
 (0)