Skip to content

Commit

Permalink
test_runner: switched to internal readline interface
Browse files Browse the repository at this point in the history
Switched to using internal interface after

PR-URL: #54000
Reviewed-By: Chemi Atlow <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
  • Loading branch information
Emiltayeb authored and RafaelGSS committed Aug 5, 2024
1 parent 10bea42 commit 3999021
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions lib/internal/test_runner/runner.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,7 @@ const { spawn } = require('child_process');
const { finished } = require('internal/streams/end-of-stream');
const { resolve } = require('path');
const { DefaultDeserializer, DefaultSerializer } = require('v8');
// TODO(aduh95): switch to internal/readline/interface when backporting to Node.js 16.x is no longer a concern.
const { createInterface } = require('readline');
const { Interface } = require('internal/readline/interface');
const { deserializeError } = require('internal/error_serdes');
const { Buffer } = require('buffer');
const { FilesWatcher } = require('internal/watch_mode/files_watcher');
Expand Down Expand Up @@ -354,7 +353,7 @@ function runTestFile(path, filesWatcher, opts) {
subtest.parseMessage(data);
});

const rl = createInterface({ __proto__: null, input: child.stderr });
const rl = new Interface({ __proto__: null, input: child.stderr });
rl.on('line', (line) => {
if (isInspectorMessage(line)) {
process.stderr.write(line + '\n');
Expand Down

0 comments on commit 3999021

Please sign in to comment.