Skip to content

chore: avoid test flakiness in v2 API tests caused by WaitTillChain#13033

Merged
masih merged 1 commit intomasterfrom
masih/fix-flaly-api-v2-test
Apr 11, 2025
Merged

chore: avoid test flakiness in v2 API tests caused by WaitTillChain#13033
masih merged 1 commit intomasterfrom
masih/fix-flaly-api-v2-test

Conversation

@masih
Copy link
Copy Markdown
Member

@masih masih commented Apr 11, 2025

Use a height that is sufficiently behind the target height to avoid race conditions in WaitTillChain that could result in tests running earlier than state change is reflected across the system.

Observed in:

Use a height that is sufficiently behind the target height to avoid race
conditions in WaitTillChain that could result in tests running earlier
than state change is reflected across the system.

Observed in:
* https://github.com/filecoin-project/lotus/actions/runs/14401881056/job/40389322945?pr=13026
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Apr 11, 2025
@masih masih requested review from rjan90 and rvagg April 11, 2025 12:25
@masih masih added the skip/changelog This change does not require CHANGELOG.md update label Apr 11, 2025
@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Apr 11, 2025
@masih masih merged commit 6283d3e into master Apr 11, 2025
92 of 93 checks passed
@masih masih deleted the masih/fix-flaly-api-v2-test branch April 11, 2025 12:34
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants