Commit 21e69d1
test: fix test-tty-get-color-depth
If getTTYfd returns 0, and stdin is not writable (like on Windows),
trying to create WriteStream will fail. This commit fixes that by
skipping fd 0.
PR-URL: #18478
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 441175c commit 21e69d1
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -10,7 +10,7 @@ const { WriteStream } = require('tty'); | |||
| 10 | 10 | ||
| 11 | 11 | // Do our best to grab a tty fd. | |
| 12 | 12 | function getTTYfd() { | |
| 13 | - const ttyFd = [0, 1, 2, 4, 5].find(tty.isatty); | ||
| 13 | + const ttyFd = [1, 2, 4, 5].find(tty.isatty); | ||
| 14 | 14 | if (ttyFd === undefined) { | |
| 15 | 15 | try { | |
| 16 | 16 | return openSync('/dev/tty'); | |
0 commit comments