From 7e573bdb9b24bf06d7ad41afbd823cff13733b9c Mon Sep 17 00:00:00 2001
From: Alejandro Celaya <alejandrocelaya@gmail.com>
Date: Fri, 8 Nov 2024 09:58:02 +0100
Subject: [PATCH] Add tests for RenameApiKeyCOmmand and ApiKeyMeta

---
 .../Command/Api/RenameApiKeyCommandTest.php   | 83 +++++++++++++++++++
 .../Rest/test/ApiKey/Model/ApiKeyMetaTest.php | 35 ++++++++
 2 files changed, 118 insertions(+)
 create mode 100644 module/CLI/test/Command/Api/RenameApiKeyCommandTest.php
 create mode 100644 module/Rest/test/ApiKey/Model/ApiKeyMetaTest.php

diff --git a/module/CLI/test/Command/Api/RenameApiKeyCommandTest.php b/module/CLI/test/Command/Api/RenameApiKeyCommandTest.php
new file mode 100644
index 00000000..41e5689f
--- /dev/null
+++ b/module/CLI/test/Command/Api/RenameApiKeyCommandTest.php
@@ -0,0 +1,83 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ShlinkioTest\Shlink\CLI\Command\Api;
+
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\MockObject\MockObject;
+use PHPUnit\Framework\TestCase;
+use Shlinkio\Shlink\CLI\Command\Api\RenameApiKeyCommand;
+use Shlinkio\Shlink\Core\Model\Renaming;
+use Shlinkio\Shlink\Rest\ApiKey\Model\ApiKeyMeta;
+use Shlinkio\Shlink\Rest\Entity\ApiKey;
+use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
+use ShlinkioTest\Shlink\CLI\Util\CliTestUtils;
+use Symfony\Component\Console\Tester\CommandTester;
+
+class RenameApiKeyCommandTest extends TestCase
+{
+    private CommandTester $commandTester;
+    private MockObject & ApiKeyServiceInterface $apiKeyService;
+
+    protected function setUp(): void
+    {
+        $this->apiKeyService = $this->createMock(ApiKeyServiceInterface::class);
+        $this->commandTester = CliTestUtils::testerForCommand(new RenameApiKeyCommand($this->apiKeyService));
+    }
+
+    #[Test]
+    public function oldNameIsRequestedIfNotProvided(): void
+    {
+        $oldName = 'old name';
+        $newName = 'new name';
+
+        $this->apiKeyService->expects($this->once())->method('listKeys')->willReturn([
+            ApiKey::fromMeta(ApiKeyMeta::fromParams(name: 'foo')),
+            ApiKey::fromMeta(ApiKeyMeta::fromParams(name: $oldName)),
+            ApiKey::fromMeta(ApiKeyMeta::fromParams(name: 'bar')),
+        ]);
+        $this->apiKeyService->expects($this->once())->method('renameApiKey')->with(
+            Renaming::fromNames($oldName, $newName),
+        );
+
+        $this->commandTester->setInputs([$oldName]);
+        $this->commandTester->execute([
+            'newName' => $newName,
+        ]);
+    }
+
+    #[Test]
+    public function newNameIsRequestedIfNotProvided(): void
+    {
+        $oldName = 'old name';
+        $newName = 'new name';
+
+        $this->apiKeyService->expects($this->never())->method('listKeys');
+        $this->apiKeyService->expects($this->once())->method('renameApiKey')->with(
+            Renaming::fromNames($oldName, $newName),
+        );
+
+        $this->commandTester->setInputs([$newName]);
+        $this->commandTester->execute([
+            'oldName' => $oldName,
+        ]);
+    }
+
+    #[Test]
+    public function apiIsRenamedWithProvidedNames(): void
+    {
+        $oldName = 'old name';
+        $newName = 'new name';
+
+        $this->apiKeyService->expects($this->never())->method('listKeys');
+        $this->apiKeyService->expects($this->once())->method('renameApiKey')->with(
+            Renaming::fromNames($oldName, $newName),
+        );
+
+        $this->commandTester->execute([
+            'oldName' => $oldName,
+            'newName' => $newName,
+        ]);
+    }
+}
diff --git a/module/Rest/test/ApiKey/Model/ApiKeyMetaTest.php b/module/Rest/test/ApiKey/Model/ApiKeyMetaTest.php
new file mode 100644
index 00000000..dfd5b9f9
--- /dev/null
+++ b/module/Rest/test/ApiKey/Model/ApiKeyMetaTest.php
@@ -0,0 +1,35 @@
+<?php
+
+declare(strict_types=1);
+
+namespace ShlinkioTest\Shlink\Rest\ApiKey\Model;
+
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Test;
+use PHPUnit\Framework\TestCase;
+use Shlinkio\Shlink\Rest\ApiKey\Model\ApiKeyMeta;
+
+use function sprintf;
+use function substr;
+
+class ApiKeyMetaTest extends TestCase
+{
+    #[Test, DataProvider('provideNames')]
+    public function nameIsInferredWhenNotProvided(string|null $key, string|null $name, callable $getExpectedName): void
+    {
+        $meta = ApiKeyMeta::fromParams(key: $key, name: $name);
+        $expectedName = $getExpectedName($meta);
+
+        self::assertEquals($expectedName, $meta->name);
+    }
+
+    public static function provideNames(): iterable
+    {
+        yield 'name' => [null, 'the name', static fn (ApiKeyMeta $meta) => 'the name'];
+        yield 'key' => ['the key', null, static fn (ApiKeyMeta $meta) => 'the key'];
+        yield 'generated key' => [null, null, static fn (ApiKeyMeta $meta) => sprintf(
+            '%s-****-****-****-************',
+            substr($meta->key, offset: 0, length: 8),
+        )];
+    }
+}