RegisterUseCaseでArticleCounterを使うように

Author: 77web

src/Service/UseCase/Article/RegisterUseCase.php

@@ -4,6 +4,7 @@
 
 use App\DTO\Request\PostedArticle;
 use App\Entity\Article;
+use App\Model\ArticleCounterInterface;
 use App\Model\ArticlePersisterInterface;
 use App\Service\UseCase\Article\Exception\RegisterException;
 
@@ -14,12 +15,19 @@ class RegisterUseCase
      */
     private $articlePersister;
 
+    /**
+     * @var ArticleCounterInterface
+     */
+    private $articleCounter;
+
     /**
      * @param ArticlePersisterInterface $articlePersister
+     * @param ArticleCounterInterface $articleCounter
      */
-    public function __construct(ArticlePersisterInterface $articlePersister)
+    public function __construct(ArticlePersisterInterface $articlePersister, ArticleCounterInterface $articleCounter)
     {
         $this->articlePersister = $articlePersister;
+        $this->articleCounter = $articleCounter;
     }
 
     /**
@@ -29,9 +37,11 @@ public function __construct(ArticlePersisterInterface $articlePersister)
     public function register(PostedArticle $postedArticle): void
     {
         try {
-            $this->articlePersister->persist($postedArticle, new Article());
+            $savedArticle = $this->articlePersister->persist($postedArticle, new Article());
         } catch (\Exception $e) {
-            throw new RegisterException($e->getMessage(), '', $e);
+            throw new RegisterException($e->getMessage(), $e->getCode(), $e);
         }
+
+        $this->articleCounter->count($savedArticle);
     }
 }

tests/Service/UseCase/Article/RegisterUseCaseTest.php

@@ -4,9 +4,12 @@
 
 use App\DTO\Request\PostedArticle;
 use App\Entity\Article;
+use App\Model\ArticleCounterInterface;
 use App\Model\ArticlePersisterInterface;
+use App\Service\UseCase\Article\Exception\RegisterException;
 use App\Service\UseCase\Article\RegisterUseCase;
 use PHPUnit\Framework\TestCase;
+use Prophecy\Argument;
 use Prophecy\Prophecy\ObjectProphecy;
 
 class RegisterUseCaseTest extends TestCase
@@ -16,22 +19,49 @@ class RegisterUseCaseTest extends TestCase
      */
     private $articlePersisterP;
 
+    /**
+     * @var ArticleCounterInterface|ObjectProphecy
+     */
+    private $articleCounterP;
+
     protected function setUp(): void
     {
         $this->articlePersisterP = $this->prophesize(ArticlePersisterInterface::class);
+        $this->articleCounterP = $this->prophesize(ArticleCounterInterface::class);
+    }
+
+    protected function tearDown(): void
+    {
+        $this->articlePersisterP = null;
+        $this->articleCounterP = null;
     }
 
     public function test()
     {
         $postedArticle = $this->prophesize(PostedArticle::class)->reveal();
 
         $this->articlePersisterP->persist($postedArticle, new Article())->willReturnArgument(1)->shouldBeCalled();
+        $this->articleCounterP->count(Argument::type(Article::class))->shouldBeCalled();
+
+        $this->getSUT()->register($postedArticle);
+    }
+
+    public function test_保存でエラーが出たらcountは更新しない()
+    {
+        $this->expectException(RegisterException::class);
+
+        $postedArticle = $this->prophesize(PostedArticle::class)->reveal();
+        $this->articlePersisterP->persist($postedArticle, new Article())->willThrow( new \Exception('dummy-error'))->shouldBeCalled();
+        $this->articleCounterP->count(Argument::any())->shouldNotBeCalled();
 
         $this->getSUT()->register($postedArticle);
     }
 
     private function getSUT(): RegisterUseCase
     {
-        return new RegisterUseCase($this->articlePersisterP->reveal());
+        return new RegisterUseCase(
+            $this->articlePersisterP->reveal(),
+            $this->articleCounterP->reveal()
+        );
     }
 }