test: refactor test-process-kill-null

Use `common.mustCall()` instead of `called` variable.

PR-URL: #16236
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>

Author: lpinca

test/parallel/test-process-kill-null.js

@@ -1,27 +1,21 @@
 'use strict';
-require('../common');
+const { mustCall } = require('../common');
 const assert = require('assert');
-const spawn = require('child_process').spawn;
+const { spawn } = require('child_process');
 
 const cat = spawn('cat');
-let called;
 
 assert.ok(process.kill(cat.pid, 0));
 
-cat.on('exit', function() {
+cat.on('exit', mustCall(function() {
   assert.throws(function() {
     process.kill(cat.pid, 0);
   }, Error);
-});
+}));
 
-cat.stdout.on('data', function() {
-  called = true;
+cat.stdout.on('data', mustCall(function() {
   process.kill(cat.pid, 'SIGKILL');
-});
+}));
 
 // EPIPE when null sig fails
 cat.stdin.write('test');
-
-process.on('exit', function() {
-  assert.ok(called);
-});