feat!: remove --segfault-retry (#5514)

vitest-dev · May 6, 2024 · ed60e40 · ed60e40
1 parent dbbbe43
commit ed60e40
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 161 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -14,7 +14,6 @@ concurrency:
   cancel-in-progress: true
 
 env:
-  VITEST_SEGFAULT_RETRY: 3
   PLAYWRIGHT_BROWSERS_PATH: ${{ github.workspace }}/.cache/ms-playwright
   CYPRESS_CACHE_FOLDER: ${{ github.workspace }}/.cache/Cypress
 

diff --git a/packages/vitest/rollup.config.js b/packages/vitest/rollup.config.js
@@ -19,7 +19,6 @@ const entries = {
   'path': 'src/paths.ts',
   'index': 'src/index.ts',
   'cli': 'src/node/cli.ts',
-  'cli-wrapper': 'src/node/cli/cli-wrapper.ts',
   'node': 'src/node.ts',
   'suite': 'src/suite.ts',
   'browser': 'src/browser.ts',

diff --git a/packages/vitest/src/node/cli/cli-wrapper.ts b/packages/vitest/src/node/cli/cli-wrapper.ts
diff --git a/packages/vitest/vitest.mjs b/packages/vitest/vitest.mjs
@@ -1,2 +1,2 @@
 #!/usr/bin/env node
-import './dist/cli-wrapper.js'
+import './dist/cli.js'
diff --git a/test/config/test/exec-args.test.ts b/test/config/test/exec-args.test.ts
@@ -1,17 +1,7 @@
-import { afterAll, beforeAll, expect, test } from 'vitest'
+import { expect, test } from 'vitest'
 import { execa } from 'execa'
 import { runVitest } from '../../test-utils'
 
-// VITEST_SEGFAULT_RETRY messes with the node flags, as can be seen in packages/vitest/src/node/cli-wrapper.ts
-// so here we remove it to make sure the tests are not affected by it
-const ORIGIN_VITEST_SEGFAULT_RETRY = process.env.VITEST_SEGFAULT_RETRY
-beforeAll(() => {
-  delete process.env.VITEST_SEGFAULT_RETRY
-})
-afterAll(() => {
-  process.env.VITEST_SEGFAULT_RETRY = ORIGIN_VITEST_SEGFAULT_RETRY
-})
-
 test.each([
   { pool: 'forks', execArgv: ['--hash-seed=1', '--random-seed=1', '--no-opt'] },
   { pool: 'threads', execArgv: ['--inspect-brk'] },