Paths

Table of Contentst

Differential D12672

[Cashtab] Remove BCH param artifact from Configure snapshot tests
ClosedPublic
Actions

Authored by bytesofman on Nov 28 2022, 08:33.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCc0d62048a9b8: [Cashtab] Remove BCH param artifact from Configure snapshot tests

Summary

T2730

The BCH param is not in Configure.js but it still exists as an artifact in Configure.test.js. Remove.

Test Plan

npm test
Confirm BCH does not appear in Configure.js

Diff Detail

Repository

rABC Bitcoin ABC

Branch

patch-configure-bch-param

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21214
Build 42080: Build Diff	cashtab-tests
Build 42079: arc lint + arc unit

Event Timeline

bytesofman created this revision.Nov 28 2022, 08:33

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 28 2022, 08:33

bytesofman requested review of this revision.Nov 28 2022, 08:33

Harbormaster completed remote builds in B21214: Diff 36740.Nov 28 2022, 08:35

emack accepted this revision.Nov 29 2022, 23:48

This revision is now accepted and ready to land.Nov 29 2022, 23:48

Closed by commit rABCc0d62048a9b8: [Cashtab] Remove BCH param artifact from Configure snapshot tests (authored by bytesofman). · Explain WhyNov 30 2022, 08:08

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCc0d62048a9b8: [Cashtab] Remove BCH param artifact from Configure snapshot tests.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Configure/

__tests__/

Configure.test.js

11 lines

Diff 36740

View Options

web/cashtab/src/components/Configure/tests/Configure.test.js

[Cashtab] Remove BCH param artifact from Configure snapshot testsClosedPublicActions