Skip to content

Commit a4fc0af

Browse files
AchoArnoldCopilot
andcommitted
test: cover not-archived non-received unarchive branch
Remove an unrelated webhook email payload formatting plan doc that was accidentally included in this feature branch, and add the missing shouldCheckUnarchive case for a non-archived thread with a non-received status so every branch of the predicate is exercised. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: bf78429e-573c-406c-9f7c-1d6e1bddbbb5
1 parent 3fc08ae commit a4fc0af

2 files changed

Lines changed: 1 addition & 584 deletions

File tree

api/pkg/services/message_thread_service_test.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,4 +19,5 @@ func TestShouldCheckUnarchive(t *testing.T) {
1919
assert.True(t, service.shouldCheckUnarchive(archived, received), "archived + inbound -> consult phone setting")
2020
assert.False(t, service.shouldCheckUnarchive(archived, sent), "outbound status -> no check")
2121
assert.False(t, service.shouldCheckUnarchive(notArchived, received), "already unarchived -> no check")
22+
assert.False(t, service.shouldCheckUnarchive(notArchived, sent), "not archived + outbound -> no check")
2223
}

0 commit comments

Comments
 (0)