Differential D8860

[test framework] Simplify version message handling
ClosedPublic
Actions

Authored by deadalnix on Sat, Jan 9, 21:00.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC3584de484d22: [test framework] Simplify version message handling

Summary

We do not expect to run outdated version of the software through this test framework, so hanling outdated version messages is just dead code.

Test Plan

ninja check-all

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

deadalnix created this revision.Sat, Jan 9, 21:00

Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Jan 9, 21:00

deadalnix requested review of this revision.Sat, Jan 9, 21:00

Harbormaster completed remote builds in B14683: Diff 26824.Sat, Jan 9, 21:13

PiRK accepted this revision.Sat, Jan 9, 21:59

This revision is now accepted and ready to land.Sat, Jan 9, 21:59

deadalnix added a child revision: D8861: [net] Add extra entropy to the version message.Sat, Jan 9, 22:02

Closed by commit rABC3584de484d22: [test framework] Simplify version message handling (authored by deadalnix). · Explain WhySat, Jan 9, 22:03

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABC3584de484d22: [test framework] Simplify version message handling.

Revision Contents
Changeset List

Path

Size

test/

functional/

test_framework/

messages.py

9 lines

Diff 26826

View Options

test/functional/test_framework/messages.py

[test framework] Simplify version message handlingClosedPublicActions