From 63b06b55d75ce5f983dbfbe3b0b310201c700e3e 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 5ca27a0d17fc79..fad7f7d1d6b3c3 100644 --- a/test/sequential/test-http-max-http-headers.js +++ b/test/sequential/test-http-max-http-headers.js @@ -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');