Skip to content

chore(coprocessor): improve host-listener tests performance#2113

Merged
mergify[bot] merged 5 commits intomainfrom
panos/improve-host-listener-test-performance
Mar 16, 2026
Merged

chore(coprocessor): improve host-listener tests performance#2113
mergify[bot] merged 5 commits intomainfrom
panos/improve-host-listener-test-performance

Conversation

@PanGan21
Copy link
Copy Markdown
Contributor

@PanGan21 PanGan21 commented Mar 13, 2026

Summary

  • Reduce NB_EVENTS_PER_WALLET from 200 to 50 in restart/reorg tests — total events are sufficient to test catchup, restart, and reorgs
  • Replace hardcoded sleep waits with poll-based approach that exit as soon as the expected condition is met
  • Reduce slow-lane test block time from 3.0s to 1.0s and replace sleeps with Anvil block number polling

Performance

  • main branch: 14m 45s
  • current branch: 7m 42s

Ref: https://github.com/zama-ai/fhevm-internal/issues/1155

…tests from 565 to 480 seconds by replacing hardcoded sleeps with polling
@PanGan21 PanGan21 requested a review from a team as a code owner March 13, 2026 13:44
@cla-bot cla-bot bot added the cla-signed label Mar 13, 2026
@PanGan21 PanGan21 changed the title chore(coprocessor): Improve host-listener tests performance chore(coprocessor): improve host-listener tests performance Mar 13, 2026
…tests from 480 to 450 secs by replacing fixed sleeps with block polling in slow lane tests
@PanGan21 PanGan21 force-pushed the panos/improve-host-listener-test-performance branch from 704ad74 to 9de47d8 Compare March 13, 2026 13:47
@github-actions
Copy link
Copy Markdown

Changed Lines Coverage

Coverage of added/modified lines: N/A

Per-file breakdown

Diff Coverage

Diff: origin/main...HEAD, staged and unstaged changes

No lines with coverage information in this diff.

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 13, 2026

🧪 CI Insights

Here's what we observed from your CI run for ab2d23b.

🟢 All jobs passed!

But CI Insights is watching 👀

@PanGan21
Copy link
Copy Markdown
Contributor Author

@mergify queue

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 16, 2026

Merge Queue Status

This pull request spent 2 hours 47 minutes 42 seconds in the queue, including 1 hour 46 minutes 21 seconds running CI.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • branch-protection-review-decision = APPROVED [🛡 GitHub branch protection]
  • check-success = run-e2e-tests / fhevm-e2e-test
  • any of [🛡 GitHub branch protection]:
    • check-success = common-pull-request/lint (bpr)
    • check-neutral = common-pull-request/lint (bpr)
    • check-skipped = common-pull-request/lint (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-listener-tests/cargo-tests (bpr)
    • check-success = coprocessor-cargo-listener-tests/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-success = coprocessor-cargo-test/cargo-tests (bpr)
    • check-neutral = coprocessor-cargo-test/cargo-tests (bpr)
    • check-skipped = coprocessor-cargo-test/cargo-tests (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-success = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-neutral = coprocessor-dependency-analysis/dependencies-check (bpr)
    • check-skipped = coprocessor-dependency-analysis/dependencies-check (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-neutral = gateway-contracts-deployment-tests/sc-deploy (bpr)
    • check-success = gateway-contracts-deployment-tests/sc-deploy (bpr)
  • any of [🛡 GitHub branch protection]:
    • check-skipped = kms-connector-tests/test-connector (bpr)
    • check-neutral = kms-connector-tests/test-connector (bpr)
    • check-success = kms-connector-tests/test-connector (bpr)

mergify bot added a commit that referenced this pull request Mar 16, 2026
mergify bot added a commit that referenced this pull request Mar 16, 2026
@mergify mergify bot merged commit c5178f3 into main Mar 16, 2026
63 checks passed
@mergify mergify bot deleted the panos/improve-host-listener-test-performance branch March 16, 2026 12:39
@mergify mergify bot removed the queued label Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants