T3444
Implement new standardized test wrapper in SendToken tests. Stop using data-testid in SendToken tests and delete from SendToken.
Update placeholder text for Burn to improve both the test and the UI.
Details
Details
- Reviewers
PiRK - Group Reviewers
Restricted Project - Commits
- rABC9693acd7dc34: [Cashtab] Update SendToken tests to new standardized wrapper
npm test
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| cashtab/src/components/Send/__tests__/SendToken.test.js | ||
|---|---|---|
| 110–127 ↗ | (On Diff #45669) | This data is duplicated in most tests in this diff, with no differences AFAICT. Can't it be assigned to a variable in the outer scope? |
| cashtab/src/components/Send/__tests__/SendToken.test.js | ||
|---|---|---|
| 110–127 ↗ | (On Diff #45669) | Good call. I was able to move the mock creation steps there. However, render is not allowed to go in a beforeEach step |