As a precursor to combining the app integration tests into one file, update the original tests to use new standardized wrappers
Details
Details
- Reviewers
emack - Group Reviewers
Restricted Project - Commits
- rABC0f8f0d53943e: [Cashtab] Apply standardized test wrappers to App.test.js
npm test
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- update-app-tests
- Lint
Lint Errors Severity Location Code Message Error cashtab/src/components/__tests__/App.test.js:5 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:6 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:7 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:9 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:10 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:13 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:14 ESLINT no-unused-vars Error cashtab/src/components/__tests__/App.test.js:14 ESLINT no-unused-vars - Unit
No Test Coverage - Build Status
Buildable 27210 Build 53984: Build Diff cashtab-tests Build 53983: arc lint + arc unit
Event Timeline
| cashtab/src/components/__tests__/App.test.js | ||
|---|---|---|
| 80 ↗ | (On Diff #45335) | Does the absence of the route param result in this wrapper defaulting to the App (i.e. home) route? |
| cashtab/src/components/__tests__/App.test.js | ||
|---|---|---|
| 80 ↗ | (On Diff #45335) | /wallet is specified as a default param in CashtabTestWrapper.js |