[bisq-network/bisq] make sure that signed witness is immutable. (#3108)
Manfred Karrer
notifications at github.com
Tue Aug 20 10:39:47 UTC 2019
ManfredKarrer approved this pull request.
utACK
see Nit comment...
>
+ @Test
+ public void isImmutable() {
+ byte[] signerPubkey = arbitrator1Key.getPubKey();
+ SignedWitness signedWitness = new SignedWitness(true, witnessHash, witnessHashSignature, signerPubkey, witnessOwner1PubKey, Instant.now().getEpochSecond(), 100);
+ byte[] witnessHash = signedWitness.getWitnessHash().clone();
Maybe better to use a different name? Now the this.witnessHash and witnessHash are different, which makes the test a bit harder to read IMO.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/3108#pullrequestreview-277076718
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190820/b1e51e2e/attachment.html>
More information about the bisq-github
mailing list