From 410c3fac1edf6493c1bdf8e49ee83fc09010285a Mon Sep 17 00:00:00 2001
From: Eduardo Bravo <eduardosnm@gmail.com>
Date: Sat, 9 Oct 2021 16:41:41 -0300
Subject: [PATCH 1/2] Update video command

---
 .../Update/UpdateVideoTitleCommand.php        | 28 +++++++++++++++++++
 .../Update/UpdateVideoTitleCommandHandler.php | 26 +++++++++++++++++
 2 files changed, 54 insertions(+)
 create mode 100644 src/Mooc/Videos/Application/Update/UpdateVideoTitleCommand.php
 create mode 100644 src/Mooc/Videos/Application/Update/UpdateVideoTitleCommandHandler.php

diff --git a/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommand.php b/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommand.php
new file mode 100644
index 00000000..e8b48e04
--- /dev/null
+++ b/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommand.php
@@ -0,0 +1,28 @@
+<?php
+
+namespace CodelyTv\Mooc\Videos\Application\Update;
+
+use CodelyTv\Shared\Domain\Bus\Command\Command;
+
+class UpdateVideoTitleCommand implements Command
+{
+
+    public function __construct(
+        private string $id,
+        private string $title
+    )
+    {
+    }
+
+    public function id(): string
+    {
+        return $this->id;
+    }
+
+    public function title(): string
+    {
+        return $this->title;
+    }
+
+
+}
diff --git a/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommandHandler.php b/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommandHandler.php
new file mode 100644
index 00000000..69b54def
--- /dev/null
+++ b/src/Mooc/Videos/Application/Update/UpdateVideoTitleCommandHandler.php
@@ -0,0 +1,26 @@
+<?php
+
+namespace CodelyTv\Mooc\Videos\Application\Update;
+
+use CodelyTv\Mooc\Videos\Domain\VideoId;
+use CodelyTv\Mooc\Videos\Domain\VideoTitle;
+use CodelyTv\Shared\Domain\Bus\Command\CommandHandler;
+
+class UpdateVideoTitleCommandHandler implements CommandHandler
+{
+
+    private VideoTitleUpdater $titleUpdater;
+
+    public function __construct(VideoTitleUpdater $titleUpdater)
+    {
+        $this->titleUpdater = $titleUpdater;
+    }
+
+    public function __invoke(UpdateVideoTitleCommand $command)
+    {
+        $id = new VideoId($command->id());
+        $title = new VideoTitle($command->title());
+
+        $this->titleUpdater->__invoke($id, $title);
+    }
+}

From 12a5f80a142165d330fd5a693ef690605cb5a79e Mon Sep 17 00:00:00 2001
From: Eduardo Bravo <eduardosnm@gmail.com>
Date: Sat, 9 Oct 2021 16:42:46 -0300
Subject: [PATCH 2/2] Update video command tests

---
 .../Update/UpdateVideoTitleCommandMother.php  | 34 ++++++++++++++
 .../Application/Update/VideoFinderTest.php    | 36 +++++++++++++++
 .../Update/VideoRenamerCommandTest.php        | 44 ++++++++++++++++++
 tests/Mooc/Videos/Domain/VideoIdMother.php    | 17 +++++++
 tests/Mooc/Videos/Domain/VideoMother.php      | 46 +++++++++++++++++++
 tests/Mooc/Videos/Domain/VideoTitleMother.php | 14 ++++++
 tests/Mooc/Videos/Domain/VideoTypeMother.php  | 13 ++++++
 tests/Mooc/Videos/Domain/VideoUrlMother.php   | 14 ++++++
 .../Mooc/Videos/VideosModuleUnitTestCase.php  | 42 +++++++++++++++++
 tests/Shared/Domain/UrlMother.php             | 11 +++++
 10 files changed, 271 insertions(+)
 create mode 100644 tests/Mooc/Videos/Application/Update/UpdateVideoTitleCommandMother.php
 create mode 100644 tests/Mooc/Videos/Application/Update/VideoFinderTest.php
 create mode 100644 tests/Mooc/Videos/Application/Update/VideoRenamerCommandTest.php
 create mode 100644 tests/Mooc/Videos/Domain/VideoIdMother.php
 create mode 100644 tests/Mooc/Videos/Domain/VideoMother.php
 create mode 100644 tests/Mooc/Videos/Domain/VideoTitleMother.php
 create mode 100644 tests/Mooc/Videos/Domain/VideoTypeMother.php
 create mode 100644 tests/Mooc/Videos/Domain/VideoUrlMother.php
 create mode 100644 tests/Mooc/Videos/VideosModuleUnitTestCase.php
 create mode 100644 tests/Shared/Domain/UrlMother.php

