tests: Add assert_less_than and assert_less_than_or_equal to test framework #23337

pull dougEfresh wants to merge 1 commits into bitcoin:master from dougEfresh:23119-assert_less changing 10 files +43 −21
  1. dougEfresh commented at 11:50 am on October 22, 2021: contributor
    per #23119 and #23127 adding assert_less* helper functions.
  2. tests: Add assert_less_than and assert_less_than_or_equal to test framework 2f9d6aa03f
  3. fanquake added the label Tests on Oct 22, 2021
  4. vincenzopalazzo commented at 11:59 am on October 22, 2021: none
    NIT, IMO this change should be inside the #23127 it is not useful at all to make these changes in X different PRs as mentioned in different messages already and discussed on IRC.
  5. dougEfresh commented at 12:12 pm on October 22, 2021: contributor
    @vincenzopalazzo noob here. what would you like me to do? open a pr to your branch?
  6. DrahtBot commented at 1:17 pm on October 22, 2021: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #23127 (tests: Use assert_equal* utils where possible by vincenzopalazzo)
    • #22867 (test: Extend test coverage of BIP125 and document confusing/inconsistent behavior by mjdietzx)
    • #17794 (test: Use assert_greater_than_or_equal helper method to aid debugging. by jbampton)

    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.

  7. dougEfresh closed this on Nov 2, 2021

  8. DrahtBot locked this on Nov 2, 2022

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-10-04 19:12 UTC

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