MissingMessageIdException

Author: tatevikg1

src/Domain/Analytics/Exception/MissingMessageIdException.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\Core\Domain\Analytics\Exception;
+
+use LogicException;
+
+class MissingMessageIdException extends LogicException
+{
+    public function __construct()
+    {
+        parent::__construct('Message must have an ID');
+    }
+}

src/Domain/Analytics/Service/LinkTrackService.php

@@ -4,8 +4,8 @@
 
 namespace PhpList\Core\Domain\Analytics\Service;
 
-use InvalidArgumentException;
 use PhpList\Core\Core\ConfigProvider;
+use PhpList\Core\Domain\Analytics\Exception\MissingMessageIdException;
 use PhpList\Core\Domain\Analytics\Model\LinkTrack;
 use PhpList\Core\Domain\Analytics\Repository\LinkTrackRepository;
 use PhpList\Core\Domain\Messaging\Model\Message;
@@ -36,7 +36,7 @@ public function isExtractAndSaveLinksApplicable(): bool
      * Extract links from message content and save them to the LinkTrackRepository
      *
      * @return LinkTrack[] The saved LinkTrack entities
-     * @throws InvalidArgumentException if the message doesn't have an ID
+     * @throws MissingMessageIdException
      */
     public function extractAndSaveLinks(Message $message, int $userId): array
     {
@@ -48,7 +48,7 @@ public function extractAndSaveLinks(Message $message, int $userId): array
         $messageId = $message->getId();
 
         if ($messageId === null) {
-            throw new InvalidArgumentException('Message must have an ID');
+            throw new MissingMessageIdException();
         }
 
         $links = $this->extractLinksFromHtml($content->getText() ?? '');

tests/Unit/Domain/Analytics/Service/LinkTrackServiceTest.php

@@ -4,8 +4,8 @@
 
 namespace PhpList\Core\Tests\Unit\Domain\Analytics\Service;
 
-use InvalidArgumentException;
 use PhpList\Core\Core\ConfigProvider;
+use PhpList\Core\Domain\Analytics\Exception\MissingMessageIdException;
 use PhpList\Core\Domain\Analytics\Model\LinkTrack;
 use PhpList\Core\Domain\Analytics\Repository\LinkTrackRepository;
 use PhpList\Core\Domain\Analytics\Service\LinkTrackService;
@@ -172,7 +172,7 @@ public function testExtractAndSaveLinksWithMessageWithoutId(): void
         $message->method('getId')->willReturn(null);
         $message->method('getContent')->willReturn($messageContent);
 
-        $this->expectException(InvalidArgumentException::class);
+        $this->expectException(MissingMessageIdException::class);
         $this->expectExceptionMessage('Message must have an ID');
 
         $this->subject->extractAndSaveLinks($message, $userId);