test: refactor test-timer-close

Refactor and simplify parallel/test-timer-close.js. Add comment to
describe the test case.

PR-URL: #10517
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Santiago Gimeno <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>

Author: BethGriggs

test/parallel/test-timer-close.js

@@ -1,16 +1,11 @@
 'use strict';
-require('../common');
-const assert = require('assert');
+const common = require('../common');
 
-var t = new (process.binding('timer_wrap').Timer)();
-var called = 0;
-function onclose() {
-  called++;
-}
+// Make sure handle._handle.close(callback) is idempotent by closing a timer
+// twice. The first function should be called, the second one should not.
 
-t.close(onclose);
-t.close(onclose);
+const Timer = process.binding('timer_wrap').Timer;
+const t = new Timer();
 
-process.on('exit', function() {
-  assert.equal(1, called);
-});
+t.close(common.mustCall(function() {}));
+t.close(() => common.fail('This should never be called'));