Browse Source

test: fix old tty tests

test-tty-wrap hasn’t worked since StreamBase was introduced, I think.
test-tty-stdout-end also happens to works with PipeWrap-s.

Refs: https://github.com/nodejs/node/pull/7360
PR-URL: https://github.com/nodejs/node/pull/7613
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v4.x
Jeremiah Senkpiel 8 years ago
committed by Myles Borins
parent
commit
cd9e8e0361
  1. 1
      test/pseudo-tty/test-tty-stdout-end.js
  2. 0
      test/pseudo-tty/test-tty-stdout-end.out
  3. 31
      test/pseudo-tty/test-tty-wrap.js
  4. 2
      test/pseudo-tty/test-tty-wrap.out

1
test/pseudo-tty/test-tty-stdout-end.js

@ -1,5 +1,4 @@
'use strict';
// Can't test this when 'make test' doesn't assign a tty to the stdout.
require('../common');
const assert = require('assert');

0
test/pseudo-tty/test-tty-stdout-end.out

31
test/pseudo-tty/test-tty-wrap.js

@ -1,28 +1,11 @@
'use strict';
const common = require('../common');
var assert = require('assert');
require('../common');
var TTY = process.binding('tty_wrap').TTY;
var isTTY = process.binding('tty_wrap').isTTY;
const TTY = process.binding('tty_wrap').TTY;
const WriteWrap = process.binding('stream_wrap').WriteWrap;
if (isTTY(1) == false) {
common.skip('fd 1 is not a tty.');
return;
}
const handle = new TTY(1);
const req = new WriteWrap();
var handle = new TTY(1);
var callbacks = 0;
var req1 = handle.writeBuffer(Buffer('hello world\n'));
req1.oncomplete = function() {
callbacks++;
};
var req2 = handle.writeBuffer(Buffer('hello world\n'));
req2.oncomplete = function() {
callbacks++;
};
process.on('exit', function() {
assert.equal(2, callbacks);
});
handle.writeBuffer(req, Buffer.from('hello world 1\n'));
handle.writeBuffer(req, Buffer.from('hello world 2\n'));

2
test/pseudo-tty/test-tty-wrap.out

@ -0,0 +1,2 @@
hello world 1
hello world 2
Loading…
Cancel
Save