Skip to content

Commit 9e399b9

Browse files
committed
test: check parameter validity in rpc_signmessage.py
checks parameter validity and error codes for the related RPCs: - signmessagewithprivkey - signmessage - verifymessage
1 parent e62f0c7 commit 9e399b9

File tree

1 file changed

+21
-1
lines changed

1 file changed

+21
-1
lines changed

test/functional/rpc_signmessage.py

+21-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,10 @@
55
"""Test RPC commands for signing and verifying messages."""
66

77
from test_framework.test_framework import BitcoinTestFramework
8-
from test_framework.util import assert_equal
8+
from test_framework.util import (
9+
assert_equal,
10+
assert_raises_rpc_error,
11+
)
912

1013
class SignMessagesTest(BitcoinTestFramework):
1114
def set_test_params(self):
@@ -38,5 +41,22 @@ def run_test(self):
3841
assert not self.nodes[0].verifymessage(other_address, signature, message)
3942
assert not self.nodes[0].verifymessage(address, other_signature, message)
4043

44+
self.log.info('test parameter validity and error codes')
45+
# signmessage(withprivkey) have two required parameters
46+
for num_params in [0, 1, 3, 4, 5]:
47+
param_list = ["dummy"]*num_params
48+
assert_raises_rpc_error(-1, "signmessagewithprivkey", self.nodes[0].signmessagewithprivkey, *param_list)
49+
assert_raises_rpc_error(-1, "signmessage", self.nodes[0].signmessage, *param_list)
50+
# verifymessage has three required parameters
51+
for num_params in [0, 1, 2, 4, 5]:
52+
param_list = ["dummy"]*num_params
53+
assert_raises_rpc_error(-1, "verifymessage", self.nodes[0].verifymessage, *param_list)
54+
# invalid key or address provided
55+
assert_raises_rpc_error(-5, "Invalid private key", self.nodes[0].signmessagewithprivkey, "invalid_key", message)
56+
assert_raises_rpc_error(-5, "Invalid address", self.nodes[0].signmessage, "invalid_addr", message)
57+
assert_raises_rpc_error(-5, "Invalid address", self.nodes[0].verifymessage, "invalid_addr", signature, message)
58+
# malformed signature provided
59+
assert_raises_rpc_error(-3, "Malformed base64 encoding", self.nodes[0].verifymessage, self.nodes[0].getnewaddress(), "invalid_sig", message)
60+
4161
if __name__ == '__main__':
4262
SignMessagesTest().main()

0 commit comments

Comments
 (0)