diff options
| author | fanquake <fanquake@gmail.com> | 2023-05-04 17:19:15 +0100 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2023-05-04 17:19:26 +0100 |
| commit | 6c7ebcc14b7908a67a8f8764b398e76c8fb4fe8b (patch) | |
| tree | 819fef884b4dc91652414eca8f9f95fd407346fb | |
| parent | 30bf70c8b60f070c47ce9b51b5d2520c386d3559 (diff) | |
| parent | 7e3d4f8e86e86f32d8911abd458b9e7c939ef3d5 (diff) | |
Merge bitcoin/bitcoin#27422: test: add coverage to rpc_scantxoutset.py
7e3d4f8e86e86f32d8911abd458b9e7c939ef3d5 test: add coverage to ensure the first arg of scantxoutset is needed (ismaelsadeeq)
Pull request description:
Include a test that checks whether the first argument of scantxoutset RPC call "start" is required.
The rpc call should fail if the "start" argument is not provided.
ACKs for top commit:
MarcoFalke:
lgtm ACK 7e3d4f8e86e86f32d8911abd458b9e7c939ef3d5
Tree-SHA512: 6a456af9f3ccd5437be2edcd61936eb9f9c21ab926a6056c2c11b6b5121d1caca4e1f2ffd09015f9414af152c635a20e1da041eefdef980afbe8a0e8ccce07bd
| -rwxr-xr-x | test/functional/rpc_scantxoutset.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/functional/rpc_scantxoutset.py b/test/functional/rpc_scantxoutset.py index f97636aae7..9f87f18144 100755 --- a/test/functional/rpc_scantxoutset.py +++ b/test/functional/rpc_scantxoutset.py @@ -120,6 +120,9 @@ class ScantxoutsetTest(BitcoinTestFramework): assert_equal(self.nodes[0].scantxoutset("status"), None) assert_equal(self.nodes[0].scantxoutset("abort"), False) + # check that first arg is needed + assert_raises_rpc_error(-1, "scantxoutset \"action\" ( [scanobjects,...] )", self.nodes[0].scantxoutset) + # Check that second arg is needed for start assert_raises_rpc_error(-1, "scanobjects argument is required for the start action", self.nodes[0].scantxoutset, "start") |