qa: Run tests with UPnP disabled #16646

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:add_upnp_to_test_framework_conf changing 1 files +1 −0

fanquake commented at 2:15 AM on August 19, 2019: member
This replaces #16560 by adding upnp=0 to bitcoin.conf rather than passing it to nodes.

Needed for builds configured with --enable-upnp-default

You can test this change using:
```
./configure --enable-upnp-default && make -j6 && test/functional/test_runner.py feature_config_args.py
```
on master the test will fail without this change.

Bugfix: QA: Run tests with UPnP disabled

Needed for builds configured with --enable-upnp-default

b168dd30cf

fanquake added the label Tests on Aug 19, 2019
fanquake cross-referenced this on Aug 19, 2019 from issue Bugfix: QA: Run tests with UPnP disabled by luke-jr
practicalswift commented at 7:24 AM on August 19, 2019: contributor

ACK b168dd30cf71ac176e271bc610b0b1a79ceaf075 -- diff looks correct
laanwj renamed this:
~~Bugfix: QA: Run tests with UPnP disabled~~
qa: Run tests with UPnP disabled
on Aug 19, 2019
laanwj referenced this in commit 27ee0cc5a6 on Aug 19, 2019
laanwj merged this on Aug 19, 2019
laanwj closed this on Aug 19, 2019
fanquake deleted the branch on Aug 19, 2019
luke-jr referenced this in commit 200a764469 on Aug 23, 2019
luke-jr cross-referenced this on Aug 23, 2019 from issue [0.18.2] Backports by fanquake
fanquake referenced this in commit 9ec2c026a8 on Aug 24, 2019
luke-jr referenced this in commit 70b48569a4 on Sep 3, 2019
fanquake referenced this in commit 9fe8d283e0 on Sep 23, 2019
laanwj referenced this in commit 29d70264fb on Nov 25, 2019
linuxsh2 referenced this in commit b7e4be7f61 on Aug 11, 2021
bitcoin locked this on Dec 16, 2021

Contributors

Labels