fix: jestLogger to include stacktrace always (not just for error level)

Cause it should be safe to print stacktraces in test environment
NaturalCycles · Nov 27, 2021 · b8de50e · b8de50e
1 parent 1c83f29
commit b8de50e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/test/setupJest.ts b/src/test/setupJest.ts
@@ -1,5 +1,7 @@
-// console.log('setupJest.ts')
-
 import { jestOffline } from '../jestOffline.util'
+import { jestLog, jestLogger } from '../testing'
 
 jestOffline()
+
+console.log = console.warn = jestLog
+console.error = jestLogger.error.bind(jestLogger)
diff --git a/src/testing/testing.util.ts b/src/testing/testing.util.ts
@@ -31,13 +31,13 @@ export function silentConsole(): void {
   console.table = () => {}
 }
 
-export const jestLogger = commonLoggerCreate((level, args) => {
+export const jestLogger = commonLoggerCreate((_level, args) => {
   if (process.env['JEST_SILENT']) return // no-op
   process.stdout.write(
     args
       .map(a =>
         inspectAny(a, {
-          includeErrorStack: level === 'error',
+          includeErrorStack: true,
         }),
       )
       .join(' ') + '\n',