From 286c24f8c02efe660c5eb382bf582435db5197fb Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandro@alejandrocelaya.com>
Date: Sat, 15 Jul 2017 12:09:25 +0200
Subject: [PATCH] Improved TagServiceTest

---
 .../Core/test/Service/Tag/TagServiceTest.php  | 43 +++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/module/Core/test/Service/Tag/TagServiceTest.php b/module/Core/test/Service/Tag/TagServiceTest.php
index 206964a8..fcae18d0 100644
--- a/module/Core/test/Service/Tag/TagServiceTest.php
+++ b/module/Core/test/Service/Tag/TagServiceTest.php
@@ -7,6 +7,7 @@ use Prophecy\Argument;
 use Prophecy\Prophecy\MethodProphecy;
 use Prophecy\Prophecy\ObjectProphecy;
 use Shlinkio\Shlink\Core\Entity\Tag;
+use Shlinkio\Shlink\Core\Exception\EntityDoesNotExistException;
 use Shlinkio\Shlink\Core\Repository\TagRepository;
 use Shlinkio\Shlink\Core\Service\Tag\TagService;
 use PHPUnit\Framework\TestCase;
@@ -88,4 +89,46 @@ class TagServiceTest extends TestCase
         $persist->shouldHaveBeenCalledTimes(2);
         $flush->shouldHaveBeenCalled();
     }
+
+    /**
+     * @test
+     */
+    public function renameInvalidTagThrowsException()
+    {
+        $repo = $this->prophesize(TagRepository::class);
+        /** @var MethodProphecy $find */
+        $find = $repo->findOneBy(Argument::cetera())->willReturn(null);
+        /** @var MethodProphecy $getRepo */
+        $getRepo = $this->em->getRepository(Tag::class)->willReturn($repo->reveal());
+
+        $find->shouldBeCalled();
+        $getRepo->shouldBeCalled();
+        $this->expectException(EntityDoesNotExistException::class);
+
+        $this->service->renameTag('foo', 'bar');
+    }
+
+    /**
+     * @test
+     */
+    public function renameValidTagChangesItsName()
+    {
+        $expected = new Tag();
+
+        $repo = $this->prophesize(TagRepository::class);
+        /** @var MethodProphecy $find */
+        $find = $repo->findOneBy(Argument::cetera())->willReturn($expected);
+        /** @var MethodProphecy $getRepo */
+        $getRepo = $this->em->getRepository(Tag::class)->willReturn($repo->reveal());
+        /** @var MethodProphecy $flush */
+        $flush = $this->em->flush($expected)->willReturn(null);
+
+        $tag = $this->service->renameTag('foo', 'bar');
+
+        $this->assertSame($expected, $tag);
+        $this->assertEquals('bar', $tag->getName());
+        $find->shouldHaveBeenCalled();
+        $getRepo->shouldHaveBeenCalled();
+        $flush->shouldHaveBeenCalled();
+    }
 }