Rename course use case (#14)

* Add course finder

* Rename course

Author: rgomezcasas

src/Mooc/Courses/Application/Find/CourseFinder.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace CodelyTv\Mooc\Courses\Application\Find;
+
+use CodelyTv\Mooc\Courses\Domain\Course;
+use CodelyTv\Mooc\Courses\Domain\CourseNotExist;
+use CodelyTv\Mooc\Courses\Domain\CourseRepository;
+use CodelyTv\Mooc\Shared\Domain\Course\CourseId;
+
+final class CourseFinder
+{
+    private $repository;
+
+    public function __construct(CourseRepository $repository)
+    {
+        $this->repository = $repository;
+    }
+
+    public function __invoke(CourseId $id): Course
+    {
+        $course = $this->repository->search($id);
+
+        if (null === $course) {
+            throw new CourseNotExist($id);
+        }
+
+        return $course;
+    }
+}

src/Mooc/Courses/Application/Update/CourseRenamer.php

@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace CodelyTv\Mooc\Courses\Application\Update;
+
+use CodelyTv\Mooc\Courses\Application\Find\CourseFinder;
+use CodelyTv\Mooc\Courses\Domain\CourseName;
+use CodelyTv\Mooc\Courses\Domain\CourseRepository;
+use CodelyTv\Mooc\Shared\Domain\Course\CourseId;
+use CodelyTv\Shared\Domain\Bus\Event\EventBus;
+
+final class CourseRenamer
+{
+    private $repository;
+    private $finder;
+    private $bus;
+
+    public function __construct(CourseRepository $repository, EventBus $bus)
+    {
+        $this->repository = $repository;
+        $this->finder     = new CourseFinder($repository);
+        $this->bus        = $bus;
+    }
+
+    public function __invoke(CourseId $id, CourseName $newName): void
+    {
+        $course = $this->finder->__invoke($id);
+
+        $course->rename($newName);
+
+        $this->repository->save($course);
+        $this->bus->publish(...$course->pullDomainEvents());
+    }
+}

src/Mooc/Courses/Domain/Course.php

@@ -43,4 +43,9 @@ public function duration(): CourseDuration
     {
         return $this->duration;
     }
+
+    public function rename(CourseName $newName): void
+    {
+        $this->name = $newName;
+    }
 }

src/Mooc/Courses/Domain/CourseNotExist.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace CodelyTv\Mooc\Courses\Domain;
+
+use CodelyTv\Mooc\Shared\Domain\Course\CourseId;
+use CodelyTv\Shared\Domain\DomainError;
+
+final class CourseNotExist extends DomainError
+{
+    private $id;
+
+    public function __construct(CourseId $id)
+    {
+        $this->id = $id;
+
+        parent::__construct();
+    }
+
+    public function errorCode(): string
+    {
+        return 'course_not_exist';
+    }
+
+    protected function errorMessage(): string
+    {
+        return sprintf('The course <%s> does not exist', $this->id->value());
+    }
+}

tests/src/Mooc/Courses/Application/Update/CourseRenamerTest.php

@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace CodelyTv\Tests\Mooc\Courses\Application\Update;
+
+use CodelyTv\Mooc\Courses\Application\Update\CourseRenamer;
+use CodelyTv\Mooc\Courses\Domain\CourseNotExist;
+use CodelyTv\Tests\Mooc\Courses\CoursesModuleUnitTestCase;
+use CodelyTv\Tests\Mooc\Courses\Domain\CourseIdMother;
+use CodelyTv\Tests\Mooc\Courses\Domain\CourseMother;
+use CodelyTv\Tests\Mooc\Courses\Domain\CourseNameMother;
+use CodelyTv\Tests\Shared\Domain\DuplicatorMother;
+
+final class CourseRenamerTest extends CoursesModuleUnitTestCase
+{
+    private $renamer;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->renamer = new CourseRenamer($this->repository(), $this->eventBus());
+    }
+
+    /** @test */
+    public function it_should_rename_an_existing_course(): void
+    {
+        $course        = CourseMother::random();
+        $newName       = CourseNameMother::random();
+        $renamedCourse = DuplicatorMother::with($course, ['name' => $newName]);
+
+        $this->shouldSearch($course->id(), $course);
+        $this->shouldSave($renamedCourse);
+        $this->shouldNotPublishDomainEvent();
+
+        $this->renamer->__invoke($course->id(), $newName);
+    }
+
+    /** @test */
+    public function it_should_throw_an_exception_when_the_course_not_exist(): void
+    {
+        $this->expectException(CourseNotExist::class);
+
+        $id = CourseIdMother::random();
+
+        $this->shouldSearch($id, null);
+
+        $this->renamer->__invoke($id, CourseNameMother::random());
+    }
+}

tests/src/Mooc/Courses/CoursesModuleUnitTestCase.php

@@ -6,6 +6,7 @@
 
 use CodelyTv\Mooc\Courses\Domain\Course;
 use CodelyTv\Mooc\Courses\Domain\CourseRepository;
+use CodelyTv\Mooc\Shared\Domain\Course\CourseId;
 use CodelyTv\Tests\Shared\Infrastructure\PhpUnit\UnitTestCase;
 use Mockery\MockInterface;
 
@@ -22,6 +23,15 @@ protected function shouldSave(Course $course): void
             ->andReturnNull();
     }
 
+    protected function shouldSearch(CourseId $id, ?Course $course): void
+    {
+        $this->repository()
+            ->shouldReceive('search')
+            ->with($this->similarTo($id))
+            ->once()
+            ->andReturn($course);
+    }
+
     /** @return CourseRepository|MockInterface */
     protected function repository(): MockInterface
     {

tests/src/Shared/Domain/DuplicatorMother.php

@@ -0,0 +1,30 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace CodelyTv\Tests\Shared\Domain;
+
+use ReflectionObject;
+use ReflectionProperty;
+use function Lambdish\Phunctional\each;
+
+final class DuplicatorMother
+{
+    public static function with($object, array $newParams)
+    {
+        $duplicated = clone $object;
+        $reflection = new ReflectionObject($duplicated);
+
+        each(
+            static function (ReflectionProperty $property) use ($duplicated, $newParams) {
+                if (isset($newParams[$property->getName()])) {
+                    $property->setAccessible(true);
+                    $property->setValue($duplicated, $newParams[$property->getName()]);
+                }
+            },
+            $reflection->getProperties()
+        );
+
+        return $duplicated;
+    }
+}

tests/src/Shared/Infrastructure/PhpUnit/UnitTestCase.php

@@ -34,6 +34,14 @@ protected function shouldPublishDomainEvent(DomainEvent $domainEvent): void
             ->andReturnNull();
     }
 
+    protected function shouldNotPublishDomainEvent(): void
+    {
+        $this->eventBus()
+            ->shouldReceive('publish')
+            ->withNoArgs()
+            ->andReturnNull();
+    }
+
     /** @return EventBus|MockInterface */
     protected function eventBus(): MockInterface
     {