Paths

Table of Contentst

Differential D15460

[Cashtab] Combine App integration tests into one file using now-standardized approach
AbandonedPublic
Actions

Authored by bytesofman on Feb 15 2024, 19:25.

Tags

None

Subscribers

None

Details

Reviewers

None

Group Reviewers

Restricted Project

Summary

Incrementally standardize Cashtab unit tests to use actual useWallet hook and mock chronik client instead of various shims that limit selected components.

Start by combining existing App integration tests. Remove legacy tests that are already covered by AppIntegrations tests. For tests that are not covered, load a full mock to execute.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

extend-generalized-tests

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27205
Build 53974: Build Diff	cashtab-tests
Build 53973: arc lint + arc unit

Event Timeline

bytesofman created this revision.Feb 15 2024, 19:25

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 15 2024, 19:25

Harbormaster completed remote builds in B27166: Diff 45278.Feb 15 2024, 19:32

bytesofman published this revision for review.Feb 15 2024, 19:42

implement now-landed test wrapper

Harbormaster completed remote builds in B27204: Diff 45327.Feb 17 2024, 01:00

lint

Harbormaster completed remote builds in B27205: Diff 45328.Feb 17 2024, 01:04

While this diff is not doing anything too complicated, it is difficult to review the way it has been presented and should be split into parts.

superseded by D15492

Revision Contents
Changeset List

Path

Size

cashtab/

package-lock.json

4 lines

2 lines

src/

components/

__tests__/

337 lines

AppIntegrations.test.js

Diff 45328

cashtab/package-lock.json

Loading...

cashtab/package.json

Loading...

cashtab/src/components/tests/App.test.js

Loading...

cashtab/src/components/tests/AppIntegrations.test.js

Loading...