Page MenuHomePhabricator

Add test case and release notes for auto finalization
ClosedPublic

Authored by deadalnix on Tue, Nov 20, 01:45.

Details

Summary

As per title.

Depends on D2102

Test Plan
make check
./test/functional/test_runner.py abc-finalize-block

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix created this revision.Tue, Nov 20, 01:45
Herald added a reviewer: Restricted Project. · View Herald TranscriptTue, Nov 20, 01:45
deadalnix changed the visibility from "Public (No Login Required)" to "Restricted Project (Project)".Tue, Nov 20, 01:45
deadalnix edited the test plan for this revision. (Show Details)
schancel accepted this revision.Tue, Nov 20, 01:48
This revision is now accepted and ready to land.Tue, Nov 20, 01:48
schancel added inline comments.Tue, Nov 20, 01:50
test/functional/abc-finalize-block.py
57 ↗(On Diff #5953)

Please assert this again here.

assert_equal(node.getbestblockhash(), tip)
deadalnix updated this revision to Diff 5954.Tue, Nov 20, 01:52

Add assertion

jasonbcox added inline comments.
doc/release-notes.md
9 ↗(On Diff #5953)

penality -> penalty

alternatives chain -> alternative chains

deadalnix updated this revision to Diff 5955.Tue, Nov 20, 01:58

Update release notes

jasonbcox accepted this revision.Tue, Nov 20, 01:59
This revision was automatically updated to reflect the committed changes.
deadalnix changed the visibility from "Restricted Project (Project)" to "Public (No Login Required)".Wed, Nov 21, 13:29