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

test: loading assumeutxo snapshot start states #29681

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BrandonOdiwuor
Copy link
Contributor

@BrandonOdiwuor BrandonOdiwuor commented Mar 20, 2024

Fixes: #28648

Add assumeutxo test:

  • loading snapshot with current chain tip at snapshot_base_height
  • loading snapshot with current chain tip a descendant of snapshot block

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 20, 2024

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #29996 (test: Assumeutxo: import snapshot in a node with a divergent chain by alfonsoromanz)
  • #29553 (assumeutxo: Add dumptxoutset height param, remove shell scripts by fjahr)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@BrandonOdiwuor BrandonOdiwuor force-pushed the test_assumeutxo_snapshot_height branch 3 times, most recently from a2fdb5e to bd36ec3 Compare March 23, 2024 07:39
@DrahtBot
Copy link
Contributor

🚧 At least one of the CI tasks failed. Make sure to run all tests locally, according to the
documentation.

Possibly this is due to a silent merge conflict (the changes in this pull request being
incompatible with the current code in the target branch). If so, make sure to rebase on the latest
commit of the target branch.

Leave a comment here, if you need help tracking down a confusing failure.

Debug: https://github.com/bitcoin/bitcoin/runs/23007390361

@BrandonOdiwuor BrandonOdiwuor changed the title test: loading assumeutxo snapshot with current chaintip at snapshot_base_height test: loading assumeutxo snapshot start states Mar 25, 2024
@maflcko
Copy link
Member

maflcko commented Apr 23, 2024

Are you still working on this?

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.

test: Write assumeutxo tests
3 participants