Instead of skipping the whole test, only skip the wallet specific section of a test if the wallet is not compiled in. This is mostly an indentation change, so can be reviewed with --ignore-all-space.
qa: Run more tests with wallet disabled #14324
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1809-qaWalletNo changing 6 files +63 −54-
MarcoFalke commented at 10:09 PM on September 25, 2018: member
-
qa: Run more tests with wallet disabled faa4043c66
- fanquake added the label Tests on Sep 25, 2018
-
DrahtBot commented at 1:20 AM on September 26, 2018: contributor
<!--e57a25ab6845829454e8d69fc972939a-->Reviewers, this pull request conflicts with the following ones:
- #13621 (Check for datadir after the config files were read by Flowdalic)
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.
- DrahtBot cross-referenced this on Sep 26, 2018 from issue Check for datadir after the config files were read by Flowdalic
- jamesob approved
-
jamesob commented at 2:13 PM on September 27, 2018: member
-
DrahtBot commented at 3:14 AM on September 28, 2018: contributor
<!--32850dd3fdea838b4049e64f46995ea2-->
Coverage Change (pull 14324) Reference (master) Lines +0.0000 % 87.0361 % Functions +0.0154 % 84.1130 % Branches -0.0047 % 51.5451 % - MarcoFalke merged this on Oct 8, 2018
- MarcoFalke closed this on Oct 8, 2018
- MarcoFalke referenced this in commit d12e3d9acc on Oct 8, 2018
- MarcoFalke deleted the branch on Oct 8, 2018
- Bushstar cross-referenced this on Oct 11, 2018 from issue Updates from bitcoin/master by Bushstar
- MarcoFalke cross-referenced this on Oct 23, 2018 from issue [wallet] Deprecate generate RPC method by jnewbery
- deadalnix referenced this in commit 4340dc14f4 on May 20, 2020
- bitcoin locked this on Sep 8, 2021
Contributors
Labels