diff --git a/tests/Mooc/Videos/Application/Update/UpdateVideoTitleCommandMother.php b/tests/Mooc/Videos/Application/Update/UpdateVideoTitleCommandMother.php
new file mode 100644
index 00000000..e7248da6
--- /dev/null
+++ b/tests/Mooc/Videos/Application/Update/UpdateVideoTitleCommandMother.php
@@ -0,0 +1,34 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Application\Update;
+
+use CodelyTv\Mooc\Videos\Application\Update\UpdateVideoTitleCommand;
+use CodelyTv\Mooc\Videos\Domain\VideoId;
+use CodelyTv\Mooc\Videos\Domain\VideoTitle;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoIdMother;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoTitleMother;
+use PHPUnit\Framework\TestCase;
+
+class UpdateVideoTitleCommandMother extends TestCase
+{
+    public static function create(
+        ?VideoId $id = null,
+        ?VideoTitle $title = null
+    ): UpdateVideoTitleCommand
+    {
+        return new UpdateVideoTitleCommand(
+            $id->value() ?? VideoIdMother::create()->value(),
+            $title->value() ?? VideoTitleMother::create()->value()
+        );
+    }
+
+    public static function withId(VideoId $id): UpdateVideoTitleCommand
+    {
+        return self::create($id, VideoTitleMother::create());
+    }
+
+    public static function withIdAndTitle(VideoId $id, VideoTitle $title): UpdateVideoTitleCommand
+    {
+        return self::create($id, $title);
+    }
+}
diff --git a/tests/Mooc/Videos/Application/Update/VideoFinderTest.php b/tests/Mooc/Videos/Application/Update/VideoFinderTest.php
new file mode 100644
index 00000000..e41401c0
--- /dev/null
+++ b/tests/Mooc/Videos/Application/Update/VideoFinderTest.php
@@ -0,0 +1,36 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Application\Update;
+
+use CodelyTv\Mooc\Videos\Application\Update\UpdateVideoTitleCommandHandler;
+use CodelyTv\Mooc\Videos\Application\Update\VideoTitleUpdater;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoMother;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoTitleMother;
+use CodelyTv\Tests\Mooc\Videos\VideosModuleUnitTestCase;
+use CodelyTv\Tests\Shared\Domain\DuplicatorMother;
+
+class VideoFinderTest extends VideosModuleUnitTestCase
+{
+    private UpdateVideoTitleCommandHandler|null $handler;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->handler = new UpdateVideoTitleCommandHandler(new VideoTitleUpdater($this->repository()));
+    }
+
+    /** @test */
+    public function it_should_update_video_title_when_video_exists(): void
+    {
+        $video        = VideoMother::create();
+        $newTitle     = VideoTitleMother::create();
+        $command      = UpdateVideoTitleCommandMother::withIdAndTitle($video->id(), $newTitle);
+        $renamedVideo = DuplicatorMother::with($video, ['title' => $newTitle]);
+
+        $this->shouldSearch($video->id(), $video);
+        $this->shouldSave($renamedVideo);
+
+        $this->handler->__invoke($command);
+    }
+}
diff --git a/tests/Mooc/Videos/Application/Update/VideoRenamerCommandTest.php b/tests/Mooc/Videos/Application/Update/VideoRenamerCommandTest.php
new file mode 100644
index 00000000..ae6b91dd
--- /dev/null
+++ b/tests/Mooc/Videos/Application/Update/VideoRenamerCommandTest.php
@@ -0,0 +1,44 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Application\Update;
+
+use CodelyTv\Mooc\Videos\Application\Find\VideoFinder;
+use CodelyTv\Mooc\Videos\Domain\VideoNotFound;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoIdMother;
+use CodelyTv\Tests\Mooc\Videos\Domain\VideoMother;
+use CodelyTv\Tests\Mooc\Videos\VideosModuleUnitTestCase;
+
+class VideoRenamerCommandTest extends VideosModuleUnitTestCase
+{
+    private VideoFinder|null $finder;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+        $this->finder = new VideoFinder($this->repository());
+    }
+
+    /** @test */
+    public function it_should_throw_an_exception_when_the_video_not_exist()
+    {
+        $this->expectException(VideoNotFound::class);
+
+        $id = VideoIdMother::create();
+
+        $this->shouldSearch($id, null);
+
+        $this->finder->__invoke($id);
+    }
+
+    /** @test */
+    public function it_should_find_an_existing_video(): void
+    {
+        $video = VideoMother::create();
+
+        $this->shouldSearch($video->id(), $video);
+
+        $videoFromRepository = $this->finder->__invoke($video->id());
+
+        $this->assertEquals($videoFromRepository, $video);
+    }
+}
diff --git a/tests/Mooc/Videos/Domain/VideoIdMother.php b/tests/Mooc/Videos/Domain/VideoIdMother.php
new file mode 100644
index 00000000..1f272cce
--- /dev/null
+++ b/tests/Mooc/Videos/Domain/VideoIdMother.php
@@ -0,0 +1,17 @@
+<?php
+
+declare(strict_types=1);
+
+namespace CodelyTv\Tests\Mooc\Videos\Domain;
+
+use CodelyTv\Mooc\Shared\Domain\Courses\CourseId;
+use CodelyTv\Mooc\Videos\Domain\VideoId;
+use CodelyTv\Tests\Shared\Domain\UuidMother;
+
+final class VideoIdMother
+{
+    public static function create(?string $value = null): VideoId
+    {
+        return new VideoId($value ?? UuidMother::create());
+    }
+}
diff --git a/tests/Mooc/Videos/Domain/VideoMother.php b/tests/Mooc/Videos/Domain/VideoMother.php
new file mode 100644
index 00000000..53748357
--- /dev/null
+++ b/tests/Mooc/Videos/Domain/VideoMother.php
@@ -0,0 +1,46 @@
+<?php
+
+declare(strict_types=1);
+
+namespace CodelyTv\Tests\Mooc\Videos\Domain;
+
+use CodelyTv\Mooc\Courses\Application\Create\CreateCourseCommand;
+use CodelyTv\Mooc\Courses\Domain\Course;
+use CodelyTv\Mooc\Shared\Domain\Courses\CourseId;
+use CodelyTv\Mooc\Shared\Domain\Videos\VideoUrl;
+use CodelyTv\Mooc\Videos\Application\Create\CreateVideoCommand;
+use CodelyTv\Mooc\Videos\Domain\Video;
+use CodelyTv\Mooc\Videos\Domain\VideoId;
+use CodelyTv\Mooc\Videos\Domain\VideoTitle;
+use CodelyTv\Mooc\Videos\Domain\VideoType;
+use CodelyTv\Tests\Mooc\Courses\Domain\CourseIdMother;
+
+final class VideoMother
+{
+    public static function create(
+        ?VideoId $id = null,
+        ?VideoType $type = null,
+        ?VideoTitle $title = null,
+        ?VideoUrl $url = null,
+        ?CourseId $courseId = null
+    ): Video {
+        return new Video(
+            $id ?? VideoIdMother::create(),
+            $type ?? VideoTypeMother::create(),
+            $title ?? VideoTitleMother::create(),
+            $url ?? VideoUrlMother::create(),
+            $courseId ?? CourseIdMother::create()
+        );
+    }
+
+    public static function fromRequest(CreateVideoCommand $request): Video
+    {
+        return self::create(
+            VideoIdMother::create($request->id()),
+            VideoTypeMother::create($request->type()),
+            VideoTitleMother::create($request->title()),
+            VideoUrlMother::create($request->url()),
+            CourseIdMother::create($request->courseId()),
+        );
+    }
+}
diff --git a/tests/Mooc/Videos/Domain/VideoTitleMother.php b/tests/Mooc/Videos/Domain/VideoTitleMother.php
new file mode 100644
index 00000000..b7320b89
--- /dev/null
+++ b/tests/Mooc/Videos/Domain/VideoTitleMother.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Domain;
+
+use CodelyTv\Mooc\Videos\Domain\VideoTitle;
+use CodelyTv\Tests\Shared\Domain\WordMother;
+
+class VideoTitleMother
+{
+    public static function create(?string $value = null): VideoTitle
+    {
+        return new VideoTitle($value ?? WordMother::create());
+    }
+}
diff --git a/tests/Mooc/Videos/Domain/VideoTypeMother.php b/tests/Mooc/Videos/Domain/VideoTypeMother.php
new file mode 100644
index 00000000..b8eedf83
--- /dev/null
+++ b/tests/Mooc/Videos/Domain/VideoTypeMother.php
@@ -0,0 +1,13 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Domain;
+
+use CodelyTv\Mooc\Videos\Domain\VideoType;
+
+class VideoTypeMother
+{
+    public static function create(?string $value = null): VideoType
+    {
+        return new VideoType($value ?? VideoType::random()->value());
+    }
+}
diff --git a/tests/Mooc/Videos/Domain/VideoUrlMother.php b/tests/Mooc/Videos/Domain/VideoUrlMother.php
new file mode 100644
index 00000000..409f0303
--- /dev/null
+++ b/tests/Mooc/Videos/Domain/VideoUrlMother.php
@@ -0,0 +1,14 @@
+<?php
+
+namespace CodelyTv\Tests\Mooc\Videos\Domain;
+
+use CodelyTv\Mooc\Shared\Domain\Videos\VideoUrl;
+use CodelyTv\Tests\Shared\Domain\UrlMother;
+
+class VideoUrlMother
+{
+    public static function create(?string $value = null): VideoUrl
+    {
+        return new VideoUrl($value ?? UrlMother::create());
+    }
+}
diff --git a/tests/Mooc/Videos/VideosModuleUnitTestCase.php b/tests/Mooc/Videos/VideosModuleUnitTestCase.php
new file mode 100644
index 00000000..15fa9e56
--- /dev/null
+++ b/tests/Mooc/Videos/VideosModuleUnitTestCase.php
@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace CodelyTv\Tests\Mooc\Videos;
+
+use CodelyTv\Mooc\Courses\Domain\Course;
+use CodelyTv\Mooc\Courses\Domain\CourseRepository;
+use CodelyTv\Mooc\Shared\Domain\Courses\CourseId;
+use CodelyTv\Mooc\Videos\Domain\Video;
+use CodelyTv\Mooc\Videos\Domain\VideoId;
+use CodelyTv\Mooc\Videos\Domain\VideoRepository;
+use CodelyTv\Tests\Shared\Infrastructure\PhpUnit\UnitTestCase;
+use Mockery\MockInterface;
+
+abstract class VideosModuleUnitTestCase extends UnitTestCase
+{
+    private VideoRepository|MockInterface|null $repository;
+
+    protected function shouldSave(Video $video): void
+    {
+        $this->repository()
+            ->shouldReceive('save')
+            ->with($this->similarTo($video))
+            ->once()
+            ->andReturnNull();
+    }
+
+    protected function shouldSearch(VideoId $id, ?Video $video): void
+    {
+        $this->repository()
+            ->shouldReceive('search')
+            ->with($this->similarTo($id))
+            ->once()
+            ->andReturn($video);
+    }
+
+    protected function repository(): VideoRepository|MockInterface
+    {
+        return $this->repository = $this->repository ?? $this->mock(VideoRepository::class);
+    }
+}
diff --git a/tests/Shared/Domain/UrlMother.php b/tests/Shared/Domain/UrlMother.php
new file mode 100644
index 00000000..38b59e7d
--- /dev/null
+++ b/tests/Shared/Domain/UrlMother.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace CodelyTv\Tests\Shared\Domain;
+
+class UrlMother
+{
+    public static function create(): string
+    {
+        return MotherCreator::random()->url;
+    }
+}