more debugging

oven-sh · Jan 9, 2025 · 4a332d2 · 4a332d2
1 parent 1043e0e
commit 4a332d2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/js/internal/util/colors.ts b/src/js/internal/util/colors.ts
@@ -19,7 +19,11 @@ let exports = {
   reset: "",
   hasColors: false,
   shouldColorize(stream: WriteStream) {
-    if (stream?.isTTY) return lazyGetColorDepth().$call(stream) > 2;
+    if (stream?.isTTY) {
+      const depth = lazyGetColorDepth().$call(stream);
+      console.error("stream is a tty with color depth", depth);
+      return depth > 2;
+    }
 
     // do not cache these since users may update them as the process runs
     const { NO_COLOR, NODE_DISABLE_COLORS, FORCE_COLOR } = process.env;

diff --git a/test/js/node/test/parallel/test-child-process-spawnsync-input.js b/test/js/node/test/parallel/test-child-process-spawnsync-input.js
@@ -35,7 +35,7 @@ const msgErrBuf = Buffer.from(`${msgErr}\n`);
 
 const args = [
   '-e',
-  `console.log(process.env); console.log("${msgOut}"); console.error("${msgErr}");`,
+  `console.log("${msgOut}"); console.error("${msgErr}");`,
 ];
 
 let ret;
@@ -48,9 +48,6 @@ function checkSpawnSyncRet(ret) {
 
 function verifyBufOutput(ret) {
   checkSpawnSyncRet(ret);
-  // FIXME: here for debugging. will be removed before PR is merged
-  console.log("====== parent env")
-  console.log(process.env)
   assert.deepStrictEqual(ret.stdout.toString('utf8'), msgOutBuf.toString('utf8'));
   assert.deepStrictEqual(ret.stdout, msgOutBuf);
   assert.deepStrictEqual(ret.stderr.toString('utf8'), msgErrBuf.toString('utf8'));