Get list-wallets to 100% test coverage
Closed this issue · 2 comments
Bounty: 100 tokens
The scope of this Issue is to add tests to the list wallets tests in order to achieve 100% test coverage. It is currently at 70% test coverage. Make any changes needed to the list wallets command necessary to facilitate mocks and test coverage.
-
The burn-tokens tests have achieved 100% test coverage. This provides a good example of the patterns that will be needed to complete this Issue.
-
This YouTube video covers JavaScript design and testing patterns that should be followed.
-
This YouTube video shows how to generate test coverage reports. These reports will help you identify the lines that are not being covered by current tests.
Acceptance Criteria
- A Pull Request (PR) should be submitted that meet the above criteria. The PR should be from a branch and not a fork. In order to create a branch, you'll need commit to access to this repository.
- All tests should pass when running
npm test
. - The command targeted by this Issue should show 100% test coverage when tests are run.
- All commands should still be fully functional.
- The bounty will be awarded upon successfully merging of the PR into the
master
branch.
I've pushed a major change to the master
branch. Be sure to pull the latest and update any branch prior to submitting a PR.
This issue resolved by this PR:
Permissionless-Software-Foundation/slp-cli-wallet#1