-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Flapy integration test #910
Comments
obol-bulldozer bot
pushed a commit
that referenced
this issue
Aug 8, 2022
Disables flapping monitoring API test until it is fixed. This isn't a critical test either. category: test ticket: #910
When last did this flap? |
https://github.com/ObolNetwork/charon/runs/7656657905?check_suite_focus=true |
Lets close it then until we can find another occurrence |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
馃悶 Bug Report
Description
We have some integration tests that are slow, which is why they run CI builds. However, some of these tests require some condition to satisfy within a specific duration. If condition isn't satisfied in that duration the test will fail with error:
Condition never satisfied
.Has this worked before in a previous version?
We have observed this flapping of test in previous versions as well.
馃敩 Minimal Reproduction
This is a non-deterministic test which uses assert.Eventually to trigger a condition within a specific duration and a ticker:
charon/app/app_test.go
Line 324 in d05c54d
We can reproduce it by running it multiple times on local system.
馃敟 Error
https://github.com/ObolNetwork/charon/runs/7656657905?check_suite_focus=true
The text was updated successfully, but these errors were encountered: