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 wallets protection api tests #4296

Merged
merged 2 commits into from
Jun 25, 2020

Conversation

ghubstan
Copy link
Contributor

(This PR should be merged after PR 4295. Its branch is based on PR 4295's branch.)

This commit includes the following changes:

  • New tests for methods lockwallet, unlockwallet,
    removewalletpassword, and setwalletpassword.

  • New getbalance method error handing tests to verify
    error message correctness when wallet is locked.

  • Update to getversion method test -- now expects 1.3.4.

  • Check for new [params] column header in help text.

This PR replaces 4243, which had not been kept in sync with commits to tested methods in 4214. It also replaces 4268, which was based on a stale main branch.

Running the test suite will verify all tests pass

(1) Clone and build branch wallets-protection-api-tests

  $ git clone -b wallets-protection-api-tests https://github.com/ghubstan/bisq.git wallets-protection-api-tests

  $ cd wallets-protection-api-tests

  $ ./gradlew clean build -x test

(2) Run the server on Linux or OSX:

  $ ./bisq-daemon --apiPassword=xyz --appDataDir=/tmp/newbisqdatadir

(3) Let the server initialize (sync) for at least 2 minutes.

(4) Run the bats test suite:

  $ bats cli/test.sh

All tests should pass.

ghubstan added 2 commits June 12, 2020 15:24
This change fixes the ambiguity in the original class name, which
implied it was a btc wallet service, not a bsq and btc wallets service.
This commit includes the following changes:

 * New tests for methods `lockwallet`, `unlockwallet`,
   `removewalletpassword`, and `setwalletpassword`.

 * New `getbalance` method error handing tests to verify
   error message correctness when wallet is locked.

 * Update to `getversion` method test -- now expects `1.3.4`.

 * Check for new `[params]` column header in help text.
Copy link
Contributor

@dmos62 dmos62 left a comment

Choose a reason for hiding this comment

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

ACK. Got a tingly feeling running Bisq daemon for the first time 🤘

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.

ACK

@sqrrm sqrrm merged commit cfd126f into bisq-network:master Jun 25, 2020
@ghubstan ghubstan deleted the wallets-protection-api-tests branch June 26, 2020 20:42
@ripcurlx ripcurlx added this to the v1.3.6 milestone Jul 6, 2020
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.

4 participants