tty: expose stream API from readline methods

This commit exposes the return value and callback of the
underlying readline APIs from the tty module.

PR-URL: https://github.com/nodejs/node/pull/28721
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>

1 files changed, 25 insertions, 6 deletions

diff --git a/test/parallel/test-tty-backwards-api.js b/test/parallel/test-tty-backwards-api.js
index 7b6dc3c092..b4005bad61 100644
--- a/test/parallel/test-tty-backwards-api.js
+++ b/test/parallel/test-tty-backwards-api.js
@@ -1,6 +1,8 @@
 // Flags: --expose-internals
 'use strict';
 const common = require('../common');
+const assert = require('assert');
+const readline = require('readline');
 
 const noop = () => {};
 const { internalBinding } = require('internal/test/binding');
@@ -13,15 +15,32 @@ TTY.prototype = {
 
 const { WriteStream } = require('tty');
 
-const methods = [
+[
   'cursorTo',
   'moveCursor',
   'clearLine',
   'clearScreenDown'
-];
+].forEach((method) => {
+  readline[method] = common.mustCall(function() {
+    const lastArg = arguments[arguments.length - 1];
 
-methods.forEach((method) => {
-  require('readline')[method] = common.mustCall();
-  const writeStream = new WriteStream(1);
-  writeStream[method](1, 2);
+    if (typeof lastArg === 'function') {
+      process.nextTick(lastArg);
+    }
+
+    return true;
+  }, 2);
 });
+
+const writeStream = new WriteStream(1);
+
+// Verify that the corresponding readline methods are called, that the return
+// values are propagated, and any callbacks are invoked.
+assert.strictEqual(writeStream.cursorTo(1, 2), true);
+assert.strictEqual(writeStream.cursorTo(1, 2, common.mustCall()), true);
+assert.strictEqual(writeStream.moveCursor(1, 2), true);
+assert.strictEqual(writeStream.moveCursor(1, 2, common.mustCall()), true);
+assert.strictEqual(writeStream.clearLine(1), true);
+assert.strictEqual(writeStream.clearLine(1, common.mustCall()), true);
+assert.strictEqual(writeStream.clearScreenDown(), true);
+assert.strictEqual(writeStream.clearScreenDown(common.mustCall()), true);