Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Aug 17, 2016

There seem to be quite a few assertions failing lately. A lot of them which assert balances: https://travis-ci.org/bitcoin/bitcoin/jobs/152925845

This changes assert() to assert_equal(), so it is easier to see what the value was that caused it to fail.

@maflcko maflcko added the Tests label Aug 17, 2016
@laanwj
Copy link
Member

laanwj commented Aug 17, 2016

Concept ACK, good idea

@laanwj laanwj merged commit fa64306 into bitcoin:master Aug 18, 2016
laanwj added a commit that referenced this pull request Aug 18, 2016
fa64306 [qa] abandonconflict: Use assert_equal (MarcoFalke)
@maflcko maflcko deleted the Mf1608-qaAssert branch August 18, 2016 07:18
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Sep 21, 2016
codablock pushed a commit to codablock/dash that referenced this pull request Jan 8, 2018
fa64306 [qa] abandonconflict: Use assert_equal (MarcoFalke)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants