Skip to content

Conversation

@xdustinface
Copy link
Collaborator

Add integration test infrastructure for SPV sync against a local dashd regtest node using a pre-generated blockchain (yet to be improved) which is stored in https://github.com/xdustinface/regtest-blockchain for now.

This currently adds just one basic wallet sync test which still fails at the moment on v0.41-dev. The PRs to fix it will follow soon:

test_wallet_sync

  • Connecting the DashSpvClient to the dashd node
  • Fully sync up to it with a wallet loaded that contains bunch of transactions
  • Validate the wallet data (utxos, transactions, balance)

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 10, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@xdustinface xdustinface force-pushed the tests/dashd-sync branch 2 times, most recently from fe69fa3 to 12dee54 Compare December 10, 2025 05:51
@xdustinface xdustinface changed the title tests: Add dashd_sync.rs with test_wallet_sync tests: add dashd_sync.rs with test_wallet_sync Dec 28, 2025
@xdustinface xdustinface changed the base branch from v0.41-dev to v0.42-dev December 30, 2025 22:35
Add integration test infrastructure for SPV sync against a local `dashd` regtest node using a pre-generated blockchain which is currently stored in https://github.com/xdustinface/regtest-blockchain.

This currently adds just one test which still fails at the moment on `v0.41-dev`. The PRs to fix it will follow _soon_.
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