From 48701c19f8b2661c7e2cd997e62070d74baa86d0 Mon Sep 17 00:00:00 2001 From: Marvin Hagemeister Date: Mon, 19 Aug 2024 17:13:09 +0200 Subject: fix(node/tty): fix `tty.WriteStream.hasColor` with different args (#25094) The check in `tty.WriteStream.prototype.hasColors()` was incorrect leading to the [`yoctocolors`](https://github.com/sindresorhus/yoctocolors) package not printing any colors. Fixes https://github.com/denoland/deno/issues/24407 --- tests/unit_node/tty_test.ts | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'tests/unit_node') diff --git a/tests/unit_node/tty_test.ts b/tests/unit_node/tty_test.ts index ab3193f4a..7f0395682 100644 --- a/tests/unit_node/tty_test.ts +++ b/tests/unit_node/tty_test.ts @@ -38,6 +38,10 @@ Deno.test("[node/tty WriteStream.isTTY] returns true when fd is a tty", () => { Deno.test("[node/tty WriteStream.hasColors] returns true when colors are supported", () => { assert(tty.WriteStream.prototype.hasColors() === !Deno.noColor); + assert(tty.WriteStream.prototype.hasColors({}) === !Deno.noColor); + + assert(tty.WriteStream.prototype.hasColors(1)); + assert(tty.WriteStream.prototype.hasColors(1, {})); }); Deno.test("[node/tty WriteStream.getColorDepth] returns current terminal color depth", () => { -- cgit v1.2.3