Freegle / Iznik / 5885

70%

push

circleci

test(coverage): chat notification filtering + location_test unused var fix TDD coverage (#251)

Add 9 integration tests verifying PR #245 fix (don't notify users who have
closed or blocked a chat):

- TestClosedChatHiddenFromUserChatList: status=Closed removes chat from /api/chat
- TestBlockedChatHiddenFromUserChatList: status=Blocked removes chat from /api/chat
- TestClosedChatVisibleWithIncludeClosed: includeClosed=true overrides filter
- TestOnlySenderClosedChatRecipientStillSees: filter is per-viewer (c1.userid=myid)
- TestBothUsersClosedNeitherSeeChat: both close → neither sees the chat
- TestNullRosterStatusShowsChat: null/absent roster entry shows chat normally
- TestUser2ModClosedByMemberHiddenFromMember: member closing User2Mod chat hides it from member; mod still sees via ModTools
- TestMTUnseenCountExcludesClosedModChat: getModeratorChatIDs() filters status!=Closed
- TestClosedChatUnseenIsHiddenNotZeroed: documents that notifications are prevented by hiding the chat, not zeroing unseen count

Also fix pre-existing build failure in location_test.go (unused variable
initialPolygon declared but never read).

1936 Go tests pass.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

13314 of 20196 branches covered (65.92%)

Branch coverage included in aggregate %.

96344 of 132900 relevant lines covered (72.49%)

21.5 hits per line