Add retries for the integration test asserts
- Dominant language
- C#
- Stars
- 478
- Forks
- 146
- Avg merge
- 7d 21h
- Merged PRs (30d)
- 2
Description
We've hit a situation where we try to assert the final repo state, but github is not quite done digesting the data yet and returning a 404 when checking the commits (but when I manually checked out the repo 20 mins later it looked fine). This is causing the test assert to fail because it thinks the repo is uninitialized (which it probably is at that moment in time). But if the test just waited a few seconds it would probably sort itself out.
We could make our tests more robust by just wrapping the whole block of code that does the asserts into our retry library. If any asserts fail, just redo all the asserts (with the exponential backoff we have in our retry code).
Here's an example of a build failing because of this: https://github.com/github/gh-gei/actions/runs/4513628632/jobs/7948789505?pr=908#step:12:328



Contributor guide
Assessment
This issue has not been assessed yet.