Pass chronik as a prop to useWallet.js hook. This enables full-app integration testing by passing mock chronik client.
Add a few simple unit tests to confirm this.
Details
Details
- Reviewers
PiRK - Group Reviewers
Restricted Project - Commits
- rABCfc2531f609eb: [Cashtab] Pass chronik as a prop to useWallet hook
npm test
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- pass-chronik-as-prop
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 26919 Build 53405: Build Diff cashtab-tests Build 53404: arc lint + arc unit
Event Timeline
| cashtab/src/components/__tests__/AppIntegrations.test.js | ||
|---|---|---|
| 1 | It is a bit silly to have an AppIntegrations.test.js and an App.test.js Will consolidate these to one. For now, it is useful to have two files to show that this diff does not break old tests and keep the "new format" tests in one place. This diff will enable much better integration tests across all of Cashtab. Will see some reorganization and refactoring of existing tests to use this model going forward, which tests the app as it actually works in prod. | |