Skip to content

test(jobs): email-forwarder coverage ≥95%#48

Merged
mastermanas805 merged 1 commit into
masterfrom
coverage/worker-jobs-email-95
May 22, 2026
Merged

test(jobs): email-forwarder coverage ≥95%#48
mastermanas805 merged 1 commit into
masterfrom
coverage/worker-jobs-email-95

Conversation

@mastermanas805
Copy link
Copy Markdown
Member

Summary

  • Hermetic coverage tests (sqlmock + miniredis) for the email-forwarder jobs — no source changes.
  • Covers audit_log-kind → email-kind mappings, Brevo client classification (success/4xx/5xx/429), forwarder_sent ledger writes, every Work() branch + renderer.

Coverage (statement-weighted)

  • event_email_forwarder.go 95.03%
  • event_email_mapping.go 98.95%
  • lifecycle_emails.go 98.52%

Test plan

  • make gate green (build + vet + go test ./... -short)
  • go test ./internal/jobs -run 'TestEventEmail|TestLifecycle|TestForwarder' -coverprofile ≥95% on all three files
  • full go test ./internal/jobs green

🤖 Generated with Claude Code

Add hermetic (sqlmock + miniredis) coverage tests for the email-forwarder
jobs. Covers every audit_log-kind → email-kind mapping, the Brevo client
seam (success/4xx/5xx/429 classification), forwarder_sent ledger writes,
and every Work() branch + renderer. Test-only — no source changes.

  event_email_forwarder.go  95.03%
  event_email_mapping.go    98.95%
  lifecycle_emails.go       98.52%

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mastermanas805 mastermanas805 merged commit 18da6af into master May 22, 2026
10 checks passed
@mastermanas805 mastermanas805 deleted the coverage/worker-jobs-email-95 branch May 22, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant