[bisq-network/bisq] Replace mock arg with SEPA PaymentMethod instance (#4161)
Stan
notifications at github.com
Sat Apr 11 17:05:15 UTC 2020
Test AccountAgeWitnessServiceTest > testArbitratorSignWitness failed
in full gradle build, but passed when run as a single test:
./gradlew :core:cleanTest :core:test \
--tests "bisq.core.account.witness.AccountAgeWitnessServiceTest"
This test also passed when run in the IDE.
Solved by not passing a mocked PaymentMethod.SEPA argument into the
test's service.getTraderPaymentAccounts() method, where mock paymentId
field was null when running full build's test suites.
Fixes #4158
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/4161
-- Commit Summary --
* Replace mock arg with SEPA PaymentMethod instance
-- File Changes --
M core/src/test/java/bisq/core/account/witness/AccountAgeWitnessServiceTest.java (5)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/4161.patch
https://github.com/bisq-network/bisq/pull/4161.diff
--
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/4161
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200411/875a5bd8/attachment.html>
More information about the bisq-github
mailing list