Skip to content

test: Set per-test timeout - #301

Open
abmusse wants to merge 1 commit into
masterfrom
tests-set-timeout
Open

test: Set per-test timeout#301
abmusse wants to merge 1 commit into
masterfrom
tests-set-timeout

Conversation

@abmusse

@abmusseabmusse commented Jul 18, 2020

Copy link
Copy Markdown
Member

The per test timeout value was determined by doubling the average runtime for each test listed in #300 (comment).

For the hooks that check if an object exists I've disabled timeout at the hook level. https://mochajs.org/#hook-level

@abmusse
abmusse requested a review from kadlerJuly 20, 2020 16:16
@abmusse
abmusse marked this pull request as ready for review July 20, 2020 16:16
@abmusseabmusse added keep-open Exempts stale action from auto closing the issue/pr. and removed keep-open Exempts stale action from auto closing the issue/pr. labels Aug 13, 2020
@github-actions

Copy link
Copy Markdown

👋 Hi! This pull request has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed.

@abmusseabmusse added the keep-open Exempts stale action from auto closing the issue/pr. label Sep 16, 2020
@kadlerkadler removed the stale label Oct 6, 2020
@kadlerkadler reopened this Oct 6, 2020
@abmusseabmusse linked an issue Oct 21, 2020 that may be closed by this pull request
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

keep-openExempts stale action from auto closing the issue/pr.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set per-test timeout values

2 participants

@abmusse@kadler