HomePhabricator

[Cashtab] Remove faulty unit tests

Description

[Cashtab] Remove faulty unit tests

Summary:
While it would be nice to have automated integration tests like these -- these tests are not actually doing what they say they are doing. That they pass is misleading.

To properly automate integration tests in hooks, we need

  • localforage mock
  • different libraries with access to full component state

Remaining unit tests in useWallet.js are testing functions from the hook.

Test Plan: npm test

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Subscribers: emack

Differential Revision: https://reviews.bitcoinabc.org/D15104

Details

Provenance
bytesofmanAuthored on Jan 5 2024, 21:30
bytesofmanPushed on Jan 8 2024, 16:58
Reviewer
Restricted Project
Differential Revision
D15104: [Cashtab] Remove faulty unit tests
Parents
rABC82bbd1c2dcf0: [addrman] add tests for IsTerrible
Branches
Unknown
Tags
Unknown