Differential D14836

[avalanche] Add coverage for the avapeers dump file deserialization error behavior
ClosedPublic
Actions

Authored by Fabien on Nov 22 2023, 14:35.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCebb7113e499f: [avalanche] Add coverage for the avapeers dump file deserialization error…

Summary

Check it is decoded up to the failure point.

Test Plan

ninja check-avalanche-peermanager_tests

Diff Detail

Repository

rABC Bitcoin ABC

Branch

test_avapeers_deser_error

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25703
Build 50986: Build Diff	lint-circular-dependencies · build-diff · build-without-wallet · build-debug · build-clang-tidy · build-clang
Build 50985: arc lint + arc unit

Event Timeline

Fabien created this revision.Nov 22 2023, 14:35

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 22 2023, 14:35

Fabien requested review of this revision.Nov 22 2023, 14:35

Harbormaster completed remote builds in B25703: Diff 43216.Nov 22 2023, 15:01

PiRK accepted this revision.Nov 22 2023, 18:57

This revision is now accepted and ready to land.Nov 22 2023, 18:57

Closed by commit rABCebb7113e499f: [avalanche] Add coverage for the avapeers dump file deserialization error… (authored by Fabien). · Explain WhyNov 22 2023, 19:12

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCebb7113e499f: [avalanche] Add coverage for the avapeers dump file deserialization error….

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

peermanager_tests.cpp

59 lines

Diff 43216

View Options

src/avalanche/test/peermanager_tests.cpp

[avalanche] Add coverage for the avapeers dump file deserialization error behaviorClosedPublicActions