Skip to content

Commit 7e2a284

Browse files
jkmdevrvagg
authored andcommitted
test: use destructuring and remove unused arguments
Use destructuring consistently and remove unused function arguments in test-pipewrap.js PR-URL: #24375 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Weijia Wang <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
1 parent cdda7f4 commit 7e2a284

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/async-hooks/test-pipewrap.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ const assert = require('assert');
88
const tick = require('../common/tick');
99
const initHooks = require('./init-hooks');
1010
const { checkInvocations } = require('./hook-checks');
11-
const spawn = require('child_process').spawn;
11+
const { spawn } = require('child_process');
1212

1313
if (!common.isMainThread)
1414
common.skip('Worker bootstrapping works differently -> different async IDs');
@@ -45,7 +45,7 @@ checkInvocations(processwrap, { init: 1 },
4545
checkInvocations(x, { init: 1 }, 'pipe wrap when sleep.spawn was called');
4646
});
4747

48-
function onsleepExit(code) {
48+
function onsleepExit() {
4949
checkInvocations(processwrap, { init: 1, before: 1 },
5050
'processwrap while in onsleepExit callback');
5151
}

0 commit comments

Comments
 (0)