supabase / auth / 17190903546

71%

master: 69%

Build Type

Pull #2129

github

Committed by Chris Stockton

Commit Message

mailer: refactor mail client wiring and add validation wrapper

This change simplifies how MailClient instances are constructed and used:

- Introduce `NewMailClient` to encapsulate creation of the default
  `MailmeMailer` or `noopMailClient` based on configuration.
- Add `NewMailerWithClient` to allow injecting a custom `MailClient`.
- Extract email validation into `emailValidatorMailClient`, which wraps
  another `MailClient` and ensures `Validate` is always called before
  sending. Removed inline validation logic from `MailmeMailer`.
- Update `TemplateMailer` to consistently use `MailClient` instead of
  the previous `Mailer` field.
- Extend `noopMailClient` with an optional `Delay` field to simulate
  latency in tests.
- Clean up struct definitions by removing redundant `EmailValidator`
  from `MailmeMailer`.

Together, these changes make the mailer package easier to extend,
improve separation of concerns, and provide hooks for testing latency
and swapping in custom mail clients.

Pull Request Pull Request #2129: mailer: refactor mail client wiring and add validation wrapper

Run Details

16 of 55 new or added lines in 3 files covered. (29.09%)

1 existing line in 1 file now uncovered.

12279 of 17362 relevant lines covered (70.72%)

69.85 hits per line