novox/mesh-catalog
13
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

mesh-mail: the refused-sign-in acceptance test's wait excluded queued too, so it returned done on the very first look before the sender ever claimed anything (issue 1198)

Browse Source
This commit is contained in:
jochen
2026-10-19 11:10:36 +02:00
parent 06dfce87f5
commit dd8305d97d
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/mesh-mail/cmd/mesh-mail/acceptance_integration_test.go
+6 -1
View File
@@ -277,7 +277,12 @@ func TestMailSenderAcceptanceARefusedSignInFailsAfterItsAttemptsAndNothingIsRece
if err != nil || !ok || kept.Decided == nil {
return false, "not yet decided"
}
return kept.Decided.Outcome != "retrying" && kept.Decided.Outcome != "sending", kept.Decided.Outcome
// "queued" is the send's own starting state (queueAnApprovedSend's own write, before the sender
// ever claims it): without excluding it here too, this check reads done on the very first look,
// long before the sender connects or claims anything at all (novox/hq review: build-1792400882114092431,
// 1198/AC4).
o := kept.Decided.Outcome
return o != "queued" && o != "retrying" && o != "sending", o
})
cancel()
<-done