fix: process.stdout could be undefined on Windows (fix #1292) (#1305)

vitest-dev · May 13, 2022 · 3179ac9 · 3179ac9
1 parent c491fb4
commit 3179ac9
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
@@ -283,7 +283,7 @@ export class Vitest {
     if (this.server.config.clearScreen === false)
       return
 
-    const repeatCount = process.stdout.rows - 2
+    const repeatCount = (process.stdout?.rows ?? 0) - 2
     const blank = repeatCount > 0 ? '\n'.repeat(repeatCount) : ''
     this.console.log(blank)
     readline.cursorTo(process.stdout, 0, 0)

diff --git a/packages/vitest/src/node/diff.ts b/packages/vitest/src/node/diff.ts
@@ -3,7 +3,7 @@ import * as diff from 'diff'
 import cliTruncate from 'cli-truncate'
 
 export function formatLine(line: string, outputTruncateLength?: number) {
-  return cliTruncate(line, (outputTruncateLength ?? (process.stdout.columns || 80)) - 4)
+  return cliTruncate(line, (outputTruncateLength ?? (process.stdout?.columns || 80)) - 4)
 }
 
 export interface DiffOptions {

diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
@@ -189,7 +189,7 @@ export function generateCodeFrame(
   let count = 0
   let res: string[] = []
 
-  const columns = process.stdout.columns || 80
+  const columns = process.stdout?.columns || 80
 
   function lineNo(no: number | string = '') {
     return c.gray(`${String(no).padStart(3, ' ')}| `)

diff --git a/packages/vitest/src/node/reporters/renderers/utils.ts b/packages/vitest/src/node/reporters/renderers/utils.ts
@@ -12,7 +12,7 @@ export const pointer = c.yellow(F_POINTER)
 export const skipped = c.dim(c.gray(F_DOWN))
 
 export function getCols(delta = 0) {
-  let length = process.stdout.columns
+  let length = process.stdout?.columns
   if (!length || isNaN(length))
     length = 30
   return Math.max(length + delta, 0)