[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