Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix flakiness of MagazineActionReportsApiTest::testApiCanAcceptReport #1393

Merged
merged 4 commits into from
Jan 25, 2025

Conversation

BentiGorlich
Copy link
Member

Move the timestamp creation up so it is guaranteed to be lower than the one in the json data

Move the timestamp creation up so it is guaranteed to be lower than the one in the json data
@BentiGorlich BentiGorlich added bug Something isn't working testing Unit/Functional/Integration test issues and pull requests labels Jan 25, 2025
@BentiGorlich BentiGorlich requested a review from melroy89 January 25, 2025 15:16
@BentiGorlich BentiGorlich self-assigned this Jan 25, 2025
@BentiGorlich BentiGorlich merged commit f88c179 into main Jan 25, 2025
7 checks passed
@BentiGorlich BentiGorlich deleted the fix/flakiness-of-MagazineActionReportsApiTest branch January 25, 2025 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working testing Unit/Functional/Integration test issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants