Skip to content

test: add tests for negative waste during coin selection#25562

Merged
achow101 merged 1 commit intobitcoin:masterfrom
ishaanam:negative_waste_test
Jul 11, 2022
Merged

test: add tests for negative waste during coin selection#25562
achow101 merged 1 commit intobitcoin:masterfrom
ishaanam:negative_waste_test

Conversation

@ishaanam
Copy link
Contributor

@ishaanam ishaanam commented Jul 7, 2022

#25495 mentions that waste can be negative when the current feerate is less than the long term feerate. There are currently no waste tests for negative waste, so this PR adds two of them.

Copy link
Member

@glozow glozow left a comment

Choose a reason for hiding this comment

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

light code review ACK 98ea43d, good to have tests for negative waste

@achow101
Copy link
Member

ACK 98ea43d

@achow101 achow101 merged commit c92eb6c into bitcoin:master Jul 11, 2022
@ishaanam ishaanam deleted the negative_waste_test branch July 11, 2022 17:21
@bitcoin bitcoin locked and limited conversation to collaborators Jul 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants