Skip to content

Commit 90c8f79

Browse files
committed
test: remove redundant test values
as they are parsed identically. See AmountFromValue() / ParseFixedPoint() / UniValue#getValStr()
1 parent c3f2033 commit 90c8f79

File tree

5 files changed

+6
-6
lines changed

5 files changed

+6
-6
lines changed

test/functional/rpc_psbt.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,7 @@ def run_test(self):
327327
assert_raises_rpc_error(-3, "Invalid amount",
328328
self.nodes[1].walletcreatefundedpsbt, inputs, outputs, 0, {param: invalid_value, "add_inputs": True})
329329
# Test fee_rate values that cannot be represented in sat/vB.
330-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
330+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
331331
assert_raises_rpc_error(-3, "Invalid amount",
332332
self.nodes[1].walletcreatefundedpsbt, inputs, outputs, 0, {"fee_rate": invalid_value, "add_inputs": True})
333333

test/functional/wallet_basic.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ def run_test(self):
328328
for invalid_value in ["", 0.000000001, 1e-09, 1.111111111, 1111111111111111, "31.999999999999999999999"]:
329329
assert_raises_rpc_error(-3, msg, self.nodes[2].sendmany, amounts={address: 1.0}, fee_rate=invalid_value)
330330
# Test fee_rate values that cannot be represented in sat/vB.
331-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
331+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
332332
assert_raises_rpc_error(-3, msg, self.nodes[2].sendmany, amounts={address: 10}, fee_rate=invalid_value)
333333
# Test fee_rate out of range (negative number).
334334
assert_raises_rpc_error(-3, OUT_OF_RANGE, self.nodes[2].sendmany, amounts={address: 10}, fee_rate=-1)
@@ -523,7 +523,7 @@ def run_test(self):
523523
for invalid_value in ["", 0.000000001, 1e-09, 1.111111111, 1111111111111111, "31.999999999999999999999"]:
524524
assert_raises_rpc_error(-3, msg, self.nodes[2].sendtoaddress, address=address, amount=1.0, fee_rate=invalid_value)
525525
# Test fee_rate values that cannot be represented in sat/vB.
526-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
526+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
527527
assert_raises_rpc_error(-3, msg, self.nodes[2].sendtoaddress, address=address, amount=10, fee_rate=invalid_value)
528528
# Test fee_rate out of range (negative number).
529529
assert_raises_rpc_error(-3, OUT_OF_RANGE, self.nodes[2].sendtoaddress, address=address, amount=1.0, fee_rate=-1)

test/functional/wallet_bumpfee.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ def test_invalid_parameters(self, rbf_node, peer_node, dest_address):
141141
for invalid_value in ["", 0.000000001, 1e-09, 1.111111111, 1111111111111111, "31.999999999999999999999"]:
142142
assert_raises_rpc_error(-3, msg, rbf_node.bumpfee, rbfid, fee_rate=invalid_value)
143143
# Test fee_rate values that cannot be represented in sat/vB.
144-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
144+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
145145
assert_raises_rpc_error(-3, msg, rbf_node.bumpfee, rbfid, fee_rate=invalid_value)
146146
# Test fee_rate out of range (negative number).
147147
assert_raises_rpc_error(-3, "Amount out of range", rbf_node.bumpfee, rbfid, fee_rate=-1)

test/functional/wallet_fundrawtransaction.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -846,7 +846,7 @@ def test_option_feerate(self):
846846
for invalid_value in ["", 0.000000001, 1e-09, 1.111111111, 1111111111111111, "31.999999999999999999999"]:
847847
assert_raises_rpc_error(-3, "Invalid amount", node.fundrawtransaction, rawtx, add_inputs=True, **{param: invalid_value})
848848
# Test fee_rate values that cannot be represented in sat/vB.
849-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
849+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
850850
assert_raises_rpc_error(-3, "Invalid amount",
851851
node.fundrawtransaction, rawtx, fee_rate=invalid_value, add_inputs=True)
852852

test/functional/wallet_send.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -387,7 +387,7 @@ def run_test(self):
387387
self.test_send(from_wallet=w0, to_wallet=w1, amount=1, fee_rate=invalid_value, expect_error=(-3, msg))
388388
self.test_send(from_wallet=w0, to_wallet=w1, amount=1, arg_fee_rate=invalid_value, expect_error=(-3, msg))
389389
# Test fee_rate values that cannot be represented in sat/vB.
390-
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999, "0.0001", "0.00000001", "0.00099999", "31.99999999"]:
390+
for invalid_value in [0.0001, 0.00000001, 0.00099999, 31.99999999]:
391391
self.test_send(from_wallet=w0, to_wallet=w1, amount=1, fee_rate=invalid_value, expect_error=(-3, msg))
392392
self.test_send(from_wallet=w0, to_wallet=w1, amount=1, arg_fee_rate=invalid_value, expect_error=(-3, msg))
393393
# Test fee_rate out of range (negative number).

0 commit comments

Comments
 (0)