Skip to content

Commit

Permalink
test: remove unused function arguments in async-hooks tests
Browse files Browse the repository at this point in the history
Remove unused function arguments in two async-hooks tests.

PR-URL: #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>
  • Loading branch information
Trott authored and rvagg committed Nov 28, 2018
1 parent 5d67eec commit ee64ae0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion test/async-hooks/test-getaddrinforeqwrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -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);

Expand Down
2 changes: 1 addition & 1 deletion test/async-hooks/test-getnameinforeqwrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -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)

Expand Down

0 comments on commit ee64ae0

Please sign in to comment.