1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: remove unused function arguments in async-hooks tests

Browse Source 
Remove unused function arguments in two async-hooks tests.

PR-URL: https://github.com/nodejs/node/pull/24368
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
This commit is contained in:
Rich Trott 2018-11-14 11:09:57 -08:00
parent e1aa7301b4
commit b7e9804c90
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/async-hooks/test-getaddrinforeqwrap.js
View File

@ -14,7 +14,7 @@ const hooks = initHooks();
hooks.enable();
dns.lookup('www.google.com', 4, common.mustCall(onlookup));
function onlookup(err_, ip, family) {
function onlookup() {
// we don't care about the error here in order to allow
// tests to run offline (lookup will fail in that case and the err be set);

2
test/async-hooks/test-getnameinforeqwrap.js
View File

@ -14,7 +14,7 @@ const hooks = initHooks();
hooks.enable();
dns.lookupService('127.0.0.1', 80, common.mustCall(onlookupService));
function onlookupService(err_, ip, family) {
function onlookupService() {
// we don't care about the error here in order to allow
// tests to run offline (lookup will fail in that case and the err be set)