Page MenuHomePhabricator
Differential D15513

[ecash-coinselect] Support for slpv1 token send txs
AbandonedPublicDraft
Actions

Authored by bytesofman on Feb 21 2024, 22:18.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

Modify ecash-coinselect to support slp v1 send txs

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
upgrade-slp-send
Lint
Lint Errors
SeverityLocationCodeMessage
Errorcashtab/src/slpv1/index.js:117ESLINTno-unused-vars
Errorcashtab/src/slpv1/index.js:117ESLINTno-unused-vars
Unit
No Test Coverage
Build Status
Buildable 27310
Build 54183: Build Diffecash-coinselect · cashtab-tests
Build 54182: arc lint + arc unit

Event Timeline

bytesofman created this revision.Feb 21 2024, 22:18
Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 21 2024, 22:18
Harbormaster completed remote builds in B27310: Diff 45466.Feb 21 2024, 22:18
bytesofman updated this revision to Diff 45467.Feb 21 2024, 23:01

Add some more mocks, tests

Harbormaster completed remote builds in B27311: Diff 45467.Feb 21 2024, 23:01
teamcity added a comment.Feb 21 2024, 23:02
Build Bitcoin ABC Diffs / Diff Testing (ecash-coinselect) failed.

Tail of the build log:

> ecash-coinselect@2.0.3 test
> mocha 'test/**/*.test.js' --reporter mocha-junit-reporter --reporter-options mochaFile=test_results/ecash-coinselect-junit.xml --reporter-options testsuitesTitle=Ecash Coinselect Unit Tests --reporter-options rootSuiteTitle=Ecash Coinselect


