[Cashtab] Add missing input param to unit test
ClosedPublic
Actions

Authored by bytesofman on Jan 16 2024, 17:22.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCaffc832634a5: [Cashtab] Add missing input param to unit test

Summary

Missing a function param in a unit test. Test is passing bc it is confirming an error is thrown, and the desired error is thrown before the feeRate param is used in the function.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

unit-test-patch

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26402
Build 52374: Build Diff	cashtab-tests
Build 52373: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jan 16 2024, 17:22

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 16 2024, 17:22

bytesofman published this revision for review.Jan 16 2024, 17:22

Harbormaster completed remote builds in B26402: Diff 44251.Jan 16 2024, 17:26

Fabien accepted this revision.Jan 16 2024, 17:37

This revision is now accepted and ready to land.Jan 16 2024, 17:37

Closed by commit rABCaffc832634a5: [Cashtab] Add missing input param to unit test (authored by bytesofman). · Explain WhyJan 16 2024, 17:59

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCaffc832634a5: [Cashtab] Add missing input param to unit test.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

transactions/

fixtures/

vectors.js

1 line

Diff 44251

View Options

cashtab/src/transactions/fixtures/vectors.js

[Cashtab] Add missing input param to unit testClosedPublicActions