[Cashtab] Updating processChronikWsMsg unit test
ClosedPublic
Actions

Authored by emack on Jul 2 2023, 05:39.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC52c9b5557640: [Cashtab] Updating processChronikWsMsg unit test

Summary

The unit test for processChronikWsMsg is periodically breaking the build because even when the AddedToMempool event is picked up by the websocket, the fiatPrice input arg may not always be processed.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emack created this revision.Jul 2 2023, 05:39

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 2 2023, 05:39

emack requested review of this revision.Jul 2 2023, 05:39

Harbormaster completed remote builds in B24308: Diff 41182.Jul 2 2023, 05:42

This revision is now accepted and ready to land.Jul 2 2023, 13:02

Closed by commit rABC52c9b5557640: [Cashtab] Updating processChronikWsMsg unit test (authored by emack). · Explain WhyJul 2 2023, 13:16

This revision was automatically updated to reflect the committed changes.

emack added a commit: rABC52c9b5557640: [Cashtab] Updating processChronikWsMsg unit test.

emack mentioned this in D14190: [Cashtab] Improve isValidStoredWallet validation.Jul 2 2023, 13:22

Revision Contents
Changeset List

Path

Size

cashtab/

src/

hooks/

__tests__/

useWallet.test.js

8 lines

Diff 41187

View Options

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

[Cashtab] Updating processChronikWsMsg unit testClosedPublicActions