Freegle / Iznik / 5019

72%

master: 70%

Build Type

Pull #212

circleci

Committed by edwh

Commit Message

fix(ci): extract time-gated email health to cover it unconditionally for Coveralls-go on PR #212

Coveralls-go failed PR #212 with "Coverage decreased (-0.07%)" even
though the PR only touches iznik-nuxt3 JS. Diff of Go source coverage
between PR job 180487491 and master baseline job 180484463 pinpointed
the regression to session.go:1172-1194 — the 20 lines inside
`if nowHour >= 7 && nowHour < 22`. Master's Go job ran at 19:05 UTC
(inside the window, lines covered); this PR's Go job ran at 23:06 UTC
(outside, lines uncovered). Coverage of this block depended on the
wall-clock hour CI executed at, which is not deterministic.

Fix: extract the email health goroutines into session.FetchEmailHealth
(db, hour) so the logic is exercised regardless of CI run time, and
add a Go unit test that calls it with both in-window (7, 12, 21) and
out-of-window (0, 6, 22, 23) hours to lock in coverage of both
branches.

All 1776 Go tests pass (status API /api/tests/go).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Pull Request Pull Request #212: fix(sentry): guard 'in' operator in filter against undefined (NUXT3-D2C)

Coverage Stats

13269 of 20243 branches covered (65.55%)

Branch coverage included in aggregate %.

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

15 existing lines in 8 files now uncovered.

95784 of 132163 relevant lines covered (72.47%)

21.78 hits per line