umputun / revdiff / 25687512254

90%

master: 90%

Pull #181

github

test(toc-wheel): assert CurrentLineIdx ok flag in WheelInTOCJumpsViewport

Copilot review on #181 caught that several test sites ignored the ok return value from TOC.CurrentLineIdx(). On the clamp tests (wheel-up at top, wheel-down at bottom) that's a real defensive gap: if a regression invalidated the TOC cursor, CurrentLineIdx returns (0, false) and the assertion against the default 0 value silently passes.

Switched all 6 _ := patterns in WheelInTOCJumpsViewport to ok := with require.True. Matches the pattern already used in the two pass-through subtests (lines 1067-1068, 1085-1086). Three sites are the ones Copilot flagged; the other three are the same shape and were fixed for consistency.

10 of 10 new or added lines in 1 file covered. (100.0%)

9753 of 10862 relevant lines covered (89.79%)

3194.99 hits per line