test: fix common.mustNotCall() usage in HTTP test

addaleax · BethGriggs · commit 546fc68ae485 · 2019-02-12T16:52:45.000Z
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>
diff --git a/test/sequential/test-http-max-http-headers.js b/test/sequential/test-http-max-http-headers.js
@@ -96,7 +96,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');

-Original file line number
+Diff line change
 server.listen(0,common.mustCall(()=>{
 constport=server.address().port;
 -constclient=http.get({port: port},common.mustNotCall(()=>{}));
 +constclient=http.get({port: port},common.mustNotCall());
 client.on('error',common.mustCall((err)=>{
 assert.strictEqual(err.code,'HPE_HEADER_OVERFLOW');