Skip to content

test: lower polling times - #738

Open
alxdca wants to merge 5 commits into
mainfrom
test/lower-polling-intervals
Open

test: lower polling times#738
alxdca wants to merge 5 commits into
mainfrom
test/lower-polling-intervals

Conversation

@alxdca

@alxdca alxdca commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Reduce Nitro testnode and client polling intervals to speed up integration tests.
  • Upgrade testnode CI images to v0.2.14 and use their bundled token bridge deployment tooling.
  • Preserve Viem polling intervals when adapting public clients to ethers providers.
  • Split the build into an independent CI job so validation jobs can run in parallel.
  • Remove the obsolete token bridge contracts Docker image build.

Why / Context

Integration tests currently spend unnecessary time waiting on default polling intervals and repeat build work across CI jobs.

Using the newer testnode images and their bundled token bridge tooling simplifies setup, while shorter polling intervals improve test feedback time. Separating the build job also allows the remaining validation jobs to execute independently.

@alxdca
alxdca force-pushed the test/lower-polling-intervals branch from 8c7262a to 97128cc Compare August 3, 2026 11:19
Comment thread .github/workflows/build-test.yml Fixed
Comment thread .github/workflows/build-test.yml Fixed
@alxdca
alxdca force-pushed the test/lower-polling-intervals branch from bce10ad to c54503c Compare August 3, 2026 16:50
@alxdca
alxdca requested review from douglance and spsjvc August 4, 2026 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants