test: minor fixups for REPL eval tests

The `process.on("exit")` event handlers are unnecessary, so it’s okay to drop them. PR-URL: #11946 Ref: #11871 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Jeremiah Senkpiel <[email protected]>
nodejs · Mar 22, 2017 · 64d0a73 · 64d0a73
1 parent 76279cb
commit 64d0a73
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);
 }