test: minor fixups for REPL eval tests

The `process.on("exit")` event handlers are unnecessary, so it’s okay to drop them. Ref: nodejs#11871
addaleax · Mar 20, 2017 · 3859673 · 3859673
1 parent 2a4a5f0
commit 3859673
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/test/parallel/test-repl-empty.js b/test/parallel/test-repl-empty.js
@@ -9,7 +9,7 @@ const repl = require('repl');
   const options = {
     eval: common.mustCall((cmd, context) => {
       // Assertions here will not cause the test to exit with an error code
-      // so set a boolean that is checked in process.on('exit',...) instead.
+      // so set a boolean that is checked later instead.
       evalCalledWithExpectedArgs = (cmd === '\n');
     })
   };
@@ -23,7 +23,5 @@ const repl = require('repl');
     r.write('.exit\n');
   }
 
-  process.on('exit', () => {
-    assert(evalCalledWithExpectedArgs);
-  });
+  assert(evalCalledWithExpectedArgs);
 }
diff --git a/test/parallel/test-repl-eval.js b/test/parallel/test-repl-eval.js
@@ -9,7 +9,7 @@ const repl = require('repl');
   const options = {
     eval: common.mustCall((cmd, context) => {
       // Assertions here will not cause the test to exit with an error code
-      // so set a boolean that is checked in process.on('exit',...) instead.
+      // so set a boolean that is checked later instead.
       evalCalledWithExpectedArgs = (cmd === 'function f() {}\n' &&
                                     context.foo === 'bar');
     })
@@ -29,7 +29,5 @@ const repl = require('repl');
     r.write('.exit\n');
   }
 
-  process.on('exit', () => {
-    assert(evalCalledWithExpectedArgs);
-  });
+  assert(evalCalledWithExpectedArgs);
 }