From 7853caf133345ffbdc7fd4afc9b4ff3a8af3dd8b Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 09:27:14 +0300
Subject: [PATCH 1/8] feat: improve error logging

---
 packages/vitest/src/node/cli-api.ts        |  2 +-
 packages/vitest/src/node/core.ts           |  4 +-
 packages/vitest/src/node/error.ts          | 55 ++++++++++++++++++----
 packages/vitest/src/node/reporters/base.ts |  2 +-
 4 files changed, 49 insertions(+), 14 deletions(-)

diff --git a/packages/vitest/src/node/cli-api.ts b/packages/vitest/src/node/cli-api.ts
index 4216f1d08ff1..95eca429c299 100644
--- a/packages/vitest/src/node/cli-api.ts
+++ b/packages/vitest/src/node/cli-api.ts
@@ -76,7 +76,7 @@ export async function startVitest(cliFilters: string[], options: CliOptions, vit
   catch (e) {
     process.exitCode = 1
     ctx.error(`\n${c.red(divider(c.bold(c.inverse(' Unhandled Error '))))}`)
-    await ctx.printError(e)
+    await ctx.printError(e, true)
     ctx.error('\n\n')
     return false
   }
diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
index 4cdaf0227411..fd452c5e6f98 100644
--- a/packages/vitest/src/node/core.ts
+++ b/packages/vitest/src/node/core.ts
@@ -493,8 +493,8 @@ export class Vitest {
     return code.includes('import.meta.vitest')
   }
 
-  printError(err: unknown) {
-    return printError(err, this)
+  printError(err: unknown, fullStack = false) {
+    return printError(err, fullStack, this)
   }
 
   onServerRestarted(fn: () => void) {
diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index c973e1b22132..699811fe7103 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -1,6 +1,6 @@
 /* eslint-disable prefer-template */
 /* eslint-disable no-template-curly-in-string */
-import { existsSync, promises as fs } from 'fs'
+import { existsSync, readFileSync } from 'fs'
 import { join, relative } from 'pathe'
 import c from 'picocolors'
 import cliTruncate from 'cli-truncate'
@@ -17,7 +17,7 @@ export function fileFromParsedStack(stack: ParsedStack) {
   return stack.file
 }
 
-export async function printError(error: unknown, ctx: Vitest) {
+export async function printError(error: unknown, fullStack: boolean, ctx: Vitest) {
   let e = error as ErrorWithDiff
 
   if (typeof error === 'string') {
@@ -27,7 +27,7 @@ export async function printError(error: unknown, ctx: Vitest) {
     } as any
   }
 
-  const stacks = parseStacktrace(e)
+  const stacks = parseStacktrace(e, fullStack)
 
   await interpretSourcePos(stacks, ctx)
 
@@ -36,10 +36,12 @@ export async function printError(error: unknown, ctx: Vitest) {
       && existsSync(stack.file),
   )
 
+  const errorProperties = getErrorProperties(e)
+
   printErrorMessage(e, ctx.console)
-  await printStack(ctx, stacks, nearest, async (s, pos) => {
+  printStack(ctx, stacks, nearest, errorProperties, (s, pos) => {
     if (s === nearest && nearest) {
-      const sourceCode = await fs.readFile(fileFromParsedStack(nearest), 'utf-8')
+      const sourceCode = readFileSync(fileFromParsedStack(nearest), 'utf-8')
       ctx.log(c.yellow(generateCodeFrame(sourceCode, 4, pos)))
     }
   })
@@ -50,6 +52,27 @@ export async function printError(error: unknown, ctx: Vitest) {
     displayDiff(stringify(e.actual), stringify(e.expected), ctx.console, ctx.config.outputTruncateLength)
 }
 
+function getErrorProperties(e: ErrorWithDiff) {
+  const skip = [
+    'message',
+    'name',
+    'nameStr',
+    'stack',
+    'stacks',
+    'stackStr',
+    'showDiff',
+    'actual',
+    'expected',
+  ]
+  const errorObject = Object.create(null)
+  for (const key in e) {
+    if (!skip.includes(key))
+      errorObject[key] = e[key as keyof ErrorWithDiff]
+  }
+
+  return errorObject
+}
+
 const esmErrors = [
   'Cannot use import statement outside a module',
   'Unexpected token \'export\'',
@@ -95,29 +118,41 @@ function printErrorMessage(error: ErrorWithDiff, console: Console) {
   console.error(c.red(`${c.bold(errorName)}: ${error.message}`))
 }
 
-async function printStack(
+function printStack(
   ctx: Vitest,
   stack: ParsedStack[],
-  highlight?: ParsedStack,
+  highlight: ParsedStack | undefined,
+  errorProperties: Record<string, unknown>,
   onStack?: ((stack: ParsedStack, pos: Position) => void),
 ) {
   if (!stack.length)
     return
 
+  const hasProperties = Object.keys(errorProperties).length > 0
+
+  let stackNumber = 0
+
   for (const frame of stack) {
     const pos = frame.sourcePos || frame
     const color = frame === highlight ? c.yellow : c.gray
     const file = fileFromParsedStack(frame)
     const path = relative(ctx.config.root, file)
 
-    ctx.log(color(` ${c.dim(F_POINTER)} ${[frame.method, c.dim(`${path}:${pos.line}:${pos.column}`)].filter(Boolean).join(' ')}`))
-    await onStack?.(frame, pos)
+    const isLastStack = stackNumber === stack.length - 1 || frame.file in ctx.state.filesMap
+
+    ctx.log(color(` ${c.dim(F_POINTER)} ${[frame.method, c.dim(`${path}:${pos.line}:${pos.column}`)].filter(Boolean).join(' ')}`), isLastStack && hasProperties ? '{' : '')
+    onStack?.(frame, pos)
 
     // reached at test file, skip the follow stack
     if (frame.file in ctx.state.filesMap)
       break
+
+    stackNumber++
+  }
+  if (hasProperties) {
+    const propertiesString = stringify(errorProperties, 10, { printBasicPrototype: false }).substring(2)
+    ctx.log(propertiesString)
   }
-  ctx.log()
 }
 
 export function generateCodeFrame(
diff --git a/packages/vitest/src/node/reporters/base.ts b/packages/vitest/src/node/reporters/base.ts
index a522333777ff..45c98ea48bbb 100644
--- a/packages/vitest/src/node/reporters/base.ts
+++ b/packages/vitest/src/node/reporters/base.ts
@@ -210,7 +210,7 @@ export abstract class BaseReporter implements Reporter {
     process.on('unhandledRejection', async (err) => {
       process.exitCode = 1
       this.ctx.error(`\n${c.red(divider(c.bold(c.inverse(' Unhandled Rejection '))))}`)
-      await this.ctx.printError(err)
+      await this.ctx.printError(err, true)
       this.ctx.error('\n\n')
       process.exit(1)
     })

From 23c4e113926176fe76442e39e1036a6e1047913b Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 09:49:38 +0300
Subject: [PATCH 2/8] feat: more friendly error message for "Vitest initialized
 with Native Node"

---
 packages/vitest/src/integrations/vi.ts | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/packages/vitest/src/integrations/vi.ts b/packages/vitest/src/integrations/vi.ts
index 3eac9c9dd713..66b113d10d79 100644
--- a/packages/vitest/src/integrations/vi.ts
+++ b/packages/vitest/src/integrations/vi.ts
@@ -21,8 +21,15 @@ class VitestUtils {
     this._mocker = typeof __vitest_mocker__ !== 'undefined' ? __vitest_mocker__ : null
     this._mockedDate = null
 
-    if (!this._mocker)
-      throw new Error('Vitest was initialized with native Node instead of Vite Node')
+    if (!this._mocker) {
+      const errorMsg = 'Vitest was initialized with native Node instead of Vite Node.'
+      + '\n\nOne of the following is possible:'
+      + '\n- "vitest" is imported outside of your tests (in that case, use "vitest/node" or import.meta.vitest)'
+      + '\n- "vitest" is imported inside "globalSetup" (use "setupFiles", because "globalSetup" runs in a different context)'
+      + '\n- Your dependency inside "node_modules" imports "vitest" directly (in that case, inline that dependency, using "deps.inline" config)'
+      + '\n- Otherwise, it might be a Vitest bug. Please report it to https://github.com/vitest-dev/vitest/issues\n'
+      throw new Error(errorMsg)
+    }
   }
 
   // timers

From d6d99d631ecc6c07aca78a3f642c6999b6f062f2 Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:24:18 +0300
Subject: [PATCH 3/8] chore: don't exit the process on inhandled rejections

---
 packages/vitest/src/node/cli-api.ts           |  5 +--
 packages/vitest/src/node/core.ts              | 13 +++++---
 packages/vitest/src/node/error.ts             | 32 +++++++++++--------
 packages/vitest/src/node/pool.ts              |  5 ++-
 packages/vitest/src/node/reporters/base.ts    | 19 +++++++++--
 packages/vitest/src/node/reporters/default.ts |  4 +--
 packages/vitest/src/node/reporters/dot.ts     |  4 +--
 packages/vitest/src/node/state.ts             | 16 +++++++++-
 packages/vitest/src/runtime/worker.ts         |  4 +++
 packages/vitest/src/types/general.ts          |  1 +
 packages/vitest/src/types/reporter.ts         |  2 +-
 packages/vitest/src/types/worker.ts           |  3 +-
 packages/vitest/src/utils/source-map.ts       |  2 ++
 13 files changed, 78 insertions(+), 32 deletions(-)

diff --git a/packages/vitest/src/node/cli-api.ts b/packages/vitest/src/node/cli-api.ts
index 95eca429c299..16bc1a6945a6 100644
--- a/packages/vitest/src/node/cli-api.ts
+++ b/packages/vitest/src/node/cli-api.ts
@@ -1,11 +1,9 @@
 import { execa } from 'execa'
 import type { UserConfig as ViteUserConfig } from 'vite'
-import c from 'picocolors'
 import type { UserConfig } from '../types'
 import { ensurePackageInstalled } from '../utils'
 import { createVitest } from './create'
 import { registerConsoleShortcuts } from './stdin'
-import { divider } from './reporters/renderers/utils'
 
 export interface CliOptions extends UserConfig {
   /**
@@ -75,8 +73,7 @@ export async function startVitest(cliFilters: string[], options: CliOptions, vit
   }
   catch (e) {
     process.exitCode = 1
-    ctx.error(`\n${c.red(divider(c.bold(c.inverse(' Unhandled Error '))))}`)
-    await ctx.printError(e, true)
+    await ctx.printError(e, true, 'Unhandled Error')
     ctx.error('\n\n')
     return false
   }
diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
index fd452c5e6f98..9fe2ec5dd203 100644
--- a/packages/vitest/src/node/core.ts
+++ b/packages/vitest/src/node/core.ts
@@ -226,12 +226,17 @@ export class Vitest {
       const invalidates = Array.from(this.invalidates)
       this.invalidates.clear()
       this.snapshot.clear()
-      await this.pool.runTests(files, invalidates)
+      try {
+        await this.pool.runTests(files, invalidates)
+      }
+      catch (err) {
+        this.state.catchError(err, 'Unhandled Error')
+      }
 
       if (hasFailed(this.state.getFiles()))
         process.exitCode = 1
 
-      await this.report('onFinished', this.state.getFiles())
+      await this.report('onFinished', this.state.getFiles(), this.state.getUnhandledErrors())
     })()
       .finally(() => {
         this.runningPromise = undefined
@@ -493,8 +498,8 @@ export class Vitest {
     return code.includes('import.meta.vitest')
   }
 
-  printError(err: unknown, fullStack = false) {
-    return printError(err, fullStack, this)
+  printError(err: unknown, fullStack = false, type?: string) {
+    return printError(err, type, fullStack, this)
   }
 
   onServerRestarted(fn: () => void) {
diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index 699811fe7103..ee70065adae6 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -10,6 +10,7 @@ import { F_POINTER } from '../utils/figures'
 import { stringify } from '../integrations/chai/jest-matcher-utils'
 import type { Vitest } from './core'
 import { unifiedDiff } from './diff'
+import { divider } from './reporters/renderers/utils'
 
 export function fileFromParsedStack(stack: ParsedStack) {
   if (stack?.sourcePos?.source?.startsWith('..'))
@@ -17,7 +18,7 @@ export function fileFromParsedStack(stack: ParsedStack) {
   return stack.file
 }
 
-export async function printError(error: unknown, fullStack: boolean, ctx: Vitest) {
+export async function printError(error: unknown, type: string | undefined, fullStack: boolean, ctx: Vitest) {
   let e = error as ErrorWithDiff
 
   if (typeof error === 'string') {
@@ -38,6 +39,8 @@ export async function printError(error: unknown, fullStack: boolean, ctx: Vitest
 
   const errorProperties = getErrorProperties(e)
 
+  if (type)
+    printErrorType(type, ctx)
   printErrorMessage(e, ctx.console)
   printStack(ctx, stacks, nearest, errorProperties, (s, pos) => {
     if (s === nearest && nearest) {
@@ -52,7 +55,15 @@ export async function printError(error: unknown, fullStack: boolean, ctx: Vitest
     displayDiff(stringify(e.actual), stringify(e.expected), ctx.console, ctx.config.outputTruncateLength)
 }
 
+function printErrorType(type: string, ctx: Vitest) {
+  ctx.error(`\n${c.red(divider(c.bold(c.inverse(` ${type} `))))}`)
+}
+
 function getErrorProperties(e: ErrorWithDiff) {
+  const errorObject = Object.create(null)
+  if (e.name === 'AssertionError')
+    return errorObject
+
   const skip = [
     'message',
     'name',
@@ -60,11 +71,11 @@ function getErrorProperties(e: ErrorWithDiff) {
     'stack',
     'stacks',
     'stackStr',
+    'type',
     'showDiff',
     'actual',
     'expected',
   ]
-  const errorObject = Object.create(null)
   for (const key in e) {
     if (!skip.includes(key))
       errorObject[key] = e[key as keyof ErrorWithDiff]
@@ -128,30 +139,25 @@ function printStack(
   if (!stack.length)
     return
 
-  const hasProperties = Object.keys(errorProperties).length > 0
-
-  let stackNumber = 0
-
   for (const frame of stack) {
     const pos = frame.sourcePos || frame
     const color = frame === highlight ? c.yellow : c.gray
     const file = fileFromParsedStack(frame)
     const path = relative(ctx.config.root, file)
 
-    const isLastStack = stackNumber === stack.length - 1 || frame.file in ctx.state.filesMap
-
-    ctx.log(color(` ${c.dim(F_POINTER)} ${[frame.method, c.dim(`${path}:${pos.line}:${pos.column}`)].filter(Boolean).join(' ')}`), isLastStack && hasProperties ? '{' : '')
+    ctx.log(color(` ${c.dim(F_POINTER)} ${[frame.method, c.dim(`${path}:${pos.line}:${pos.column}`)].filter(Boolean).join(' ')}`))
     onStack?.(frame, pos)
 
     // reached at test file, skip the follow stack
     if (frame.file in ctx.state.filesMap)
       break
-
-    stackNumber++
   }
+  ctx.log()
+  const hasProperties = Object.keys(errorProperties).length > 0
   if (hasProperties) {
-    const propertiesString = stringify(errorProperties, 10, { printBasicPrototype: false }).substring(2)
-    ctx.log(propertiesString)
+    ctx.log(c.red(c.dim(divider())))
+    const propertiesString = stringify(errorProperties, 10, { printBasicPrototype: false })
+    ctx.log(c.red(c.bold('Serialized Error:')), c.gray(propertiesString))
   }
 }
 
diff --git a/packages/vitest/src/node/pool.ts b/packages/vitest/src/node/pool.ts
index 191b4f753556..b4f4ea59aa79 100644
--- a/packages/vitest/src/node/pool.ts
+++ b/packages/vitest/src/node/pool.ts
@@ -148,8 +148,11 @@ function createChannel(ctx: Vitest) {
         ctx.state.updateUserLog(log)
         ctx.report('onUserConsoleLog', log)
       },
+      onUnhandledRejection(err) {
+        ctx.state.catchError(err, 'Unhandled Rejection')
+      },
       onFinished(files) {
-        ctx.report('onFinished', files)
+        ctx.report('onFinished', files, ctx.state.getUnhandledErrors())
       },
     },
     {
diff --git a/packages/vitest/src/node/reporters/base.ts b/packages/vitest/src/node/reporters/base.ts
index 45c98ea48bbb..67eb13714ad0 100644
--- a/packages/vitest/src/node/reporters/base.ts
+++ b/packages/vitest/src/node/reporters/base.ts
@@ -53,9 +53,22 @@ export abstract class BaseReporter implements Reporter {
     return relative(this.ctx.config.root, path)
   }
 
-  async onFinished(files = this.ctx.state.getFiles()) {
+  async onFinished(files = this.ctx.state.getFiles(), errors = this.ctx.state.getUnhandledErrors()) {
     this.end = performance.now()
     await this.reportSummary(files)
+    if (errors.length) {
+      process.exitCode = 1
+      const errorMessage = c.red(c.bold(
+        `\nVitest catched ${errors.length} unhandled error${errors.length > 1 ? 's' : ''} during the test run. This might cause false positive tests.`
+        + '\nPlease, resolve all the errors to make sure your tests are not affected.',
+      ))
+      this.ctx.log(c.red(divider(c.bold(c.inverse(' Unhandled Errors ')))))
+      this.ctx.log(errorMessage)
+      await Promise.all(errors.map(async (err) => {
+        await this.ctx.printError(err, true, (err as ErrorWithDiff).type || 'Unhandled Error')
+      }))
+      this.ctx.log(c.red(divider()))
+    }
   }
 
   onTaskUpdate(packs: TaskResultPack[]) {
@@ -108,6 +121,7 @@ export abstract class BaseReporter implements Reporter {
   async onWatcherRerun(files: string[], trigger?: string) {
     this.watchFilters = files
 
+    this.ctx.state.clearErrors()
     this.ctx.clearScreen()
     this.ctx.log(`\n${c.inverse(c.bold(c.blue(' RERUN ')))}${trigger ? c.dim(` ${this.relative(trigger)}\n`) : ''}`)
     this.start = performance.now()
@@ -209,8 +223,7 @@ export abstract class BaseReporter implements Reporter {
   registerUnhandledRejection() {
     process.on('unhandledRejection', async (err) => {
       process.exitCode = 1
-      this.ctx.error(`\n${c.red(divider(c.bold(c.inverse(' Unhandled Rejection '))))}`)
-      await this.ctx.printError(err, true)
+      await this.ctx.printError(err, true, 'Unhandled Rejection')
       this.ctx.error('\n\n')
       process.exit(1)
     })
diff --git a/packages/vitest/src/node/reporters/default.ts b/packages/vitest/src/node/reporters/default.ts
index 3d067b4405d6..c26e00e9721c 100644
--- a/packages/vitest/src/node/reporters/default.ts
+++ b/packages/vitest/src/node/reporters/default.ts
@@ -30,10 +30,10 @@ export class DefaultReporter extends BaseReporter {
     }
   }
 
-  async onFinished(files = this.ctx.state.getFiles()) {
+  async onFinished(files = this.ctx.state.getFiles(), errors = this.ctx.state.getUnhandledErrors()) {
     await this.stopListRender()
     this.ctx.log()
-    await super.onFinished(files)
+    await super.onFinished(files, errors)
   }
 
   async onWatcherStart() {
diff --git a/packages/vitest/src/node/reporters/dot.ts b/packages/vitest/src/node/reporters/dot.ts
index 55bf2ef9f499..3bc3e820fcd7 100644
--- a/packages/vitest/src/node/reporters/dot.ts
+++ b/packages/vitest/src/node/reporters/dot.ts
@@ -17,10 +17,10 @@ export class DotReporter extends BaseReporter {
     }
   }
 
-  async onFinished(files = this.ctx.state.getFiles()) {
+  async onFinished(files = this.ctx.state.getFiles(), errors = this.ctx.state.getUnhandledErrors()) {
     await this.stopListRender()
     this.ctx.log()
-    await super.onFinished(files)
+    await super.onFinished(files, errors)
   }
 
   async onWatcherStart() {
diff --git a/packages/vitest/src/node/state.ts b/packages/vitest/src/node/state.ts
index 91d0785736ca..2db75565ebbe 100644
--- a/packages/vitest/src/node/state.ts
+++ b/packages/vitest/src/node/state.ts
@@ -1,9 +1,23 @@
-import type { File, Task, TaskResultPack, UserConsoleLog } from '../types'
+import type { ErrorWithDiff, File, Task, TaskResultPack, UserConsoleLog } from '../types'
 
 export class StateManager {
   filesMap = new Map<string, File>()
   idMap = new Map<string, Task>()
   taskFileMap = new WeakMap<Task, File>()
+  errorsSet = new Set<unknown>()
+
+  catchError(err: unknown, type: string) {
+    (err as ErrorWithDiff).type = type
+    this.errorsSet.add(err)
+  }
+
+  clearErrors() {
+    this.errorsSet.clear()
+  }
+
+  getUnhandledErrors() {
+    return Array.from(this.errorsSet.values())
+  }
 
   getFiles(keys?: string[]): File[] {
     if (keys)
diff --git a/packages/vitest/src/runtime/worker.ts b/packages/vitest/src/runtime/worker.ts
index 31703979a804..fe827f25c852 100644
--- a/packages/vitest/src/runtime/worker.ts
+++ b/packages/vitest/src/runtime/worker.ts
@@ -31,6 +31,10 @@ async function startViteNode(ctx: WorkerContext) {
     return processExit(code)
   }
 
+  process.on('unhandledRejection', (err) => {
+    rpc().onUnhandledRejection(err)
+  })
+
   const { config } = ctx
 
   const { run, collect } = (await executeInViteNode({
diff --git a/packages/vitest/src/types/general.ts b/packages/vitest/src/types/general.ts
index 7005ed271b98..8b621e0716c7 100644
--- a/packages/vitest/src/types/general.ts
+++ b/packages/vitest/src/types/general.ts
@@ -71,6 +71,7 @@ export interface ErrorWithDiff extends Error {
   actual?: any
   expected?: any
   operator?: string
+  type?: string
 }
 
 export interface ModuleGraphData {
diff --git a/packages/vitest/src/types/reporter.ts b/packages/vitest/src/types/reporter.ts
index 0f79bd53249e..c971c7ed18cb 100644
--- a/packages/vitest/src/types/reporter.ts
+++ b/packages/vitest/src/types/reporter.ts
@@ -5,7 +5,7 @@ import type { File, TaskResultPack } from './tasks'
 export interface Reporter {
   onInit?(ctx: Vitest): void
   onCollected?: (files?: File[]) => Awaitable<void>
-  onFinished?: (files?: File[]) => Awaitable<void>
+  onFinished?: (files?: File[], errors?: unknown[]) => Awaitable<void>
   onTaskUpdate?: (packs: TaskResultPack[]) => Awaitable<void>
 
   onTestRemoved?: (trigger?: string) => Awaitable<void>
diff --git a/packages/vitest/src/types/worker.ts b/packages/vitest/src/types/worker.ts
index bb2717620750..e48b05a59c25 100644
--- a/packages/vitest/src/types/worker.ts
+++ b/packages/vitest/src/types/worker.ts
@@ -22,9 +22,10 @@ export interface WorkerRPC {
   resolveId: ResolveIdFunction
   getSourceMap: (id: string, force?: boolean) => Promise<RawSourceMap | undefined>
 
-  onFinished: (files: File[]) => void
+  onFinished: (files: File[], errors?: unknown[]) => void
   onWorkerExit: (code?: number) => void
   onUserConsoleLog: (log: UserConsoleLog) => void
+  onUnhandledRejection: (err: unknown) => void
   onCollected: (files: File[]) => void
   onTaskUpdate: (pack: TaskResultPack[]) => void
 
diff --git a/packages/vitest/src/utils/source-map.ts b/packages/vitest/src/utils/source-map.ts
index aa776cbeed16..197be8649694 100644
--- a/packages/vitest/src/utils/source-map.ts
+++ b/packages/vitest/src/utils/source-map.ts
@@ -26,6 +26,8 @@ const stackBarePathRE = /at ?(.*) (.+):(\d+):(\d+)$/
 
 export async function interpretSourcePos(stackFrames: ParsedStack[], ctx: Vitest): Promise<ParsedStack[]> {
   for (const frame of stackFrames) {
+    if ('sourcePos' in frame)
+      continue
     const transformResult = ctx.server.moduleGraph.getModuleById(frame.file)?.ssrTransformResult
     if (!transformResult)
       continue

From 92642d8d0ab6925c2ce7d63168d793dfa864ca86 Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:31:58 +0300
Subject: [PATCH 4/8] feat: show "cause" error

---
 packages/vitest/src/node/core.ts  |  2 +-
 packages/vitest/src/node/error.ts | 10 ++++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
index 9fe2ec5dd203..48535a7c32c4 100644
--- a/packages/vitest/src/node/core.ts
+++ b/packages/vitest/src/node/core.ts
@@ -499,7 +499,7 @@ export class Vitest {
   }
 
   printError(err: unknown, fullStack = false, type?: string) {
-    return printError(err, type, fullStack, this)
+    return printError(err, type, fullStack, true, this)
   }
 
   onServerRestarted(fn: () => void) {
diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index ee70065adae6..15e06396a061 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -18,7 +18,7 @@ export function fileFromParsedStack(stack: ParsedStack) {
   return stack.file
 }
 
-export async function printError(error: unknown, type: string | undefined, fullStack: boolean, ctx: Vitest) {
+export async function printError(error: unknown, type: string | undefined, fullStack: boolean, showCodeFrame, ctx: Vitest) {
   let e = error as ErrorWithDiff
 
   if (typeof error === 'string') {
@@ -43,12 +43,17 @@ export async function printError(error: unknown, type: string | undefined, fullS
     printErrorType(type, ctx)
   printErrorMessage(e, ctx.console)
   printStack(ctx, stacks, nearest, errorProperties, (s, pos) => {
-    if (s === nearest && nearest) {
+    if (showCodeFrame && s === nearest && nearest) {
       const sourceCode = readFileSync(fileFromParsedStack(nearest), 'utf-8')
       ctx.log(c.yellow(generateCodeFrame(sourceCode, 4, pos)))
     }
   })
 
+  if (e.cause) {
+    e.cause.name = `Caused by: ${e.cause.name}`
+    await printError(e.cause, undefined, fullStack, false, ctx)
+  }
+
   handleImportOutsideModuleError(e.stack || e.stackStr || '', ctx)
 
   if (e.showDiff)
@@ -69,6 +74,7 @@ function getErrorProperties(e: ErrorWithDiff) {
     'name',
     'nameStr',
     'stack',
+    'cause',
     'stacks',
     'stackStr',
     'type',

From 97f7ba8b1d926800f15b507cf1ad18b25fdd67bc Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:37:47 +0300
Subject: [PATCH 5/8] refactor: pass options object to printError

---
 packages/vitest/src/node/core.ts  |  6 +++++-
 packages/vitest/src/node/error.ts | 11 +++++++++--
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
index 48535a7c32c4..96463c7455fe 100644
--- a/packages/vitest/src/node/core.ts
+++ b/packages/vitest/src/node/core.ts
@@ -499,7 +499,11 @@ export class Vitest {
   }
 
   printError(err: unknown, fullStack = false, type?: string) {
-    return printError(err, type, fullStack, true, this)
+    return printError(err, this, {
+      fullStack,
+      type,
+      showCodeFrame: true,
+    })
   }
 
   onServerRestarted(fn: () => void) {
diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index 15e06396a061..725f6d791d81 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -18,7 +18,14 @@ export function fileFromParsedStack(stack: ParsedStack) {
   return stack.file
 }
 
-export async function printError(error: unknown, type: string | undefined, fullStack: boolean, showCodeFrame, ctx: Vitest) {
+interface PrintErrorOptions {
+  type?: string
+  fullStack?: boolean
+  showCodeFrame?: boolean
+}
+
+export async function printError(error: unknown, ctx: Vitest, options: PrintErrorOptions = {}) {
+  const { showCodeFrame = true, fullStack = false, type } = options
   let e = error as ErrorWithDiff
 
   if (typeof error === 'string') {
@@ -51,7 +58,7 @@ export async function printError(error: unknown, type: string | undefined, fullS
 
   if (e.cause) {
     e.cause.name = `Caused by: ${e.cause.name}`
-    await printError(e.cause, undefined, fullStack, false, ctx)
+    await printError(e.cause, ctx, { fullStack, showCodeFrame: false })
   }
 
   handleImportOutsideModuleError(e.stack || e.stackStr || '', ctx)

From b687a0f4ffe1482581fe1031b37ffd982d90cede Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:41:05 +0300
Subject: [PATCH 6/8] chore: get only own proprty names for error object

---
 packages/vitest/src/node/error.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index 725f6d791d81..a902e5fec5be 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -89,7 +89,7 @@ function getErrorProperties(e: ErrorWithDiff) {
     'actual',
     'expected',
   ]
-  for (const key in e) {
+  for (const key of Object.getOwnPropertyNames(e)) {
     if (!skip.includes(key))
       errorObject[key] = e[key as keyof ErrorWithDiff]
   }

From 653d7558ba0eac5d06753747863f1360b9ee18e2 Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:43:28 +0300
Subject: [PATCH 7/8] fix: show that tests are failed if there are unhandled
 errors

---
 packages/vitest/src/node/reporters/base.ts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/packages/vitest/src/node/reporters/base.ts b/packages/vitest/src/node/reporters/base.ts
index 67eb13714ad0..0560c907e1da 100644
--- a/packages/vitest/src/node/reporters/base.ts
+++ b/packages/vitest/src/node/reporters/base.ts
@@ -102,7 +102,8 @@ export abstract class BaseReporter implements Reporter {
 
   async onWatcherStart() {
     const files = this.ctx.state.getFiles()
-    const failed = hasFailed(files)
+    const errors = this.ctx.state.getUnhandledErrors()
+    const failed = errors.length > 0 || hasFailed(files)
     const failedSnap = hasFailedSnapshot(files)
     if (failed)
       this.ctx.log(WAIT_FOR_CHANGE_FAIL)

From 79793f1fa10757175f53ffbecfeebb6fa0333e46 Mon Sep 17 00:00:00 2001
From: Vladimir Sheremet <sheremet.va@icloud.com>
Date: Thu, 28 Apr 2022 12:55:15 +0300
Subject: [PATCH 8/8] chore: ignore constructor, toString on error

---
 packages/vitest/src/node/error.ts | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/packages/vitest/src/node/error.ts b/packages/vitest/src/node/error.ts
index a902e5fec5be..a3f12ef63a76 100644
--- a/packages/vitest/src/node/error.ts
+++ b/packages/vitest/src/node/error.ts
@@ -88,6 +88,8 @@ function getErrorProperties(e: ErrorWithDiff) {
     'showDiff',
     'actual',
     'expected',
+    'constructor',
+    'toString',
   ]
   for (const key of Object.getOwnPropertyNames(e)) {
     if (!skip.includes(key))