Differential D16211

[chronik-client] More robust integration test for websocket.ts
ClosedPublic
Actions

Authored by bytesofman on May 23 2024, 13:24.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC77aa0655d3e2: [chronik-client] More robust integration test for websocket.ts

Summary

Seeing occasional flakiness failure of this test related to msgCollector being populated out of sync with test inspection.

Implement same solution used to address this issue on the token websocket tests (e.g. token_alp.ts).

Test Plan

CI test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

cc-more-robust-ws-tests

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 29107
Build 57750: Build Diff	ecash-agora-integration-tests · chronik-client-tests · ecash-lib-integration-tests · chronik-client-integration-tests
Build 57749: arc lint + arc unit

Event Timeline

bytesofman created this revision.May 23 2024, 13:24

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 23 2024, 13:24

Harbormaster completed remote builds in B29107: Diff 47925.May 23 2024, 13:33

abstract as function and apply to all ws tests

bytesofman published this revision for review.May 23 2024, 13:55

Harbormaster completed remote builds in B29108: Diff 47926.May 23 2024, 14:02

PiRK accepted this revision.May 23 2024, 14:03

This revision is now accepted and ready to land.May 23 2024, 14:03

Closed by commit rABC77aa0655d3e2: [chronik-client] More robust integration test for websocket.ts (authored by bytesofman). · Explain WhyMay 23 2024, 17:29

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC77aa0655d3e2: [chronik-client] More robust integration test for websocket.ts.

Revision Contents
Changeset List

Path

Size

modules/

chronik-client/

test/

integration/

websocket.ts

60 lines

Diff 47925

View Options

modules/chronik-client/test/integration/websocket.ts

[chronik-client] More robust integration test for websocket.tsClosedPublicActions