Skip to content

chore: increase test timeouts to avoid flaky tests - #622

Merged
tniessen merged 1 commit into
nodejs:mainfrom
tniessen:short-timeouts-are-useless
Apr 22, 2022
Merged

chore: increase test timeouts to avoid flaky tests#622
tniessen merged 1 commit into
nodejs:mainfrom
tniessen:short-timeouts-are-useless

Conversation

@tniessen

Copy link
Copy Markdown
Member

mocha defaults to a 2000ms timeout, which frequently causes CI to fail.
Since such short timeouts are pretty much useless, vastly increase the
timeout to one minute per test.

@tniessentniessen added the test Issues related to test suites label Apr 21, 2022
@codecov

codecovBot commented Apr 21, 2022

Copy link
Copy Markdown

Codecov Report

Merging #622 (e7fd056) into main (649a3b0) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## main #622 +/- ##
=======================================
Coverage 84.13% 84.13% =======================================
Files 37 37 Lines 4065 4065 =======================================
Hits 3420 3420 Misses 645 645 

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 649a3b0...e7fd056. Read the comment docs.

mocha defaults to a 2000ms timeout, which frequently causes CI to fail.
Since such short timeouts are pretty much useless, vastly increase the
timeout to one minute per test.
@tniessen
tniessenforce-pushed the short-timeouts-are-useless branch from f146809 to e7fd056CompareApril 21, 2022 23:34
@tniessentniessen changed the title Increase test timeouts to avoid flaky testschore: increase test timeouts to avoid flaky testsApr 22, 2022
@tniessen
tniessen merged commit e798b77 into nodejs:mainApr 22, 2022
@tniessen
tniessen deleted the short-timeouts-are-useless branch April 22, 2022 22:56
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testIssues related to test suites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@tniessen@targos