Page MenuHomePhabricator
Differential D16889

[chronik] Add a test for invalidated blocks on a parked chain
ClosedPublic
Actions

Authored by Fabien on Thu, Oct 10, 20:54.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABC0fb8fe8b3e1e: [chronik] Add a test for invalidated blocks on a parked chain
Summary

When a block connects on a parked chain, it gets invalidated as expected but chronik should not send a websocket message in this case: it's basically another chain at this point.

Test Plan
./test/functional/test_runner.py chronik_ws

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien created this revision.Thu, Oct 10, 20:54
Herald added a reviewer: Restricted Project. · View Herald TranscriptThu, Oct 10, 20:54
Fabien requested review of this revision.Thu, Oct 10, 20:54
Harbormaster completed remote builds in B30556: Diff 50037.Thu, Oct 10, 21:25
PiRK accepted this revision.Fri, Oct 11, 06:00
PiRK added a subscriber: PiRK.
PiRK added inline comments.
test/functional/chronik_ws.py
222 ↗(On Diff #50037)

?

This revision is now accepted and ready to land.Fri, Oct 11, 06:00
Fabien added inline comments.Fri, Oct 11, 07:49
test/functional/chronik_ws.py
222 ↗(On Diff #50037)

woups, yes it should be invalidate

Closed by commit rABC0fb8fe8b3e1e: [chronik] Add a test for invalidated blocks on a parked chain (authored by Fabien). · Explain WhyFri, Oct 11, 07:51
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABC0fb8fe8b3e1e: [chronik] Add a test for invalidated blocks on a parked chain.

Revision Contents
Changeset List

PathSize
chronik/
chronik-lib/
src/
4 lines
test/
functional/
74 lines

Diff 50048

View Options

chronik/chronik-lib/src/bridge.rs

Loading...
View Options

test/functional/chronik_ws.py

Loading...