From 9ff1abf8d9a161fa17a20bde42fa33fe7e555a73 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Fri, 30 Nov 2018 14:56:11 +0100 Subject: [PATCH] test: fix `common.mustNotCall()` usage in HTTP test The argument to `common.mustNotCall()` is a message, not a function. PR-URL: https://github.com/nodejs/node/pull/24750 Reviewed-By: Colin Ihrig Reviewed-By: Luigi Pinca Reviewed-By: Richard Lau Reviewed-By: Anto Aravinth Reviewed-By: Rich Trott --- test/sequential/test-http-max-http-headers.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/sequential/test-http-max-http-headers.js b/test/sequential/test-http-max-http-headers.js index 507fcd388c1319..f38ba7d63a6645 100644 --- 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');