From 5abe8c06937ea398ecbc5d785eaa4ef8e56f95ec Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 3 Jul 2017 14:00:41 -0700 Subject: [PATCH] test: restore no-op function in test Remove common.mustCall() in test that might connect to a server already running on the local host. PR-URL: https://github.com/nodejs/node/pull/14065 Reviewed-By: Anna Henningsen Reviewed-By: Refael Ackermann Reviewed-By: Yuta Hiroto Reviewed-By: Luigi Pinca --- test/parallel/test-http-hostname-typechecking.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/parallel/test-http-hostname-typechecking.js b/test/parallel/test-http-hostname-typechecking.js index d1a407cf28dcae..74813e0582d853 100644 --- a/test/parallel/test-http-hostname-typechecking.js +++ b/test/parallel/test-http-hostname-typechecking.js @@ -1,5 +1,5 @@ 'use strict'; -const common = require('../common'); +require('../common'); const assert = require('assert'); const http = require('http'); @@ -21,7 +21,7 @@ vals.forEach((v) => { // These values are OK and should not throw synchronously ['', undefined, null].forEach((v) => { assert.doesNotThrow(() => { - http.request({hostname: v}).on('error', common.mustCall()).end(); - http.request({host: v}).on('error', common.mustCall()).end(); + http.request({hostname: v}).on('error', () => {}).end(); + http.request({host: v}).on('error', () => {}).end(); }); });