[bisq-network/bisq] Refactor apitest cases to use GrpcClient (#5244)

Stan notifications at github.com
Fri Feb 26 02:09:43 CET 2021


This is a follow up to PR https://github.com/bisq-network/bisq/pull/5240, which moved client side `gRPC` boilerplate to a new, reusable GrpcClient class.

>From this change forward, all client side `gRPC` request/response boilerplate (including tests) should now live in `GrpcClient`.

PR https://github.com/bisq-network/bisq/pull/5240 should be reviewed/merged before this one.
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/5244

-- Commit Summary --

  * Make @VisibleForTesting
  * Add CLI testing bot to :apitest
  * Merge branch 'master' into 02-scripted-bot-test
  * Fix manual shutdown exception handling
  * Merge branch 'master' into 02-scripted-bot-test
  * Add --help option to bot-script.json generator
  * Merge branch 'master' into 02-scripted-bot-test
  * Fix typo in toString
  * Adjust api to new minimum fee per vbyte
  * Prepare to adjust api to new minimum fee per vbyte (from feeService)
  * Refactor grpc stub boilerplate from CliMain to GrpcClient
  * Refactor apitest cases to use GrpcClient

-- File Changes --

    M apitest/src/test/java/bisq/apitest/ApiTestCase.java (54)
    M apitest/src/test/java/bisq/apitest/method/GetMethodHelpTest.java (7)
    M apitest/src/test/java/bisq/apitest/method/GetVersionTest.java (5)
    M apitest/src/test/java/bisq/apitest/method/MethodTest.java (420)
    M apitest/src/test/java/bisq/apitest/method/RegisterDisputeAgentsTest.java (19)
    M apitest/src/test/java/bisq/apitest/method/offer/AbstractOfferTest.java (112)
    M apitest/src/test/java/bisq/apitest/method/offer/CancelOfferTest.java (41)
    M apitest/src/test/java/bisq/apitest/method/offer/CreateOfferUsingFixedPriceTest.java (78)
    M apitest/src/test/java/bisq/apitest/method/offer/CreateOfferUsingMarketPriceMarginTest.java (96)
    M apitest/src/test/java/bisq/apitest/method/offer/ValidateCreateOfferTest.java (26)
    M apitest/src/test/java/bisq/apitest/method/payment/AbstractPaymentAccountTest.java (15)
    M apitest/src/test/java/bisq/apitest/method/payment/CreatePaymentAccountTest.java (112)
    M apitest/src/test/java/bisq/apitest/method/payment/GetPaymentMethodsTest.java (2)
    M apitest/src/test/java/bisq/apitest/method/trade/AbstractTradeTest.java (12)
    M apitest/src/test/java/bisq/apitest/method/trade/TakeBuyBTCOfferTest.java (42)
    M apitest/src/test/java/bisq/apitest/method/trade/TakeSellBTCOfferTest.java (44)
    M apitest/src/test/java/bisq/apitest/method/wallet/BsqWalletTest.java (33)
    M apitest/src/test/java/bisq/apitest/method/wallet/BtcTxFeeRateTest.java (28)
    M apitest/src/test/java/bisq/apitest/method/wallet/BtcWalletTest.java (23)
    M apitest/src/test/java/bisq/apitest/method/wallet/WalletProtectionTest.java (51)
    M apitest/src/test/java/bisq/apitest/scenario/PaymentAccountTest.java (4)
    A apitest/src/test/java/bisq/apitest/scenario/ScriptedBotTest.java (121)
    M apitest/src/test/java/bisq/apitest/scenario/WalletTest.java (3)
    A apitest/src/test/java/bisq/apitest/scenario/bot/AbstractBotTest.java (110)
    A apitest/src/test/java/bisq/apitest/scenario/bot/Bot.java (77)
    A apitest/src/test/java/bisq/apitest/scenario/bot/BotClient.java (339)
    A apitest/src/test/java/bisq/apitest/scenario/bot/BotPaymentAccountGenerator.java (68)
    A apitest/src/test/java/bisq/apitest/scenario/bot/InvalidRandomOfferException.java (35)
    A apitest/src/test/java/bisq/apitest/scenario/bot/PaymentAccountNotFoundException.java (35)
    A apitest/src/test/java/bisq/apitest/scenario/bot/RandomOffer.java (177)
    A apitest/src/test/java/bisq/apitest/scenario/bot/RobotBob.java (141)
    A apitest/src/test/java/bisq/apitest/scenario/bot/protocol/BotProtocol.java (349)
    A apitest/src/test/java/bisq/apitest/scenario/bot/protocol/MakerBotProtocol.java (114)
    A apitest/src/test/java/bisq/apitest/scenario/bot/protocol/ProtocolStep.java (17)
    A apitest/src/test/java/bisq/apitest/scenario/bot/protocol/TakerBotProtocol.java (136)
    A apitest/src/test/java/bisq/apitest/scenario/bot/script/BashScriptGenerator.java (235)
    A apitest/src/test/java/bisq/apitest/scenario/bot/script/BotScript.java (78)
    A apitest/src/test/java/bisq/apitest/scenario/bot/script/BotScriptGenerator.java (247)
    A apitest/src/test/java/bisq/apitest/scenario/bot/shutdown/ManualBotShutdownException.java (35)
    A apitest/src/test/java/bisq/apitest/scenario/bot/shutdown/ManualShutdown.java (64)
    M cli/src/main/java/bisq/cli/CliMain.java (361)
    M cli/src/main/java/bisq/cli/CurrencyFormat.java (16)
    A cli/src/main/java/bisq/cli/GrpcClient.java (426)
    M cli/src/main/java/bisq/cli/GrpcStubs.java (2)
    M cli/src/main/java/bisq/cli/TableFormat.java (4)
    M core/src/main/java/bisq/core/api/CoreWalletsService.java (10)
    M core/src/main/java/bisq/core/api/model/TxFeeRateInfo.java (4)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/5244.patch
https://github.com/bisq-network/bisq/pull/5244.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/5244
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210225/7bb300bb/attachment.htm>


More information about the bisq-github mailing list