Page MenuHomePhabricator

[Chronik] ChronikWs: Raise errors in recv caught in on_error
AbandonedPublic

Authored by tobias_ruck on Feb 17 2024, 10:19.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Summary

We collect any errors coming from WebSocket via on_error in an errors list.

But it's up to the test to ensure these errors are raised/handled, but this is very cumbersome and error-prone.

With this diff, recv will raise any received errors so they can be handled properly.

Test Plan

./test/functional/test_runner.py chronik_ws_script

Diff Detail

Repository
rABC Bitcoin ABC
Branch
chronik-python-ws-client-raise-errors
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27208
Build 53980: Build Diffbuild-without-wallet · build-diff · build-clang · build-debug · build-clang-tidy
Build 53979: arc lint + arc unit