Error: Cannot find module 'slp-mdm'
Require stack:
- /work/modules/ecash-coinselect/src/tokenType1.js
- /work/modules/ecash-coinselect/test/tokenType1/index.test.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/src/tokenType1.js:6:138)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/test/tokenType1/index.test.js:8:47)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at cjsLoader (node:internal/modules/esm/translators:356:17)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:305:7)
    at ModuleJob.run (node:internal/modules/esm/module_job:218:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:431:15)
    at async formattedImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
    at async exports.requireOrImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async exports.loadFilesAsync (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async exports.handler (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run.js:370:5)
---------------|---------|----------|---------|---------|-------------------
File           | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
---------------|---------|----------|---------|---------|-------------------
All files      |   14.06 |     1.44 |    6.25 |   14.06 |                   
 byteCount.js  |   38.46 |     6.25 |      20 |   38.46 | 24-43,62-92       
 coinSelect.js |    14.7 |        0 |       0 |    14.7 | 13-84             
 tokenType1.js |    1.66 |        0 |       0 |    1.66 | 7-202             
 utils.js      |      25 |      100 |       0 |      25 | 14-38             
---------------|---------|----------|---------|---------|-------------------

##teamcity[blockOpened name='Code Coverage Summary']
##teamcity[buildStatisticValue key='CodeCoverageAbsBCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsBTotal' value='128']
##teamcity[buildStatisticValue key='CodeCoverageAbsRCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsRTotal' value='69']
##teamcity[buildStatisticValue key='CodeCoverageAbsMCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsMTotal' value='16']
##teamcity[buildStatisticValue key='CodeCoverageAbsLCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsLTotal' value='128']
##teamcity[blockClosed name='Code Coverage Summary']
mv: cannot stat 'test_results/ecash-coinselect-junit.xml': No such file or directory
Build ecash-coinselect failed with exit code 1
bytesofman updated this revision to Diff 45471.Feb 22 2024, 11:45

more refactors, tests

Harbormaster completed remote builds in B27314: Diff 45471.Feb 22 2024, 11:45
teamcity added a comment.Feb 22 2024, 11:46
Build Bitcoin ABC Diffs / Diff Testing (ecash-coinselect) failed.

Tail of the build log:

> ecash-coinselect@2.0.3 test
> mocha 'test/**/*.test.js' --reporter mocha-junit-reporter --reporter-options mochaFile=test_results/ecash-coinselect-junit.xml --reporter-options testsuitesTitle=Ecash Coinselect Unit Tests --reporter-options rootSuiteTitle=Ecash Coinselect


Error: Cannot find module 'slp-mdm'
Require stack:
- /work/modules/ecash-coinselect/src/tokenType1.js
- /work/modules/ecash-coinselect/test/tokenType1/index.test.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/src/tokenType1.js:6:138)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/test/tokenType1/index.test.js:8:47)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at cjsLoader (node:internal/modules/esm/translators:356:17)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:305:7)
    at ModuleJob.run (node:internal/modules/esm/module_job:218:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:431:15)
    at async formattedImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
    at async exports.requireOrImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async exports.loadFilesAsync (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async exports.handler (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run.js:370:5)
---------------|---------|----------|---------|---------|-------------------
File           | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
---------------|---------|----------|---------|---------|-------------------
All files      |   14.06 |     1.44 |    6.25 |   14.06 |                   
 byteCount.js  |   38.46 |     6.25 |      20 |   38.46 | 24-43,62-92       
 coinSelect.js |    14.7 |        0 |       0 |    14.7 | 13-84             
 tokenType1.js |    1.66 |        0 |       0 |    1.66 | 7-202             
 utils.js      |      25 |      100 |       0 |      25 | 14-38             
---------------|---------|----------|---------|---------|-------------------

##teamcity[blockOpened name='Code Coverage Summary']
##teamcity[buildStatisticValue key='CodeCoverageAbsBCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsBTotal' value='128']
##teamcity[buildStatisticValue key='CodeCoverageAbsRCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsRTotal' value='69']
##teamcity[buildStatisticValue key='CodeCoverageAbsMCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsMTotal' value='16']
##teamcity[buildStatisticValue key='CodeCoverageAbsLCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsLTotal' value='128']
##teamcity[blockClosed name='Code Coverage Summary']
mv: cannot stat 'test_results/ecash-coinselect-junit.xml': No such file or directory
Build ecash-coinselect failed with exit code 1
bytesofman updated this revision to Diff 45472.Feb 22 2024, 11:47

lint

Harbormaster completed remote builds in B27315: Diff 45472.Feb 22 2024, 11:49
teamcity added a comment.Feb 22 2024, 11:49
Build Bitcoin ABC Diffs / Diff Testing (ecash-coinselect) failed.

Tail of the build log:

> ecash-coinselect@2.0.3 test
> mocha 'test/**/*.test.js' --reporter mocha-junit-reporter --reporter-options mochaFile=test_results/ecash-coinselect-junit.xml --reporter-options testsuitesTitle=Ecash Coinselect Unit Tests --reporter-options rootSuiteTitle=Ecash Coinselect


Error: Cannot find module 'slp-mdm'
Require stack:
- /work/modules/ecash-coinselect/src/tokenType1.js
- /work/modules/ecash-coinselect/test/tokenType1/index.test.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/src/tokenType1.js:6:138)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/work/modules/ecash-coinselect/test/tokenType1/index.test.js:8:47)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module.replacementCompile (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:60:13)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.<anonymous> (/usr/lib/node_modules/nyc/node_modules/append-transform/index.js:64:4)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Module._load (node:internal/modules/cjs/loader:1023:12)
    at cjsLoader (node:internal/modules/esm/translators:356:17)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:305:7)
    at ModuleJob.run (node:internal/modules/esm/module_job:218:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:329:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:431:15)
    at async formattedImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
    at async exports.requireOrImport (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
    at async exports.loadFilesAsync (/work/modules/ecash-coinselect/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
    at async singleRun (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run-helpers.js:125:3)
    at async exports.handler (/work/modules/ecash-coinselect/node_modules/mocha/lib/cli/run.js:370:5)
---------------|---------|----------|---------|---------|-------------------
File           | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
---------------|---------|----------|---------|---------|-------------------
All files      |   14.06 |     1.44 |    6.25 |   14.06 |                   
 byteCount.js  |   38.46 |     6.25 |      20 |   38.46 | 24-43,62-92       
 coinSelect.js |    14.7 |        0 |       0 |    14.7 | 13-84             
 tokenType1.js |    1.66 |        0 |       0 |    1.66 | 7-202             
 utils.js      |      25 |      100 |       0 |      25 | 14-38             
---------------|---------|----------|---------|---------|-------------------

##teamcity[blockOpened name='Code Coverage Summary']
##teamcity[buildStatisticValue key='CodeCoverageAbsBCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsBTotal' value='128']
##teamcity[buildStatisticValue key='CodeCoverageAbsRCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsRTotal' value='69']
##teamcity[buildStatisticValue key='CodeCoverageAbsMCovered' value='1']
##teamcity[buildStatisticValue key='CodeCoverageAbsMTotal' value='16']
##teamcity[buildStatisticValue key='CodeCoverageAbsLCovered' value='18']
##teamcity[buildStatisticValue key='CodeCoverageAbsLTotal' value='128']
##teamcity[blockClosed name='Code Coverage Summary']
mv: cannot stat 'test_results/ecash-coinselect-junit.xml': No such file or directory
Build ecash-coinselect failed with exit code 1
bytesofman updated this revision to Diff 45481.Feb 22 2024, 14:12

Update coinselect to accept tokenInputs as param

Harbormaster completed remote builds in B27322: Diff 45481.Feb 22 2024, 14:15
bytesofman updated this revision to Diff 45493.Feb 22 2024, 19:33

lib and cashtab can send etokens better

Harbormaster completed remote builds in B27330: Diff 45493.Feb 22 2024, 19:33
teamcity added a comment.Feb 22 2024, 19:36
Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.

Tail of the build log:

/work/cashtab /work/abc-ci-builds/cashtab-tests
npm WARN deprecated rollup-plugin-terser@7.0.2: This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated @babel/plugin-proposal-numeric-separator@7.18.6: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-numeric-separator instead.
npm WARN deprecated @babel/plugin-proposal-nullish-coalescing-operator@7.18.6: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-nullish-coalescing-operator instead.
npm WARN deprecated @babel/plugin-proposal-private-methods@7.18.6: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-private-methods instead.
npm WARN deprecated @babel/plugin-proposal-class-properties@7.18.6: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-class-properties instead.
npm WARN deprecated sourcemap-codec@1.4.8: Please use @jridgewell/sourcemap-codec instead
npm WARN deprecated stable@0.1.8: Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
npm WARN deprecated @babel/plugin-proposal-private-property-in-object@7.21.11: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-private-property-in-object instead.
npm WARN deprecated abab@2.0.6: Use your platform's native atob() and btoa() methods instead
npm WARN deprecated domexception@4.0.0: Use your platform's native DOMException instead
npm WARN deprecated @babel/plugin-proposal-optional-chaining@7.21.0: This proposal has been merged to the ECMAScript standard and thus this plugin is no longer maintained. Please use @babel/plugin-transform-optional-chaining instead.
npm WARN deprecated workbox-cacheable-response@6.6.0: workbox-background-sync@6.6.0

added 1912 packages, and audited 1913 packages in 24s

263 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

> cashtab@1.1.33 build
> node scripts/build.js

Creating an optimized production build...
Failed to compile.

[eslint] 
src/components/Send/SendToken.js
  Line 33:10:  'sendToken' is defined but never used  no-unused-vars

Search for the keywords to learn more about each error.


Build cashtab-tests failed with exit code 1
bytesofman mentioned this in D15518: [ecash-coinselect] Add support for utxo format of chronik-client in-node.Feb 22 2024, 19:51
bytesofman updated this revision to Diff 45496.Feb 22 2024, 19:58

we do not need changeaddress specified

Harbormaster completed remote builds in B27333: Diff 45496.Feb 22 2024, 19:58
bytesofman updated this revision to Diff 45497.Feb 22 2024, 23:46

more tests and implement

Harbormaster completed remote builds in B27334: Diff 45497.Feb 22 2024, 23:49
bytesofman abandoned this revision.Feb 23 2024, 00:20

this is messed up in git and I was unable to resolve with squash. superseded by D15519 and D15520

bytesofman mentioned this in rABC022c18b3e28f: [ecash-coinselect] Add support for utxo format of chronik-client in-node.Feb 23 2024, 11:22

Revision Contents
Changeset List

PathSize
cashtab/
src/
slpv1/
4 lines
transactions/
1 line
modules/
ecash-coinselect/
4 lines
2 lines
src/
168 lines
test/
tokenType1/
30 lines
709 lines
80 lines

Diff 45466

View Options

cashtab/src/slpv1/index.js

Loading...
View Options

cashtab/src/transactions/index.js

Loading...
View Options

modules/ecash-coinselect/package-lock.json

Loading...
View Options

modules/ecash-coinselect/package.json

Loading...
View Options

modules/ecash-coinselect/src/tokenType1.js

Loading...
View Options

modules/ecash-coinselect/test/tokenType1/index.test.js

Loading...
View Options

modules/ecash-coinselect/test/tokenType1/mocks.js

Loading...
View Options

modules/ecash-coinselect/test/tokenType1/vectors.js

Loading...