mirror of
https://github.com/bitcoin/bitcoin.git
synced 2026-01-20 15:19:07 +01:00
66667d6512test: Use same rpc timeout for authproxy and cli (MarcoFalke) Pull request description: It seems odd to use different timeouts (and timeout factors) depending on whether the Python RPC proxy is used, or the bitcoin rpc command line interface. Fix it by using the same timeout. This can be tested by introducing a timeout error and checking it happens with and without `--usecli` after the exact same time. Example timeout error: ```diff diff --git a/test/functional/mining_template_verification.py b/test/functional/mining_template_verification.py index de0833c596..e0f93a2b1e 100755 --- a/test/functional/mining_template_verification.py +++ b/test/functional/mining_template_verification.py @@ -173,7 +173,7 @@ class MiningTemplateVerificationTest(BitcoinTestFramework): self.log.info("Submitting this block should succeed") assert_equal(node.submitblock(block.serialize().hex()), None) - node.waitforblockheight(2) + node.waitforblockheight(200000) def transaction_test(self, node, block_0_height, tx): self.log.info("make block template with a transaction") ``` Example cmd: `./bld-cmake/test/functional/mining_template_verification.py --timeout-factor=0.1 --usecli`. ACKs for top commit: brunoerg: ACK66667d6512stickies-v: tACK66667d6512Tree-SHA512: c8c21d8b9fb60ab192e3bbd45b317b96a40e10bf03704148613ac3cbdaae4abc2c03c4afbd504309ea0958201267c0d2a4bc5b40aa020917175c47e080ffe292