Page MenuHomePhabricator

[tests] Remove unused variables
ClosedPublic

Authored by nakihito on May 10 2019, 22:09.

Details

Reviewers
jasonbcox
deadalnix
Fabien
markblundeberg
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Commits
rABCfb6518323802: [tests] Remove unused variables
Summary

[tests] Remove unused variables

Backport of Core PR10047
https://github.com/bitcoin/bitcoin/pull/10047/
Commit 2/2

Depends on D3023

Test Plan
test_runner.py

Diff Detail

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

Event Timeline

nakihito created this revision.May 10 2019, 22:09
Owners added a reviewer: Restricted Owners Package.May 10 2019, 22:09
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 10 2019, 22:09
deadalnix requested changes to this revision.May 12 2019, 23:29

Where is part 1 ? Why doesn't this depends on part 1 ?

Also fix the test plan so command appear like command, prefixed with 2 spaces.

This revision now requires changes to proceed.May 12 2019, 23:29
nakihito edited the summary of this revision. (Show Details)EditedMay 13 2019, 00:10

Sorry, I must've forgotten to put the dependency in. Added now. Will fix the test plan later tonight.

nakihito edited the summary of this revision. (Show Details)May 13 2019, 17:14
nakihito edited the test plan for this revision. (Show Details)
nakihito requested review of this revision.May 15 2019, 01:00
Fabien accepted this revision.May 16 2019, 15:00
deadalnix accepted this revision.May 16 2019, 15:50
This revision is now accepted and ready to land.May 16 2019, 15:50
This revision was automatically updated to reflect the committed changes.