Page MenuHomePhabricator

Added txtools utility functions
ClosedPublic

Authored by jasonbcox on Oct 5 2018, 20:08.

Details

Summary

Part 1 of many for incorporating changes from https://github.com/bitcoinxt/bitcoinxt/pull/487
The goal is to improve the test suite so that it doesn't fail post-fork.
Starts work on T418

This diff differs from the original primarily in variable/function naming and other minor cleanups.

Co-authored-by: Jason B. Cox <contact@jasonbcox.com>

Test Plan

test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
tests
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3534
Build 5144: Bitcoin ABC Buildbot (legacy)
Build 5143: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Oct 6 2018, 09:22
deadalnix added inline comments.
test/functional/test_framework/txtools.py
26 ↗(On Diff #5296)

There was a bug with that in some test. getrandbit can starts with too many zeros once in a while and you'll get something smaller than you expect. Digging into the HF activation test history should give you all the details.

This revision now requires changes to proceed.Oct 6 2018, 09:22
deadalnix added inline comments.
test/functional/test_framework/blocktools.py
66

I think we should keep the old school style padding here. It's how it's done by miners in practice.

This revision is now accepted and ready to land.Oct 8 2018, 21:12
This revision was automatically updated to reflect the committed changes.