Paths

Table of Contentst

Differential D15340

[Cashtab] Wrap useWallet hook test in act
ClosedPublic
Actions

Authored by bytesofman on Jan 31 2024, 11:38.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC593a822456e2: [Cashtab] Wrap useWallet hook test in act

Summary

Getting away without this now but these tests will fail after upgrading React and testing libraries to latest versions

Use correct syntax and wrappers now in prep for upgrade

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

act-to-usewallet-tests

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26795
Build 53158: Build Diff	cashtab-tests
Build 53157: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jan 31 2024, 11:38

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 31 2024, 11:38

bytesofman published this revision for review.Jan 31 2024, 11:39

version bump

emack accepted this revision.Jan 31 2024, 11:51

This revision is now accepted and ready to land.Jan 31 2024, 11:51

Harbormaster completed remote builds in B26789: Diff 44783.Jan 31 2024, 12:05

rebase

Harbormaster completed remote builds in B26790: Diff 44784.Jan 31 2024, 12:27

Harbormaster completed remote builds in B26795: Diff 44790.Jan 31 2024, 12:53

Closed by commit rABC593a822456e2: [Cashtab] Wrap useWallet hook test in act (authored by bytesofman). · Explain WhyJan 31 2024, 18:16

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC593a822456e2: [Cashtab] Wrap useWallet hook test in act.

Revision Contents
Changeset List

Path

Size

cashtab/

package.json

2 lines

src/

hooks/

__tests__/

useWallet.test.js

28 lines

Diff 44790

View Options

cashtab/package.json

View Options

cashtab/src/hooks/tests/useWallet.test.js

[Cashtab] Wrap useWallet hook test in actClosedPublicActions