tests(p2p): improve protocol test

[glevco]

Motivation

The test for maximum line length in the P2P protocol is not correctly testing what it is supposed to. Currently, it checks whether the protocol is disconnecting, which happens even if the payload is smaller than the max length. Also, the size of the test payload was way larger than the actual limit. We should test the edges, 1 byte short.

Acceptance Criteria

• Improve test_invalid_size().

Checklist

• If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (126bae3) 85.31% compared to head (d02b3f4) 85.30%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #918      +/-   ##
==========================================
- Coverage   85.31%   85.30%   -0.01%     
==========================================
  Files         284      284              
  Lines       22330    22330              
  Branches     3367     3367              
==========================================
- Hits        19051    19049       -2     
- Misses       2604     2610       +6     
+ Partials      675      671       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.