Page MenuHomePhabricator

Migrate abc-invalid-chains off of the ComparisonTestFramework

Authored by jasonbcox on Aug 12 2019, 23:18.



See title.

Depends on D3862 due to a difficult to pinpoint robustness issue without that change.
Rather than attempt to solve the issue, we can get rid of it entirely by backporting D3862 first.

Progress towards T669

Test Plan abc-invalid-chains

Diff Detail

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

Event Timeline

jasonbcox created this revision.Aug 12 2019, 23:18
jasonbcox edited the summary of this revision. (Show Details)Aug 13 2019, 00:51
Fabien requested changes to this revision.Aug 13 2019, 07:49
Fabien added a subscriber: Fabien.
Fabien added inline comments.
46 ↗(On Diff #10748)

Nit: the name is confusing because sync_blocks() is already defined by the test_framework (twice...).

This revision now requires changes to proceed.Aug 13 2019, 07:49
jasonbcox added inline comments.Aug 14 2019, 22:46
46 ↗(On Diff #10748)

I further realized after copying this over from Core's tests, that this intermediate function isn't necessary in this test since there's no reconnecting going on. I'll just remove it entirely.

jasonbcox updated this revision to Diff 10801.Aug 14 2019, 22:53

Cleanup unnecessary and confusing sync_blocks function.

Fabien accepted this revision.Aug 16 2019, 08:00
This revision is now accepted and ready to land.Aug 16 2019, 08:00