Differential D2346

qa: Remove unused NodeConn members
ClosedPublic
Actions

Authored by Fabien on Jan 19 2019, 17:51.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rSTAGINGf213a7c26c78: qa: Remove unused NodeConn members
rABCf213a7c26c78: qa: Remove unused NodeConn members

Summary

Backport of core PR11677

Test Plan

./test/functional/test_runner.py --extended

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jan 19 2019, 17:51

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 19 2019, 17:51

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Harbormaster failed remote builds in B4584: Diff 6749!Jan 19 2019, 18:06

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing failed.

You should rebase this on top of something clean. The CI failed for something that looks unrelated to that patch.

The patch itself looks good. The fact that the changes were limited to abc tests illustrates that we probably aren't doing what's best in these and other tests use an alternative to achieve the same end goal. It would be interesting to look into what that alternative is.

This revision now requires changes to proceed.Jan 20 2019, 00:56

Rebase

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing failed.

Harbormaster failed remote builds in B4591: Diff 6759!Jan 20 2019, 10:40

The CI seems to have some trouble, it failed to fetch the diff.
It's likely that the abc-* tests here can be improved, I put it on my todo list but I prefer to do it in a later diff.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B4591: Diff 6759.Jan 20 2019, 11:05

deadalnix accepted this revision.Jan 20 2019, 14:53

This revision is now accepted and ready to land.Jan 20 2019, 14:53

Closed by commit rABCf213a7c26c78: qa: Remove unused NodeConn members (authored by MarcoFalke <falke.marco@gmail.com>, committed by Fabien). · Explain WhyJan 21 2019, 08:10

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rSTAGINGf213a7c26c78: qa: Remove unused NodeConn members.Sep 18 2019, 21:59