Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chain sync test #1257

Merged
merged 4 commits into from
Jun 11, 2024
Merged

chain sync test #1257

merged 4 commits into from
Jun 11, 2024

Conversation

ermalkaleci
Copy link
Contributor

@ermalkaleci ermalkaleci commented Jun 7, 2024

Add a job on release github action to test chain sync for at least 1000 blocks

@ermalkaleci ermalkaleci added ci This PR/Issue is related to the topic "CI" tests If the PR/issue is related to tests, like xcm-simulator tests, rpc-tests etc. labels Jun 7, 2024
Copy link
Member

@Dinonard Dinonard left a comment

Choose a reason for hiding this comment

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

Please make sure to add PR summaries/description.

It's the first thing reviewers sees after the PR title, and it's helpful to set the stage for the review. Even if the PR isn't very complex or big.

.github/workflows/chain-sync.yml Outdated Show resolved Hide resolved
scripts/sync.sh Outdated Show resolved Hide resolved
@ermalkaleci ermalkaleci requested a review from Dinonard June 10, 2024 21:04
Copy link

Code Coverage

Package Line Rate Branch Rate Health
precompiles/unified-accounts/src 100% 0%
pallets/collator-selection/src 92% 0%
precompiles/xcm/src 73% 0%
pallets/astar-xcm-benchmarks/src/fungible 100% 0%
precompiles/dispatch-lockdrop/src 86% 0%
pallets/price-aggregator/src 72% 0%
pallets/xvm/src 54% 0%
precompiles/substrate-ecdsa/src 74% 0%
pallets/astar-xcm-benchmarks/src/generic 100% 0%
pallets/dapp-staking-v3/src/benchmarking 98% 0%
precompiles/dapp-staking-v3/src 90% 0%
primitives/src 61% 0%
precompiles/dapp-staking-v3/src/test 0% 0%
pallets/dapp-staking-v3/src 92% 0%
primitives/src/xcm 64% 0%
pallets/unified-accounts/src 86% 0%
pallets/xc-asset-config/src 64% 0%
pallets/ethereum-checked/src 79% 0%
chain-extensions/pallet-assets/src 56% 0%
chain-extensions/xvm/src 0% 0%
precompiles/assets-erc20/src 81% 0%
chain-extensions/types/assets/src 0% 0%
pallets/dapp-staking-v3/rpc/runtime-api/src 0% 0%
pallets/dapp-staking-v3/src/test 0% 0%
precompiles/xvm/src 75% 0%
pallets/static-price-provider/src 52% 0%
pallets/astar-xcm-benchmarks/src 88% 0%
chain-extensions/types/xvm/src 0% 0%
chain-extensions/unified-accounts/src 0% 0%
precompiles/sr25519/src 64% 0%
chain-extensions/types/unified-accounts/src 0% 0%
pallets/dapp-staking-migration/src 0% 0%
pallets/oracle-benchmarks/src 0% 0%
pallets/dynamic-evm-base-fee/src 92% 0%
pallets/inflation/src 83% 0%
Summary 78% (3600 / 4635) 0% (0 / 0)

Minimum allowed line rate is 50%

Copy link
Member

@Dinonard Dinonard left a comment

Choose a reason for hiding this comment

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

LGTM!

@ermalkaleci ermalkaleci merged commit 05f63c3 into master Jun 11, 2024
9 checks passed
@ermalkaleci ermalkaleci deleted the feat/chain-sync-test branch June 11, 2024 08:16
@ermalkaleci ermalkaleci mentioned this pull request Jun 18, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This PR/Issue is related to the topic "CI" tests If the PR/issue is related to tests, like xcm-simulator tests, rpc-tests etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants