Page MenuHomePhabricator
Differential D3878

Read more reject messages from debug logs in feature_cltv.py
ClosedPublic
Actions

Authored by jasonbcox on Aug 14 2019, 19:09.

Details

Reviewers
deadalnix
Fabien
Group Reviewers
Restricted Project
Commits
rSTAGING958009f15f09: Read more reject messages from debug logs in feature_cltv.py
rABC958009f15f09: Read more reject messages from debug logs in feature_cltv.py
Summary

Partial backport of Core PR14119
https://github.com/bitcoin/bitcoin/pull/14119/files#diff-a22114218da7fa06c1bcd16d75c29bc7

Depends on D3862

Reviewer note: Some parts of the test differ due to changes done in D521. I left out the forced single-threaded validation since it's no longer relevant.

Test Plan

test_runner.py feature_cltv

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr14119-p6
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7152
Build 12349: Bitcoin ABC Buildbot (legacy)
Build 12348: arc lint + arc unit

Event Timeline

jasonbcox created this revision.Aug 14 2019, 19:09
Herald added a reviewer: deadalnix. · View Herald TranscriptAug 14 2019, 19:09
teamcity added a comment.Aug 14 2019, 19:09

Build Bitcoin-ABC / Diffs / Diff Testing started.

jasonbcox edited the summary of this revision. (Show Details)Aug 14 2019, 19:14
teamcity added a comment.Aug 14 2019, 19:15

Build Bitcoin-ABC / Diffs / Diff Testing passed.

Harbormaster completed remote builds in B7152: Diff 10796.Aug 14 2019, 19:15
Fabien accepted this revision.Aug 16 2019, 07:21
This revision is now accepted and ready to land.Aug 16 2019, 07:21
Closed by commit rABC958009f15f09: Read more reject messages from debug logs in feature_cltv.py (authored by jasonbcox). · Explain WhyAug 16 2019, 23:10
This revision was automatically updated to reflect the committed changes.
jasonbcox added a commit: rABC958009f15f09: Read more reject messages from debug logs in feature_cltv.py.
jasonbcox added a commit: rSTAGING958009f15f09: Read more reject messages from debug logs in feature_cltv.py.Sep 19 2019, 01:53
majcosta mentioned this in D8762: [backport#16658] validation: Rename CheckInputs to CheckInputScripts.Jan 13 2021, 13:58

Revision Contents
Changeset List

PathSize
test/
functional/
42 lines

Diff 10796

View Options

test/functional/feature_cltv.py

Loading...