test: DEFI-2100: Migrate disable-api-if-not-fully-synced-flag e2e test to PocketIC - #537
Draft
mbjorkqvist wants to merge 1 commit into
Draft
test: DEFI-2100: Migrate disable-api-if-not-fully-synced-flag e2e test to PocketIC#537mbjorkqvist wants to merge 1 commit into
mbjorkqvist wants to merge 1 commit into
Conversation
…t to PocketIC Replaces the dfx-based disable-api-if-not-fully-synced-flag.sh e2e test with a Rust integration test driven by PocketIC, continuing the DEFI-2100 migration of e2e tests off dfx. The test reuses the existing source canister (which serves 2 full blocks plus 3 next block headers, leaving the canister permanently not-fully-synced) and asserts both directions of the flag: with disable_api_if_not_fully_synced enabled every bitcoin_* API call traps with "Canister state is not fully synced.", and with it disabled the same calls succeed and return real data. Folds the test into the e2e-scenario CI matrix (building the source canister wasm), drops the standalone dfx job, and excludes the crate from the workspace cargo test run since it needs the prebuilt bitcoin canister wasm. Adds a query_raw helper to the shared e2e harness, mirroring update_raw. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Continues the DEFI-2100 migration of e2e tests off dfx and onto PocketIC.
Replaces the dfx-based
disable-api-if-not-fully-synced-flag.shscript with a Rust integration test driven by PocketIC. The test reuses the existing source canister — which serves 2 full blocks plus 3 next block headers, leaving the canister permanently not-fully-synced — and asserts both directions of thedisable_api_if_not_fully_syncedflag:bitcoin_*API call (update and query) traps with "Canister state is not fully synced.";CI changes: folds the test into the
e2e-scenariomatrix (building the source canister wasm), drops the standalone dfx job and its gate check, and excludes the crate from the workspacecargo testrun since it needs the prebuilt bitcoin canister wasm. Adds aquery_rawhelper to the shared e2e harness, mirroringupdate_raw.🤖 Generated with Claude Code