tests: move pwalletMain to wallet test fixture #11718

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2017_11_wallet_test_fixture changing 4 files +11 −15

laanwj commented at 1:14 PM on November 18, 2017: member

Scope the variable instead of using an external global; this is how test fixtures are intended to be used.

Followup to #11713.
laanwj added the label Tests on Nov 18, 2017
laanwj force-pushed on Nov 18, 2017

tests: move pwalletMain to wallet test fixture

Scope the variable instead of using an external global; this is how test
fixtures are intended to be used.

Followup to #11713.

49bd6590fe

laanwj force-pushed on Nov 18, 2017
jamesob commented at 7:50 PM on November 18, 2017: member

utACK https://github.com/bitcoin/bitcoin/pull/11718/commits/49bd6590feb83ee422e489fd3930e8b700c5b4a7
sipa commented at 8:11 PM on November 18, 2017: member

utACK 49bd6590feb83ee422e489fd3930e8b700c5b4a7
laanwj merged this on Nov 19, 2017
laanwj closed this on Nov 19, 2017
laanwj referenced this in commit 9cdd2bcf67 on Nov 19, 2017
MarcoFalke commented at 4:17 PM on November 27, 2017: member

post merge utACK 49bd6590feb83ee422e489fd3930e8b700c5b4a7
PastaPastaPasta referenced this in commit 152cf71e22 on Feb 13, 2020
PastaPastaPasta referenced this in commit 9b636ee799 on Feb 27, 2020
PastaPastaPasta referenced this in commit 6fc9f432be on Mar 14, 2020
PastaPastaPasta referenced this in commit bdb2f7e9dd on Mar 21, 2020
PastaPastaPasta referenced this in commit d94dd15de7 on Mar 24, 2020
ckti referenced this in commit 21277153dc on Mar 28, 2021
bitcoin locked this on Sep 8, 2021

Contributors

Labels