ISSUE-12: configs + style

Author: tatevikg1

src/Domain/Messaging/Message/SubscriberConfirmationMessage.php

@@ -13,6 +13,9 @@ class SubscriberConfirmationMessage
     private string $uniqueId;
     private bool $htmlEmail;
 
+    /**
+     * @SuppressWarnings("BooleanArgumentFlag")
+     */
     public function __construct(
         string $email,
         string $uniqueId,

src/Domain/Messaging/MessageHandler/SubscriberConfirmationMessageHandler.php

@@ -35,14 +35,14 @@ public function __invoke(SubscriberConfirmationMessage $message): void
         $textContent = "Thank you for subscribing!\n\n"
             . "Please confirm your subscription by clicking the link below:\n"
             . $confirmationLink . "\n\n"
-            . "If you did not request this subscription, please ignore this email.";
+            . 'If you did not request this subscription, please ignore this email.';
 
         $htmlContent = '';
         if ($message->hasHtmlEmail()) {
-            $htmlContent = "<p>Thank you for subscribing!</p>"
-                . "<p>Please confirm your subscription by clicking the link below:</p>"
-                . "<p><a href=\"" . $confirmationLink . "\">Confirm Subscription</a></p>"
-                . "<p>If you did not request this subscription, please ignore this email.</p>";
+            $htmlContent = '<p>Thank you for subscribing!</p>'
+                . '<p>Please confirm your subscription by clicking the link below:</p>'
+                . '<p><a href="' . $confirmationLink . '">Confirm Subscription</a></p>'
+                . '<p>If you did not request this subscription, please ignore this email.</p>';
         }
 
         $email = (new Email())

src/Domain/Messaging/Service/EmailService.php

@@ -122,5 +122,4 @@ public function sendBulkEmailSync(
             $this->sendEmailSync($email, [], [], [], $attachments);
         }
     }
-
 }

src/Domain/Subscription/Service/Manager/SubscriberManager.php

@@ -21,7 +21,7 @@ class SubscriberManager
     private MessageBusInterface $messageBus;
 
     public function __construct(
-        SubscriberRepository $subscriberRepository, 
+        SubscriberRepository $subscriberRepository,
         EntityManagerInterface $entityManager,
         MessageBusInterface $messageBus
     ) {

tests/Unit/Domain/Messaging/MessageHandler/SubscriberConfirmationMessageHandlerTest.php

@@ -34,13 +34,13 @@ public function testInvokeWithTextEmail(): void
             ->with($this->callback(function (Email $email) use ($subscriberEmail, $uniqueId) {
                 $this->assertEquals([$subscriberEmail], $this->getEmailAddresses($email->getTo()));
                 $this->assertEquals('Please confirm your subscription', $email->getSubject());
-                
+
                 $textContent = $email->getTextBody();
                 $this->assertStringContainsString('Thank you for subscribing', $textContent);
                 $this->assertStringContainsString($this->confirmationUrl . $uniqueId, $textContent);
-                
+
                 $this->assertEmpty($email->getHtmlBody());
-                
+
                 return true;
             }));
 
@@ -58,15 +58,16 @@ public function testInvokeWithHtmlEmail(): void
             ->with($this->callback(function (Email $email) use ($subscriberEmail, $uniqueId) {
                 $this->assertEquals([$subscriberEmail], $this->getEmailAddresses($email->getTo()));
                 $this->assertEquals('Please confirm your subscription', $email->getSubject());
-                
+
                 $textContent = $email->getTextBody();
                 $this->assertStringContainsString('Thank you for subscribing', $textContent);
                 $this->assertStringContainsString($this->confirmationUrl . $uniqueId, $textContent);
-                
+
                 $htmlContent = $email->getHtmlBody();
                 $this->assertStringContainsString('<p>Thank you for subscribing!</p>', $htmlContent);
-                $this->assertStringContainsString('<a href="' . $this->confirmationUrl . $uniqueId . '">', $htmlContent);
-                
+                $linkStart = '<a href="' . $this->confirmationUrl . $uniqueId . '">';
+                $this->assertStringContainsString($linkStart, $htmlContent);
+
                 return true;
             }));