Как отправлять письма в cqs команде?

Question

[superkolya]

Допустим, у нас есть команда. У команды есть обработчик и в нем нам нужно отправить письмо (письмо - это прям доменное требование). Подскажите как это делать, чтобы еще и было тестируемо юнит тестами? Просто принимать какой-нибудь swiftmailer, забить на принцип инверсии зависимостей, stub'нуть swiftmailer и пусть так и живет?

Answer 1

[Максим]

Если будете напрямую подключать swiftmailer будете зависеть от него. Сделайте свой Sender, можно в виде интерфейса. Через DI будете подключать нужную реализацию. Вызывайте в команде. Тестируете сам сервис отправки, а не всю команду целиком.

Comments

[superkolya]

А можно посмотреть реализацию сендера? Тут тогда получается такая ситуация, что если у нас несколько таких команд, то для каждой такой команды нужна реализация интерфейса сендера. А содержание для каждой команды может быть свое, адресаты, копии и тд. для каждый команды может быть свое, то у нас получится куча классов - реализаций интерфейса сендера. Это норм Вы считаете?

[Максим]

superkolya,

class ResetTokenSender
{
    private $mailer;
    private $twig;

    public function __construct(\Swift_Mailer $mailer, Environment $twig)
    {
        $this->mailer = $mailer;
        $this->twig = $twig;
    }

    public function send(Email $email, Token $token): void
    {
        $message = (new \Swift_Message('Password resetting'))
            ->setTo($email->getValue())
            ->setBody($this->twig->render('mail/user/reset.html.twig', [
                'token' => $token->getToken()
            ]), 'text/html');

        if (!$this->mailer->send($message)) {
            throw new \RuntimeException('Unable to send message.');
        }
    }
}

[Максим]

superkolya, это норм, если это требуется. Если они разные, то можно делать разные. Если одинаковые, то на все SenderInterface можно повесить один обработчик. Все зависит от конкретных задач. Копипаст не всегда плохо. Иногда оправданно. Но интерфейсы точно должны быть разные везде.

[Roll Back]

superkolya, это называется разделение ответсвенности, копайте в эту сторону. Реализация на 90% будет зависеть от Ваших требований.