Page MenuHomePhabricator

[Cashtab] [BCH deprecation] [TxBuilder] Pt 5/8 - Add local TxBuilder to sendToken
ClosedPublic

Authored by emack on Nov 23 2022, 11:49.

Details

Summary

T2730

Depends on D12575

This diff updates the sendToken() function to use the local transaction builder module and compares the txBuilder object after adding input, output and signing/building tx.

Pt 1/8 - Localize Transaction Builder module
Pt 2/8 - Deprecate BCH.TransactionBuilder in sendXec
Pt 3/8 - Add local TxBuilder to createToken
Pt 4/8 - Deprecate BCH.TransactionBuilder in createToken
Pt 5/8 - Add local TxBuilder to sendToken
Pt 6/8 - Deprecate BCH.TransactionBuilder in sendToken
Pt 7/8 - Add local TxBuilder to burnToken
Pt 8/8 - Deprecate BCH.TransactionBuilder in burnToken

Test Plan

npm ci
npm test and ensure no errors with txBuilder.test.js
npm start
send an existing token and ensure the following 3 console log messages are displayed:
post-generateTokenTxInput() output matches
post-generateTokenTxOutput() output matches
post-signAndBuildTx() output matches
send a newly created token and ensure the same 3 console log messages are displayed

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable