[bisq-network/bisq] Tidy up apitest (#5024)
sqrrm
notifications at github.com
Fri Jan 1 16:57:45 CET 2021
@sqrrm approved this pull request.
utACK
Branch name could be more descriptive
> @@ -134,9 +134,9 @@ public void testAliceSendBTCToBob(TestInfo testInfo) {
log.debug("{} Bob's BTC Balances:\n{}",
testName(testInfo),
formatBtcBalanceInfoTbl(bobsBalances));
- // We cannot (?) predict the exact tx size and calculate how much in tx fees were
- // deducted from the 5.5 BTC sent to Bob, but we do know Bob should have something
- // between 15.49978000 and 15.49978100 BTC.
+ // The sendbtc tx weight and size randomly varies between two distinct values
I assume this is a 1 byte difference in address length that is allowed.
--
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/5024#pullrequestreview-560592939
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210101/34c479ea/attachment.htm>
More information about the bisq-github
mailing list