Skip to content

Commit

Permalink
test: uv_tty_init now returns EINVAL on IBM i
Browse files Browse the repository at this point in the history
Since the PR libuv/libuv#2753
has been landed, we need to revert the code change in
PR nodejs#32338.

PR-URL: nodejs#33629
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
dmabupt authored and richardlau committed Jun 1, 2020
1 parent 2935f72 commit 7774458
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions test/parallel/test-ttywrap-invalid-fd.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,13 @@ assert.throws(

{
const info = {
code: common.isWindows || common.isIBMi ? 'EBADF' : 'EINVAL',
message: common.isWindows ||
common.isIBMi ? 'bad file descriptor' : 'invalid argument',
errno: common.isWindows || common.isIBMi ? UV_EBADF : UV_EINVAL,
code: common.isWindows ? 'EBADF' : 'EINVAL',
message: common.isWindows ? 'bad file descriptor' : 'invalid argument',
errno: common.isWindows ? UV_EBADF : UV_EINVAL,
syscall: 'uv_tty_init'
};

const suffix = common.isWindows || common.isIBMi ?
const suffix = common.isWindows ?
'EBADF (bad file descriptor)' : 'EINVAL (invalid argument)';
const message = `TTY initialization failed: uv_tty_init returned ${suffix}`;

Expand Down

0 comments on commit 7774458

Please sign in to comment.