Page MenuHomePhabricator

test: Adapt test framework for chains other than "regtest"
ClosedPublic

Authored by deadalnix on May 2 2020, 19:34.

Details

Summary
  • test: Fix “local variable 'e' is assigned to but never used”

flake8 F841 lints, as of flake8 3.6.0

  • test: Adapt test framework for chains other than "regtest"

Co-Authored-By: Jorge Timón <jtimon@jtimon.cc>

  • test: Avoid hardcoding the chain name in combine_logs

This is a backport of Core PR16509

The original PR contains a lot of formatting that was made obsolete by our process.

Test Plan
ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr16509
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 10560
Build 18931: Default Diff Build & Tests
Build 18930: arc lint + arc unit

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.May 3 2020, 09:11