Page MenuHomePhabricator

[qa] Ensure bitcoind processes are cleaned up when tests end
AbandonedPublic

Authored by nakihito on Mar 25 2019, 21:41.

Details

Reviewers
jasonbcox
deadalnix
Fabien
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

[qa] Ensure bitcoind processes are cleaned up when tests end
Backport of PR12904
https://github.com/bitcoin/bitcoin/pull/12904/files

Completes T446

Test Plan

make check
test_runner.py
Interrupting (though keyboard for example) test_runner.py should not leave any zombie bitcoind instances running (checked with top)

Diff Detail

Repository
rABC Bitcoin ABC
Branch
BackportPR12904
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5295
Build 8652: Bitcoin ABC Buildbot (legacy)
Build 8651: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Mar 25 2019, 21:41

Fixed unwanted linter changes.

test/functional/test_framework/test_framework.py
1 ↗(On Diff #7825)

There are numerous linter errors outside of the scope of this backport. See history for the potential linter changes.

Fabien requested changes to this revision.Mar 26 2019, 08:59

This PR depends on PR12843 (T569) for the feature_help.py script.
This one in turns has dependencies which require T417 to be completed first (see T540, T568).

test/functional/test_framework/test_framework.py
1 ↗(On Diff #7825)

@nakihito I'm interested in knowing your autopep8 version, your python linter seems more aggressive than mine.
Also can you check if you have some configuration file which may enable more aggressive linting ?

For example I have zero error with arc lint -- test/functional/test_framework/test_framework.py.

test/functional/test_framework/test_node.py
90 ↗(On Diff #7825)

Move comment above

This revision now requires changes to proceed.Mar 26 2019, 08:59
nakihito marked an inline comment as done.

Change comment position in test_node.py.

test/functional/test_framework/test_framework.py
1 ↗(On Diff #7825)

I have autopep8 version 0.9.1. I never touched the linter's config file. I can copy-paste it if you would like.

Fabien requested changes to this revision.Mar 28 2019, 08:39

This diff should be updated after the dependencies are backported, I request change to clear my queue.
Please update and request review again when the missing file is added.

test/functional/test_framework/test_framework.py
1 ↗(On Diff #7825)

Thanks, your version is far behind mine and @jasonbcox's.
I will enforce a minimum version, you'll need to update.

This revision now requires changes to proceed.Mar 28 2019, 08:39