[Cashtab] [Simplify wallet structure p5] Remove function specific variable name change
ClosedPublic
Actions

Authored by bytesofman on Nov 28 2022, 04:56.

Details

Reviewers

kieran709

Group Reviewers

Restricted Project

Commits

rABC7eb7f5f1b470: [Cashtab] [Simplify wallet structure p5] Remove function specific variable name…

Summary

T2663

Depends on D12625

There is a case to be made for renaming slpUtxos and nonSlpUtxos to tokenUtxos and xecUtxos everywhere, but in this case I think it's better to stick to the original syntax. At any rate, we should be consistent about the name everywhere in the app.

Test Plan

Confirm diff is changing variable names
Confirm I didn't mix them up and that tokenUtxos became slpUtxos, and xecUtxos became nonSlpUtxos
npm test
npm start, send a token tx, burn a token

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Nov 28 2022, 04:56

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 28 2022, 04:56

bytesofman requested review of this revision.Nov 28 2022, 04:56

Harbormaster completed remote builds in B21200: Diff 36726.Nov 28 2022, 05:09

bytesofman added a child revision: D12660: [Cashtab] [Simplify wallet structure p6] Ensure last final variable calcs match final variable names.Nov 28 2022, 05:10

kieran709 accepted this revision.Nov 28 2022, 23:28

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

rebase

Harbormaster completed remote builds in B21256: Diff 36806.Nov 30 2022, 08:42

Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.

Failed tests logs:

====== CashTab Unit Tests: useBCH hook Throws error attempting to burn an eToken ID that is not within the wallet's utxo ======
Error: expect(received).toStrictEqual(expected) // deep equality

Expected: [Error: No token UTXOs for the specified token could be found.]
Received: [TypeError: Cannot destructure property 'tokenId' of 'undefined' as it is undefined.]
    at Object.<anonymous> (/work/web/cashtab/src/hooks/__tests__/useBCH.test.js:211:29)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)

Each failure log is accessible here:
CashTab Unit Tests: useBCH hook Throws error attempting to burn an eToken ID that is not within the wallet's utxo