Skip to content

Commit

Permalink
fix: run flaky tests on GitHub action
Browse files Browse the repository at this point in the history
  • Loading branch information
bonyuta0204 committed Jan 8, 2024
1 parent a73c1c2 commit 2ea952c
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 12 deletions.
5 changes: 0 additions & 5 deletions test/coverage-test/testing.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,8 @@ const UPDATE_SNAPSHOTS = false

const provider = process.argv[1 + process.argv.indexOf('--provider')]
const isBrowser = process.argv.includes('--browser')
const isCI = process.env.GITHUB_ACTIONS
process.env.COVERAGE_PROVIDER = provider

// TODO: Fix flakiness and enable on CI -- browser picks test files that don't exist and fails, some tests fail because of the multi environment mismatch
if (isCI)
process.exit(0)

const poolConfigs = [
{ pool: 'threads', poolOptions: { threads: { } } },
{ pool: 'forks', poolOptions: { forks: { } } },
Expand Down
4 changes: 0 additions & 4 deletions test/watch/test/file-watching.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,6 @@ afterEach(() => {
cleanups.splice(0).forEach(cleanup => cleanup())
})

// TODO: Fix flakiness and enable on CI
if (process.env.GITHUB_ACTIONS)
test.only('skip tests on CI', () => {})

test('editing source file triggers re-run', async () => {
const vitest = await runVitestCli(...cliArgs)

Expand Down
3 changes: 0 additions & 3 deletions test/watch/test/stdin.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,6 @@ afterEach(() => {
cleanups.splice(0).forEach(fn => fn())
})

// TODO: Fix flakiness and enable on CI
if (process.env.GITHUB_ACTIONS)
test.only('skip tests on CI', () => {})

Check failure on line 20 in test/watch/test/stdin.test.ts

View workflow job for this annotation

GitHub Actions / lint

More than 1 blank line not allowed
test('quit watch mode', async () => {
const vitest = await runVitestCli(...cliArgs)
Expand Down

0 comments on commit 2ea952c

Please sign in to comment.