You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The file is the largest uncovered implementation unit in the project. 42 existing test cases only exercise 29.2% of lines, indicating shallow coverage along the main path with most state-machine branches and error-handling branches unreached.
Why
connection.cpp is core QUIC transport logic. Regressions here surface as subtle wire-level bugs that are expensive to diagnose in production.
Part of #953
What
Raise line coverage of
src/protocols/quic/connection.cppfrom 29.2% to >= 70%, and branch coverage from 17.5% to >= 60%.Current state (2026-04-17 measurement, source PR #989):
tests/unit/quic_connection_test.cpp— 529 LOC, 42 GTEST casesThe file is the largest uncovered implementation unit in the project. 42 existing test cases only exercise 29.2% of lines, indicating shallow coverage along the main path with most state-machine branches and error-handling branches unreached.
Why
connection.cppis core QUIC transport logic. Regressions here surface as subtle wire-level bugs that are expensive to diagnose in production.Where
src/protocols/quic/connection.cppinclude/kcenon/network/.../connection.h(verify exact path during implementation)tests/unit/quic_connection_test.cpptests/unit/quic_connection_state_test.cpp(state transitions),tests/unit/quic_connection_error_test.cpp(error paths)How
Approach
gcovr --show-branchesor opencoverage_html/src/protocols/quic/connection.cpp.gcov.htmlfrom PR ci(coverage): enable messaging_bridge in coverage build #989's coverage artifact to pinpoint uncovered blocks.Acceptance Criteria
src/protocols/quic/connection.cppline coverage >= 70%src/protocols/quic/connection.cppbranch coverage >= 60%tests/unit/quic_connection_*.cpp(one file or multiple)Out of Scope
connection.cppitself. Implementation changes should be opened as a separate issue.integration_tests/, not this issue).Dependencies