Referrals email vars (#11522)

* [Referrals] Fix email vars * Send report email --------- Co-authored-by: Rémi <[email protected]>
parti-renaissance · Feb 26, 2025 · 4b44982 · 4b44982
1 parent e98e1e3
commit 4b44982
Show file tree

Hide file tree

Showing 6 changed files with 71 additions and 14 deletions.
diff --git a/features/api/referrals.feature b/features/api/referrals.feature
@@ -147,9 +147,13 @@ Feature:
                         {
                             "rcpt": "[email protected]",
                             "vars": [
+                                {
+                                    "content": "Michelle",
+                                    "name": "referrer_first_name"
+                                },
                                 {
                                     "content": "Jane",
-                                    "name": "first_name"
+                                    "name": "referred_first_name"
                                 },
                                 {
                                     "content": "http://test.renaissance.code/invitation/adhesion/@string@",

diff --git a/src/Adherent/Referral/Notifier.php b/src/Adherent/Referral/Notifier.php
@@ -7,6 +7,7 @@
 use App\Entity\Referral;
 use App\Mailer\MailerService;
 use App\Mailer\Message\Renaissance\Referral\ReferralAdhesionCreatedMessage;
+use App\Mailer\Message\Renaissance\Referral\ReferralReportedMessage;
 use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
 
 class Notifier
@@ -21,6 +22,7 @@ public function sendAdhesionMessage(Referral $referral): void
     {
         $this->transactionalMailer->sendMessage(
             ReferralAdhesionCreatedMessage::create(
+                $referral->referrer->getFirstName(),
                 $referral->emailAddress,
                 $referral->firstName,
                 $this->generateUrl(
@@ -39,6 +41,21 @@ public function sendAdhesionMessage(Referral $referral): void
         );
     }
 
+    public function sendReportMessage(Referral $referral): void
+    {
+        if (!$referral->referrer) {
+            return;
+        }
+
+        $this->transactionalMailer->sendMessage(
+            ReferralReportedMessage::create(
+                $referral->referrer->getEmailAddress(),
+                $referral->referrer->getFirstName(),
+                $referral->firstName
+            )
+        );
+    }
+
     private function generateUrl(string $routeName, array $parameters = []): string
     {
         return $this->urlGenerator->generate($routeName, $parameters, UrlGeneratorInterface::ABSOLUTE_URL);

diff --git a/src/Adherent/Referral/ReportHandler.php b/src/Adherent/Referral/ReportHandler.php
@@ -7,14 +7,18 @@
 
 class ReportHandler
 {
-    public function __construct(private readonly EntityManagerInterface $entityManager)
-    {
+    public function __construct(
+        private readonly EntityManagerInterface $entityManager,
+        private readonly Notifier $notifier,
+    ) {
     }
 
     public function report(Referral $referral): void
     {
         $referral->report();
 
         $this->entityManager->flush();
+
+        $this->notifier->sendReportMessage($referral);
     }
 }
diff --git a/src/Mailer/Message/Renaissance/Referral/ReferralAdhesionCreatedMessage.php b/src/Mailer/Message/Renaissance/Referral/ReferralAdhesionCreatedMessage.php
@@ -8,19 +8,21 @@
 class ReferralAdhesionCreatedMessage extends AbstractRenaissanceMessage
 {
     public static function create(
-        string $email,
-        string $firstName,
+        string $referrerFirstName,
+        string $referredEmail,
+        string $referredFirstName,
         string $adhesionLink,
         string $reportLink,
     ): self {
         return new self(
             Uuid::uuid4(),
-            $email,
-            $firstName,
+            $referredEmail,
+            $referredFirstName,
             'Nouveau parrainage',
             [],
             [
-                'first_name' => self::escape($firstName),
+                'referrer_first_name' => self::escape($referrerFirstName),
+                'referred_first_name' => self::escape($referredFirstName),
                 'adhesion_link' => $adhesionLink,
                 'report_link' => $reportLink,
             ]

diff --git a/src/Mailer/Message/Renaissance/Referral/ReferralReportedMessage.php b/src/Mailer/Message/Renaissance/Referral/ReferralReportedMessage.php
@@ -0,0 +1,27 @@
+<?php
+
+namespace App\Mailer\Message\Renaissance\Referral;
+
+use App\Mailer\Message\Renaissance\AbstractRenaissanceMessage;
+use Ramsey\Uuid\Uuid;
+
+class ReferralReportedMessage extends AbstractRenaissanceMessage
+{
+    public static function create(
+        string $referrerEmail,
+        string $referrerFirstName,
+        string $referredFirstName,
+    ): self {
+        return new self(
+            Uuid::uuid4(),
+            $referrerEmail,
+            $referrerFirstName,
+            'Parrainage signalé',
+            [],
+            [
+                'referrer_first_name' => self::escape($referrerFirstName),
+                'referred_first_name' => self::escape($referredFirstName),
+            ]
+        );
+    }
+}
diff --git a/tests/Controller/Renaissance/Referral/Report/FormControllerTest.php b/tests/Controller/Renaissance/Referral/Report/FormControllerTest.php
@@ -4,6 +4,7 @@
 
 use App\Adherent\Referral\StatusEnum;
 use App\Entity\Referral;
+use App\Mailer\Message\Renaissance\Referral\ReferralReportedMessage;
 use App\Repository\ReferralRepository;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\Group;
@@ -20,13 +21,14 @@ class FormControllerTest extends AbstractWebTestCase
     private ?ReferralRepository $referralRepository = null;
 
     #[DataProvider('provideReportableReferralIdentifiers')]
-    public function testReferralCanBeReported(string $referralIdentifier): void
+    public function testReferralCanBeReported(string $identifier, string $referrerEmail): void
     {
-        $referral = $this->referralRepository->findOneBy(['identifier' => $referralIdentifier]);
+        $referral = $this->referralRepository->findOneBy(['identifier' => $identifier]);
         $this->assertInstanceOf(Referral::class, $referral);
         $this->assertNotEquals(StatusEnum::REPORTED, $referral->status);
+        $this->assertCountMails(0, ReferralReportedMessage::class, $referrerEmail);
 
-        $crawler = $this->client->request(Request::METHOD_GET, "/invitation/$referralIdentifier/signaler");
+        $crawler = $this->client->request(Request::METHOD_GET, "/invitation/$identifier/signaler");
         $this->assertStatusCode(Response::HTTP_OK, $this->client);
         $this->assertStringContainsString(
             'Voulez-vous vraiment signaler ce parrainage ?',
@@ -43,8 +45,9 @@ public function testReferralCanBeReported(string $referralIdentifier): void
             $crawler->text()
         );
 
-        $referral = $this->referralRepository->findOneBy(['identifier' => $referralIdentifier]);
+        $referral = $this->referralRepository->findOneBy(['identifier' => $identifier]);
         $this->assertEquals(StatusEnum::REPORTED, $referral->status);
+        $this->assertCountMails(1, ReferralReportedMessage::class, $referrerEmail);
     }
 
     #[DataProvider('provideAlreadyReportedReferralIdentifiers')]
@@ -64,8 +67,8 @@ public function testReferralIsAlreadyReported(string $referralIdentifier): void
 
     public static function provideReportableReferralIdentifiers(): iterable
     {
-        yield ['PAB123'];
-        yield ['P789YZ'];
+        yield ['PAB123', '[email protected]'];
+        yield ['P789YZ', '[email protected]'];
     }
 
     public static function provideAlreadyReportedReferralIdentifiers(): iterable