test: loading assumeutxo snapshot start states #29681

pull BrandonOdiwuor wants to merge 2 commits into bitcoin:master from BrandonOdiwuor:test_assumeutxo_snapshot_height changing 1 files +21 −5
  1. BrandonOdiwuor commented at 1:33 pm on March 20, 2024: contributor

    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
  2. DrahtBot commented at 1:33 pm on March 20, 2024: contributor

    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 (Assumeutxo: bugfix on loadtxoutset with a divergent chain + test 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.

  3. DrahtBot added the label Tests on Mar 20, 2024
  4. DrahtBot added the label Needs rebase on Mar 20, 2024
  5. BrandonOdiwuor force-pushed on Mar 22, 2024
  6. BrandonOdiwuor force-pushed on Mar 23, 2024
  7. BrandonOdiwuor force-pushed on Mar 23, 2024
  8. DrahtBot added the label CI failed on Mar 23, 2024
  9. DrahtBot commented at 7:40 am on March 23, 2024: 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

  10. DrahtBot removed the label Needs rebase on Mar 23, 2024
  11. DrahtBot removed the label CI failed on Mar 23, 2024
  12. BrandonOdiwuor renamed this:
    test: loading assumeutxo snapshot with current chaintip at snapshot_base_height
    test: loading assumeutxo snapshot start states
    on Mar 25, 2024
  13. DrahtBot added the label CI failed on Mar 25, 2024
  14. maflcko commented at 12:54 pm on April 23, 2024: member
    Are you still working on this?
  15. test: loading assumeutxo snapshot with tip at snapshot base height 867b245aad
  16. test: load assumeutxo snapshot at a descendant of snapshot block e724b44009
  17. BrandonOdiwuor force-pushed on Apr 24, 2024
  18. BrandonOdiwuor commented at 4:32 pm on July 8, 2024: contributor
  19. BrandonOdiwuor closed this on Jul 8, 2024


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-22 06:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me