tests: Skip rpc_zmq.py when ZMQ is disabled #13646

pull domob1812 wants to merge 1 commits into bitcoin:master from domob1812:skip-zmq-test changing 1 files +9 −1

domob1812 commented at 3:55 PM on July 12, 2018: contributor

Skip the new rpc_zmq.py test if ZMQ is disabled; otherwise it fails because the RPC method getzmqnotifications is not available.

This fixes an oversight in #13570.

tests: Skip rpc_zmq.py when ZMQ is disabled.

Skip the new rpc_zmq.py test if ZMQ is disabled; otherwise it fails
because the RPC method getzmqnotifications is not available.

This fixes an oversight in
https://github.com/bitcoin/bitcoin/pull/13570.

8fe3994145

MarcoFalke added this to the milestone 0.17.0 on Jul 12, 2018
MarcoFalke added the label Tests on Jul 12, 2018
DrahtBot commented at 4:11 PM on July 12, 2018: member
Note to reviewers: This pull request conflicts with the following ones:
- #13645 ([tests] skip rpc_zmq functional test as necessary by jamesob)
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.
MarcoFalke commented at 10:01 AM on July 13, 2018: member

Thanks! A fix was merged into master.
MarcoFalke closed this on Jul 13, 2018
domob1812 deleted the branch on Jul 14, 2018
DrahtBot locked this on Sep 8, 2021

Contributors

Labels

Milestone
0.17.0

Linked (view graph)

#13570 RPC: Add new "getzmqnotifications" method