[ecash-herald] Fix unit test changes from updated chronik-client syntax
ClosedPublic
Actions

Authored by bytesofman on Dec 7 2023, 13:27.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC0c0b70e78f96: [ecash-herald] Fix unit test changes from updated chronik-client syntax

Summary

These were broken when mock-chronik-client was updated. Fix to match syntax of currently used chronik-client version.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

herald-patch-updated-chronik-mock

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25868
Build 51315: Build Diff	ecash-herald-tests
Build 51314: arc lint + arc unit

Event Timeline

bytesofman created this revision.Dec 7 2023, 13:27

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 7 2023, 13:27

bytesofman requested review of this revision.Dec 7 2023, 13:27

Harbormaster completed remote builds in B25868: Diff 43463.Dec 7 2023, 13:29

Fabien accepted this revision.Dec 7 2023, 13:56

This revision is now accepted and ready to land.Dec 7 2023, 13:56

Closed by commit rABC0c0b70e78f96: [ecash-herald] Fix unit test changes from updated chronik-client syntax (authored by bytesofman). · Explain WhyDec 7 2023, 13:58

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC0c0b70e78f96: [ecash-herald] Fix unit test changes from updated chronik-client syntax.

Revision Contents
Changeset List

Path

Size

apps/

ecash-herald/

test/

chronikWsHandler.test.js

4 lines

Diff 43463

View Options

apps/ecash-herald/test/chronikWsHandler.test.js

[ecash-herald] Fix unit test changes from updated chronik-client syntaxClosedPublicActions