Skip to content

Commit

Permalink
Merge #1028: test: fix surjectionproof overflow functional test for n…
Browse files Browse the repository at this point in the history
…ew RPC changes

c0936ba test: fix surjectionproof overflow functional test for new RPC changes (Andrew Poelstra)

Pull request description:

  This fixes an undetected merge conflict between #1014 and #900.

ACKs for top commit:
  gwillen:
    utACK c0936ba
  psgreco:
    UTACK c0936ba

Tree-SHA512: a39d98db91d92f7e0e739e06d7a1bb13814d2bc7751cb6105a759af04ed0d61d206d1650f5e4e087feaeafcf1c482587b05b66d9cc497bc594a45df6bf23c07a
  • Loading branch information
apoelstra committed Aug 31, 2021
2 parents d973e11 + c0936ba commit 338ccc4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/functional/rpc_fundrawtransaction.py
Original file line number Diff line number Diff line change
Expand Up @@ -985,11 +985,11 @@ def test_surjectionproof_many_inputs(self):

# ...and try to send them all in one transaction
# This should fail but we should not see an assertation failure.
rawtx = recipient.createrawtransaction([], {wallet.getnewaddress(): 49.99})
rawtx = recipient.createrawtransaction([], [{wallet.getnewaddress(): 49.99}])
assert_raises_rpc_error(-4, "Unable to blind the transaction properly. This should not happen.", recipient.fundrawtransaction, rawtx)

# Try to send them across two transactions. This should succeed.
rawtx = recipient.createrawtransaction([], {wallet.getnewaddress(): 24.99})
rawtx = recipient.createrawtransaction([], [{wallet.getnewaddress(): 24.99}])
for i in range(0, 2):
fundedtx = recipient.fundrawtransaction(rawtx)
blindedtx = recipient.blindrawtransaction(fundedtx['hex'])
Expand Down

0 comments on commit 338ccc4

Please sign in to comment.