Revert "readline: add stricter validation for functions called after closed"

dario-piotrowicz · RafaelGSS · commit 14b357940cce · 2025-05-02T17:06:46.000-03:00
This reverts commit 8e7f32f. PR-URL: #58024 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Richard Lau <rlau@redhat.com>
diff --git a/lib/internal/readline/interface.js b/lib/internal/readline/interface.js
@@ -599,9 +599,6 @@ class Interface extends InterfaceConstructor {
    * @returns {void | Interface}
    */
   pause() {
-    if (this.closed) {
-      throw new ERR_USE_AFTER_CLOSE('readline');
-    }
     if (this.paused) return;
     this.input.pause();
     this.paused = true;
@@ -614,9 +611,6 @@ class Interface extends InterfaceConstructor {
    * @returns {void | Interface}
    */
   resume() {
-    if (this.closed) {
-      throw new ERR_USE_AFTER_CLOSE('readline');
-    }
     if (!this.paused) return;
     this.input.resume();
     this.paused = false;
@@ -637,9 +631,6 @@ class Interface extends InterfaceConstructor {
    * @returns {void}
    */
   write(d, key) {
-    if (this.closed) {
-      throw new ERR_USE_AFTER_CLOSE('readline');
-    }
     if (this.paused) this.resume();
     if (this.terminal) {
       this[kTtyWrite](d, key);
diff --git a/test/parallel/test-readline-interface.js b/test/parallel/test-readline-interface.js
@@ -1202,47 +1202,6 @@ for (let i = 0; i < 12; i++) {
     fi.emit('data', 'Node.js\n');
   }
 
-  // Call write after close
-  {
-    const [rli, fi] = getInterface({ terminal });
-    rli.question('What\'s your name?', common.mustCall((name) => {
-      assert.strictEqual(name, 'Node.js');
-      rli.close();
-      assert.throws(() => {
-        rli.write('I said Node.js');
-      }, {
-        name: 'Error',
-        code: 'ERR_USE_AFTER_CLOSE'
-      });
-    }));
-    fi.emit('data', 'Node.js\n');
-  }
-
-  // Call pause/resume after close
-  {
-    const [rli, fi] = getInterface({ terminal });
-    rli.question('What\'s your name?', common.mustCall((name) => {
-      assert.strictEqual(name, 'Node.js');
-      rli.close();
-      // No 'resume' nor 'pause' event should be emitted after close
-      rli.on('resume', common.mustNotCall());
-      rli.on('pause', common.mustNotCall());
-      assert.throws(() => {
-        rli.pause();
-      }, {
-        name: 'Error',
-        code: 'ERR_USE_AFTER_CLOSE'
-      });
-      assert.throws(() => {
-        rli.resume();
-      }, {
-        name: 'Error',
-        code: 'ERR_USE_AFTER_CLOSE'
-      });
-    }));
-    fi.emit('data', 'Node.js\n');
-  }
-
   // Can create a new readline Interface with a null output argument
   {
     const [rli, fi] = getInterface({ output: null, terminal });
diff --git a/test/parallel/test-readline-promises-interface.js b/test/parallel/test-readline-promises-interface.js
@@ -204,7 +204,7 @@ function assertCursorRowsAndCols(rli, rows, cols) {
     fi.emit('data', character);
   }
   fi.emit('data', '\n');
-  fi.end();
+  rli.close();
 }
 
 // \t when there is no completer function should behave like an ordinary
diff --git a/test/parallel/test-readline-promises-tab-complete.js b/test/parallel/test-readline-promises-tab-complete.js
@@ -80,7 +80,7 @@ if (process.env.TERM === 'dumb') {
           output = '';
         });
       }
-      fi.end();
+      rli.close();
     });
   });
 });
@@ -114,5 +114,5 @@ if (process.env.TERM === 'dumb') {
     assert.match(output, /^Tab completion error: Error: message/);
     output = '';
   });
-  fi.end();
+  rli.close();
 }
diff --git a/test/parallel/test-repl-import-referrer.js b/test/parallel/test-repl-import-referrer.js
@@ -8,24 +8,20 @@ const args = ['--interactive'];
 const opts = { cwd: fixtures.path('es-modules') };
 const child = cp.spawn(process.execPath, args, opts);
 
-const outputs = [];
+let output = '';
 child.stdout.setEncoding('utf8');
 child.stdout.on('data', (data) => {
-  outputs.push(data);
-  if (outputs.length === 3) {
-    // All the expected outputs have been received
-    // so we can close the child process's stdin
-    child.stdin.end();
-  }
+  output += data;
 });
 
 child.on('exit', common.mustCall(() => {
-  const results = outputs[2].split('\n')[0];
-  assert.strictEqual(
+  const results = output.replace(/^> /mg, '').split('\n').slice(2);
+  assert.deepStrictEqual(
     results,
-    '[Module: null prototype] { message: \'A message\' }'
+    ['[Module: null prototype] { message: \'A message\' }', '']
   );
 }));
 
 child.stdin.write('await import(\'./message.mjs\');\n');
 child.stdin.write('.exit');
+child.stdin.end();
diff --git a/test/parallel/test-repl-no-terminal.js b/test/parallel/test-repl-no-terminal.js
@@ -1,12 +1,7 @@
 'use strict';
 const common = require('../common');
-const ArrayStream = require('../common/arraystream');
-const repl = require('repl');
-
-const stream = new ArrayStream();
 
-const replServer = repl.start({ terminal: false, input: stream, output: stream });
-
-replServer.setupHistory('/nonexistent/file', common.mustSucceed(() => {
-  replServer.close();
-}));
+const repl = require('repl');
+const r = repl.start({ terminal: false });
+r.setupHistory('/nonexistent/file', common.mustSucceed());
+process.stdin.unref?.();
diff --git a/test/parallel/test-repl-uncaught-exception-async.js b/test/parallel/test-repl-uncaught-exception-async.js
@@ -34,9 +34,9 @@ r.write(
   '  throw new RangeError("abc");\n' +
   '}, 1);console.log()\n'
 );
+r.close();
 
 setTimeout(() => {
-  r.close();
   const len = process.listenerCount('uncaughtException');
   process.removeAllListeners('uncaughtException');
   assert.strictEqual(len, 0);

Original file line number	Diff line number	Diff line change
`@@ -204,7 +204,7 @@ function assertCursorRowsAndCols(rli, rows, cols) {`
`204`	`204`	`fi.emit('data', character);`
`205`	`205`	`}`
`206`	`206`	`fi.emit('data', '\n');`
`207`		`- fi.end();`
	`207`	`+ rli.close();`
`208`	`208`	`}`
`209`	`209`
`210`	`210`	`// \t when there is no completer function should behave like an ordinary`