repl: refactor tests to not rely on timing

Tests relying on synchronous timing have been migrated to use events.

PR-URL: https://github.com/nodejs/node/pull/17828
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>

1 files changed, 20 insertions, 1 deletions

diff --git a/test/parallel/test-repl-null.js b/test/parallel/test-repl-null.js
index 66d09b28f2..2748cfa780 100644
--- a/test/parallel/test-repl-null.js
+++ b/test/parallel/test-repl-null.js
@@ -1,9 +1,28 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const repl = require('repl');
 const assert = require('assert');
 
+const callbacks = [
+  common.mustCall((err, value) => {
+    assert.ifError(err);
+    assert.strictEqual(value, undefined);
+  })
+];
 const replserver = new repl.REPLServer();
+const $eval = replserver.eval;
+replserver.eval = function(code, context, file, cb) {
+  const expected = callbacks.shift();
+  return $eval.call(this, code, context, file, (...args) => {
+    try {
+      expected(...args);
+    } catch (e) {
+      console.error(e);
+      process.exit(1);
+    }
+    cb(...args);
+  });
+};
 
 replserver._inTemplateLiteral = true;