#3: close 3 known F1 gaps - folder/mailbox counters, quota-at-RCPT, alias fan-out
Per feature/3-f0-f1-validation's own gap analysis (commit 097fa37):
- InboundMessageRepository.Persist now updates folders.unread_count,
folders.total_count, and mailboxes.used_bytes in the same transaction
as the message insert (PLAN.md section 6.1 "++counters" was a no-op
before this).
- MailboxRecord carries QuotaBytes/UsedBytes; ResolveRecipient rejects
RCPT with 452 4.2.2 when a target mailbox has already reached its
quota (used_bytes >= quota_bytes), before DATA - never silently
accepting mail into a full mailbox.
- MailboxRepository.ResolveDeliveryTargets tries a direct mailbox match
first, then falls back to the aliases table (exact source_address
match takes precedence over a domain catch-all). One RCPT TO can now
fan out to multiple mailboxes; alias destinations that don't resolve
to an internal active mailbox are silently skipped, since outbound/
external forwarding doesn't exist yet. postmaster@/abuse@/dmarc@-style
aliases now actually deliver.
The E2E test's counters_updated subtest flips from a documented KNOWN
GAP (t.Skipf) to a hard assertion now that it passes, and gains two new
subtests (quota_rejected_when_mailbox_full, alias_resolves_to_destination_mailbox)
covering the other two fixes end-to-end. New unit and Postgres
integration test coverage added for all three fixes at every layer.
Verified: full suite green on Windows (unit + E2E via embedded
Postgres, no Docker needed) and via WSL Docker against the real
docker-compose Postgres (all infrastructure/postgres and diskstorage
integration tests, not just E2E). lang-lint clean.
This commit is contained in:
1 parent
097fa373e1
commit
2f7389841d
10 files changed
+472
-31
No files matched your search
@@ -23,19 +23,27 @@ func (f *fakeMailboxRepository) FindActiveByAddress(_ context.Context, address s
|
||||
return &rec, nil
|
||||
}
|
||||
|
||||
func (f *fakeMailboxRepository) ResolveDeliveryTargets(_ context.Context, address string) ([]port.MailboxRecord, error) {
|
||||
rec, ok := f.byAddress[address]
|
||||
if !ok {
|
||||
return []port.MailboxRecord{}, nil
|
||||
}
|
||||
return []port.MailboxRecord{rec}, nil
|
||||
}
|
||||
|
||||
func TestUseCase_ResolveRecipient_ReturnsRecordForActiveMailbox(t *testing.T) {
|
||||
mailboxID := uuid.New()
|
||||
mailboxes := &fakeMailboxRepository{byAddress: map[string]port.MailboxRecord{
|
||||
"user@example.test": {ID: mailboxID, MaxMessageBytes: 1000},
|
||||
"user@example.test": {ID: mailboxID, MaxMessageBytes: 1000, QuotaBytes: 1000},
|
||||
}}
|
||||
uc := NewProcessInboundEmailUseCase(mailboxes, nil, nil)
|
||||
|
||||
rec, err := uc.ResolveRecipient(context.Background(), "user@example.test")
|
||||
targets, err := uc.ResolveRecipient(context.Background(), "user@example.test")
|
||||
if err != nil {
|
||||
t.Fatalf("unexpected error: %v", err)
|
||||
}
|
||||
if rec.ID != mailboxID {
|
||||
t.Errorf("ID = %v, want %v", rec.ID, mailboxID)
|
||||
if len(targets) != 1 || targets[0].ID != mailboxID {
|
||||
t.Errorf("targets = %+v, want single record with ID %v", targets, mailboxID)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -49,6 +57,44 @@ func TestUseCase_ResolveRecipient_ReturnsErrRecipientNotFoundForUnknownAddress(t
|
||||
}
|
||||
}
|
||||
|
||||
func TestUseCase_ResolveRecipient_ReturnsErrMailboxQuotaExceededWhenFull(t *testing.T) {
|
||||
mailboxes := &fakeMailboxRepository{byAddress: map[string]port.MailboxRecord{
|
||||
"full@example.test": {ID: uuid.New(), QuotaBytes: 1000, UsedBytes: 1000},
|
||||
}}
|
||||
uc := NewProcessInboundEmailUseCase(mailboxes, nil, nil)
|
||||
|
||||
_, err := uc.ResolveRecipient(context.Background(), "full@example.test")
|
||||
if !errors.Is(err, ErrMailboxQuotaExceeded) {
|
||||
t.Fatalf("error = %v, want ErrMailboxQuotaExceeded", err)
|
||||
}
|
||||
}
|
||||
|
||||
func TestUseCase_ResolveRecipient_ReturnsErrMailboxQuotaExceededWhenAnyFanOutTargetIsFull(t *testing.T) {
|
||||
mailboxes := &fakeAliasFanOutRepository{targets: []port.MailboxRecord{
|
||||
{ID: uuid.New(), QuotaBytes: 1000, UsedBytes: 0},
|
||||
{ID: uuid.New(), QuotaBytes: 1000, UsedBytes: 1000},
|
||||
}}
|
||||
uc := NewProcessInboundEmailUseCase(mailboxes, nil, nil)
|
||||
|
||||
_, err := uc.ResolveRecipient(context.Background(), "alias@example.test")
|
||||
if !errors.Is(err, ErrMailboxQuotaExceeded) {
|
||||
t.Fatalf("error = %v, want ErrMailboxQuotaExceeded when any fan-out target is full", err)
|
||||
}
|
||||
}
|
||||
|
||||
// fakeAliasFanOutRepository simulates an alias resolving to multiple mailboxes.
|
||||
type fakeAliasFanOutRepository struct {
|
||||
targets []port.MailboxRecord
|
||||
}
|
||||
|
||||
func (f *fakeAliasFanOutRepository) FindActiveByAddress(_ context.Context, _ string) (*port.MailboxRecord, error) {
|
||||
return nil, nil
|
||||
}
|
||||
|
||||
func (f *fakeAliasFanOutRepository) ResolveDeliveryTargets(_ context.Context, _ string) ([]port.MailboxRecord, error) {
|
||||
return f.targets, nil
|
||||
}
|
||||
|
||||
type stubBlobStorage struct {
|
||||
ref port.BlobRef
|
||||
err error
|
||||
|
||||
Reference in new issue
Block a user