[qa] rpc-test: Normalize assert() #7720

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1603-qaAssertNorm changing 5 files +34 −50

MarcoFalke commented at 6:07 PM on March 19, 2016: member
Refactoring to make the test easier to read and remove redundant code:
- assert_equal( $boolean, True) can be written as assert( $boolean )
- Remove usage of errorString et al., which is error prone when the string is not erased after usage. Also made some of them more verbose/strict.
[qa] rpc-test: Normalize assert() fab3890156
jmcorgan commented at 6:17 PM on March 19, 2016: contributor

utACK
jonasschnelli added the label Tests on Mar 19, 2016
dcousens commented at 12:38 AM on March 21, 2016: contributor

utACK
laanwj merged this on Mar 21, 2016
laanwj closed this on Mar 21, 2016
laanwj referenced this in commit 3b4324b1ed on Mar 21, 2016
laanwj commented at 11:47 AM on March 21, 2016: member

utACK fab3890
MarcoFalke deleted the branch on Mar 21, 2016
MarcoFalke referenced this in commit d89fbfe899 on Apr 5, 2016
nomnombtc referenced this in commit 206148425b on Nov 11, 2016
nomnombtc referenced this in commit 82383174b2 on Nov 12, 2016
nomnombtc referenced this in commit 27505fb5da on Nov 12, 2016
nomnombtc referenced this in commit 401192e64d on Nov 13, 2016
sickpig referenced this in commit 6bd4917661 on Nov 14, 2016
DrahtBot locked this on Sep 8, 2021

Contributors

Labels