Skip to content

Commit 0a232cd

Browse files
committed
tests: sign test needs to trigger ecurve.Point side effects
1 parent 32cbd02 commit 0a232cd

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

test/wallet.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -651,10 +651,16 @@ describe('Wallet', function() {
651651
var fee = 30000
652652
sinon.spy(TransactionBuilder.prototype, "sign")
653653

654-
var tx = wallet.createTx(to, value, { fixedFee: fee })
654+
wallet.createTx(to, value, { fixedFee: fee })
655+
656+
var priv1 = wallet.getPrivateKeyForAddress(address1)
657+
var priv2 = wallet.getPrivateKeyForAddress(address2)
658+
659+
// FIXME: boo (required) side effects
660+
priv1.pub.Q.affineX, priv2.pub.Q.affineX
655661

656-
assert(TransactionBuilder.prototype.sign.calledWith(0, wallet.getPrivateKeyForAddress(address2)))
657-
assert(TransactionBuilder.prototype.sign.calledWith(1, wallet.getPrivateKeyForAddress(address1)))
662+
assert(TransactionBuilder.prototype.sign.calledWith(0, priv2))
663+
assert(TransactionBuilder.prototype.sign.calledWith(1, priv1))
658664
})
659665
})
660666

0 commit comments

Comments
 (0)