OXDEV-8489: Create new ModuleDeactivationBlockedException

src/Module/Exception/ModuleDeactivationBlockedException.php

@@ -0,0 +1,22 @@
+<?php
+
+/**
+ * Copyright © OXID eSales AG. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace OxidEsales\GraphQL\ConfigurationAccess\Module\Exception;
+
+use OxidEsales\GraphQL\Base\Exception\NotFound;
+
+final class ModuleDeactivationBlockedException extends NotFound
+{
+    private const EXCEPTION_MESSAGE = 'Module "%s" is in the blocklist and cannot be deactivated.';
+
+    public function __construct(string $moduleId)
+    {
+        parent::__construct(sprintf(self::EXCEPTION_MESSAGE, $moduleId));
+    }
+}

src/Module/Exception/ModuleDeactivationException.php

@@ -14,13 +14,9 @@
 final class ModuleDeactivationException extends Error
 {
     private const EXCEPTION_MESSAGE = "An error occurred while deactivating the module.";
-    public const BLOCKED_MODULE_MESSAGE = 'Module "%s" is in the blocklist and cannot be deactivated.';
 
-    public function __construct(string $message = null)
+    public function __construct()
     {
-        if (empty($message)) {
-            $message = self::EXCEPTION_MESSAGE;
-        }
-        parent::__construct($message);
+        parent::__construct(self::EXCEPTION_MESSAGE);
     }
 }

src/Module/Service/ModuleActivationService.php

@@ -12,6 +12,7 @@
 use OxidEsales\EshopCommunity\Internal\Framework\Module\Setup\Bridge\ModuleActivationBridgeInterface;
 use OxidEsales\EshopCommunity\Internal\Transition\Utility\ContextInterface;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleActivationException;
+use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationBlockedException;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationException;
 
 class ModuleActivationService implements ModuleActivationServiceInterface
@@ -45,9 +46,7 @@ public function activateModule(string $moduleId): bool
     public function deactivateModule(string $moduleId): bool
     {
         if ($this->moduleBlocklistService->isModuleBlocked($moduleId)) {
-            throw new ModuleDeactivationException(
-                sprintf(ModuleDeactivationException::BLOCKED_MODULE_MESSAGE, $moduleId)
-            );
+            throw new ModuleDeactivationBlockedException($moduleId);
         }
 
         $shopId = $this->context->getCurrentShopId();

src/Module/Service/ModuleActivationServiceInterface.php

@@ -9,6 +9,7 @@
 
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleActivationException;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationException;
+use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationBlockedException;
 
 interface ModuleActivationServiceInterface
 {
@@ -19,6 +20,7 @@ public function activateModule(string $moduleId): bool;
 
     /**
      * @throws ModuleDeactivationException
+     * @throws ModuleDeactivationBlockedException
      */
     public function deactivateModule(string $moduleId): bool;
 }

tests/Unit/Module/Exception/ModuleBlockListExceptionTest.php

@@ -22,6 +22,6 @@ public function testException(): void
         $exception = new ModuleBlockListException();
 
         $this->assertInstanceOf(ModuleBlockListException::class, $exception);
-        $this->assertSame(ModuleBlockListException::EXCEPTION_MESSAGE, $exception->getMessage());
+        $this->assertSame('Failed to load module blocklist from YAML file.', $exception->getMessage());
     }
 }

tests/Unit/Module/Exception/ModuleDeactivationBlockedExceptionTest.php

@@ -0,0 +1,31 @@
+<?php
+
+/**
+ * Copyright © OXID eSales AG. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace OxidEsales\GraphQL\ConfigurationAccess\Tests\Unit\Module\Exception;
+
+use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationBlockedException;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @covers \OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationBlockedException
+ */
+final class ModuleDeactivationBlockedExceptionTest extends TestCase
+{
+    public function testException(): void
+    {
+        $moduleId = uniqid();
+        $exception = new ModuleDeactivationBlockedException($moduleId);
+
+        $this->assertInstanceOf(ModuleDeactivationBlockedException::class, $exception);
+        $this->assertSame(
+            sprintf('Module "%s" is in the blocklist and cannot be deactivated.', $moduleId),
+            $exception->getMessage()
+        );
+    }
+}

tests/Unit/Module/Service/ModuleActivationsServiceTest.php

@@ -12,6 +12,7 @@
 use OxidEsales\EshopCommunity\Internal\Transition\Utility\ContextInterface;
 use OxidEsales\EshopCommunity\Internal\Framework\Module\Setup\Bridge\ModuleActivationBridgeInterface;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleActivationException;
+use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationBlockedException;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Exception\ModuleDeactivationException;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Service\ModuleActivationService;
 use OxidEsales\GraphQL\ConfigurationAccess\Module\Service\ModuleBlocklistServiceInterface;
@@ -73,6 +74,21 @@ public function testModuleActivationAndDeactivationExceptions(
         ($method === 'activate') ? $sut->activateModule($moduleId) : $sut->deactivateModule($moduleId);
     }
 
+    public function testModuleDeactivationBlockedException()
+    {
+        $moduleBlockListServiceMock = $this->createMock(ModuleBlocklistServiceInterface::class);
+        $moduleBlockListServiceMock
+            ->method('isModuleBlocked')
+            ->willReturn(true);
+
+        $sut = $this->getSut(
+            moduleBlocklistService: $moduleBlockListServiceMock
+        );
+
+        $this->expectException(ModuleDeactivationBlockedException::class);
+        $sut->deactivateModule(uniqid());
+    }
+
     public static function activationDataProvider(): \Generator
     {
         yield 'test activate module' => [