Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CLI testing bot to :apitest #5203

Merged
merged 7 commits into from
Feb 26, 2021

Conversation

ghubstan
Copy link
Contributor

@ghubstan ghubstan commented Feb 18, 2021

RobotBob reads a file bot-script.json, instructing it to make and take offers as per an actions json array, e.g. ["make","take","take","make",...], and the tester will manually run CLI commands provided by the bot during each step in a trade.

The test case (ScriptedBotTest) can be run with the test harness, which will start and shutdown all the regtest/dao apps: bitcoind, seednode, arbnode, bob & alice nodes. Or, the test case can be run without the test harness, and the user manages his own daemons.

RobotBob reads a json file instructing it to make and take offers
as per an 'actions' json array, e.g. ["make","take","take","make],
and the tester will manually run CLI commands provided by the bot
during each step in a trade.

The test case (ScriptedBotTest) can be run with the test harness,
which will start and shutdown all the regtest/dao app: bitcoind,
seednode, arbnode, bob & alice nodes.  The test case can also be
run without the test harness, and the user manages his own daemons.

Usage will be described in the PR before it leaves draft stage.
Codacy is right.  Don't use instance of ex, add a catch clause.
Also removed an unnecessary fully qualified name 'String.format'.
@ghubstan ghubstan changed the title [WIP] Add CLI testing bot to :apitest Add CLI testing bot to :apitest Feb 23, 2021
@ghubstan ghubstan marked this pull request as ready for review February 23, 2021 21:32
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@sqrrm sqrrm merged commit 8b70278 into bisq-network:master Feb 26, 2021
@ghubstan ghubstan deleted the 02-scripted-bot-test branch February 27, 2021 11:30
@ripcurlx ripcurlx added this to the v1.6.0 milestone Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants