Skip to content

Comments

Retry Failing Integration Tests up to 3 times#389

Merged
rfelber merged 3 commits intomainfrom
retry-integration-tests
Apr 23, 2021
Merged

Retry Failing Integration Tests up to 3 times#389
rfelber merged 3 commits intomainfrom
retry-integration-tests

Conversation

@J12934
Copy link
Member

@J12934 J12934 commented Apr 23, 2021

Description

This uses the https://www.npmjs.com/package/jest-retries lib to achieve the retry functionality.
This was lacking the ability to pass timeout to the test function, so i vendored the lib and modified it, the code is luckily relatively short and straight forward.

close #224

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.
  • Make codeclimate checks happy

@J12934 J12934 added testing Improvements or additions regarding the test setup ci Changes to the continuous integration setup labels Apr 23, 2021
@J12934 J12934 requested a review from nigthknight April 23, 2021 12:48
@J12934 J12934 self-assigned this Apr 23, 2021
@rfelber rfelber merged commit fffaebc into main Apr 23, 2021
@rfelber rfelber deleted the retry-integration-tests branch April 23, 2021 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to the continuous integration setup testing Improvements or additions regarding the test setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve the SCB CI pipeline to prevent flaky timeouts

2 participants