Skip to content

test(ckETH_Minter): DEFI-2559: Add test to verify minter cannot be stopped while it is scraping blocks#7962

Merged
mbjorkqvist merged 8 commits intomasterfrom
mathias/DEFI-cketh-minter-open-call-context
Dec 18, 2025
Merged

test(ckETH_Minter): DEFI-2559: Add test to verify minter cannot be stopped while it is scraping blocks#7962
mbjorkqvist merged 8 commits intomasterfrom
mathias/DEFI-cketh-minter-open-call-context

Conversation

@mbjorkqvist
Copy link
Copy Markdown
Contributor

@mbjorkqvist mbjorkqvist commented Dec 5, 2025

Add a StateMachine test to verify that the ckETH minter cannot be stopped while it is scraping blocks - in particular, while it is retrieving blocks in a loop, it will continue making HTTPS outcalls to retrieve new blocks until it reaches the tip, even if a controller has attempted to stop the canister in the meantime.

@github-actions github-actions Bot added the test label Dec 5, 2025
@mbjorkqvist mbjorkqvist changed the title test(ckETH_Minter): Upgrade minter to the same wasm to verify it can be stopped test(ckETH_Minter): DEFI-2559: Add test to verify minter cannot be stopped while it is scraping blocks Dec 8, 2025
@mbjorkqvist mbjorkqvist marked this pull request as ready for review December 8, 2025 18:11
@mbjorkqvist mbjorkqvist requested a review from a team December 8, 2025 18:11
@github-actions github-actions Bot added the @defi label Dec 8, 2025
Copy link
Copy Markdown
Contributor

@gregorydemay gregorydemay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mbjorkqvist for digging into this!

Comment thread rs/ethereum/cketh/minter/tests/cketh.rs Outdated
Comment thread rs/ethereum/cketh/test_utils/src/lib.rs
Comment thread rs/ethereum/cketh/test_utils/src/lib.rs Outdated
Copy link
Copy Markdown
Contributor

@gregorydemay gregorydemay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mbjorkqvist for that PR!

@mbjorkqvist mbjorkqvist added this pull request to the merge queue Dec 18, 2025
Merged via the queue into master with commit c51ed71 Dec 18, 2025
40 checks passed
@mbjorkqvist mbjorkqvist deleted the mathias/DEFI-cketh-minter-open-call-context branch December 18, 2025 20:56
mraszyk pushed a commit that referenced this pull request Jan 5, 2026
…opped while it is scraping blocks (#7962)

Add a `StateMachine` test to verify that the ckETH minter cannot be
stopped while it is scraping blocks - in particular, while it is
retrieving blocks in a loop, it will continue making HTTPS outcalls to
retrieve new blocks until it reaches the tip, even if a controller has
attempted to stop the canister in the meantime.

---------

Co-authored-by: IDX GitHub Automation <infra+github-automation@dfinity.org>
github-merge-queue Bot pushed a commit that referenced this pull request Mar 3, 2026
Follow-up on #7962 to ensure that the minter stops scraping the logs
when it is in the stopping state.
This is simply done by upgrading `ic-canister-runtime` to `0.2.0` which
now offers that behavior by default.

---------

Co-authored-by: IDX GitHub Automation <infra+github-automation@dfinity.org>
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