From a00546c3e1c169b616b18631a9d2d1afd0d9c46e Mon Sep 17 00:00:00 2001
From: Anton Fedurtsya <anton@fedurtsya.com>
Date: Sun, 6 Oct 2024 22:03:39 +0300
Subject: [PATCH] NFM-2 Add admin request data service

Signed-off-by: Anton Fedurtsya <anton@fedurtsya.com>
---
 services.yaml                                 |   1 +
 .../InvalidRequestParameterException.php      |  14 +++
 src/Admin/Transput/EditRequest.php            |  54 ++++++++
 src/Admin/Transput/EditRequestInterface.php   |  17 +++
 src/Admin/services.yaml                       |  10 ++
 src/Traits/ServiceContainer.php               |  34 ++++++
 .../Traits/ServiceContainerTest.php           |  34 ++++++
 tests/Unit/Admin/Transput/EditRequestTest.php | 115 ++++++++++++++++++
 8 files changed, 279 insertions(+)
 create mode 100644 src/Admin/Exception/InvalidRequestParameterException.php
 create mode 100644 src/Admin/Transput/EditRequest.php
 create mode 100644 src/Admin/Transput/EditRequestInterface.php
 create mode 100644 src/Admin/services.yaml
 create mode 100644 src/Traits/ServiceContainer.php
 create mode 100644 tests/Integration/Traits/ServiceContainerTest.php
 create mode 100644 tests/Unit/Admin/Transput/EditRequestTest.php

diff --git a/services.yaml b/services.yaml
index 496893b..447e807 100644
--- a/services.yaml
+++ b/services.yaml
@@ -1,4 +1,5 @@
 imports:
+  - { resource: src/Admin/services.yaml }
   - { resource: src/DataTypeFactory/services.yaml }
   - { resource: src/Repository/services.yaml }
 
diff --git a/src/Admin/Exception/InvalidRequestParameterException.php b/src/Admin/Exception/InvalidRequestParameterException.php
new file mode 100644
index 0000000..6b2c8cd
--- /dev/null
+++ b/src/Admin/Exception/InvalidRequestParameterException.php
@@ -0,0 +1,14 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace FreshAdvance\NutritionFacts\Admin\Exception;
+
+class InvalidRequestParameterException extends \InvalidArgumentException
+{
+}
\ No newline at end of file
diff --git a/src/Admin/Transput/EditRequest.php b/src/Admin/Transput/EditRequest.php
new file mode 100644
index 0000000..531610e
--- /dev/null
+++ b/src/Admin/Transput/EditRequest.php
@@ -0,0 +1,54 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace FreshAdvance\NutritionFacts\Admin\Transput;
+
+use FreshAdvance\NutritionFacts\Admin\Exception\InvalidRequestParameterException;
+use FreshAdvance\NutritionFacts\DataType\NutritionFactsInterface;
+use FreshAdvance\NutritionFacts\DataTypeFactory\NutritionFactsFactory;
+use FreshAdvance\NutritionFacts\DataTypeFactory\NutritionFactsFactoryInterface;
+use OxidEsales\Eshop\Core\Request;
+
+use function is_string;
+
+class EditRequest implements EditRequestInterface
+{
+    public const REQUEST_KEY_PRODUCT_ID = 'oxid';
+    public const REQUEST_KEY_NUTRITION_FACTS = 'nutritionFacts';
+
+    public function __construct(
+        protected Request $shopRequest,
+        protected NutritionFactsFactoryInterface $nutritionFactsFactory,
+    ) {
+    }
+
+    public function getProductId(): string
+    {
+        /** @var null|array|string $value */
+        $value = $this->shopRequest->getRequestParameter(self::REQUEST_KEY_PRODUCT_ID);
+
+        if (!is_string($value)) {
+            throw new InvalidRequestParameterException(self::REQUEST_KEY_PRODUCT_ID);
+        }
+
+        return $value;
+    }
+
+    public function getNutritionFacts(): NutritionFactsInterface
+    {
+        /** @var null|array|string $value */
+        $value = $this->shopRequest->getRequestParameter(self::REQUEST_KEY_NUTRITION_FACTS);
+
+        if (!is_array($value)) {
+            throw new InvalidRequestParameterException(self::REQUEST_KEY_NUTRITION_FACTS);
+        }
+
+        return $this->nutritionFactsFactory->getFromArray($value);
+    }
+}
\ No newline at end of file
diff --git a/src/Admin/Transput/EditRequestInterface.php b/src/Admin/Transput/EditRequestInterface.php
new file mode 100644
index 0000000..2ce8bf2
--- /dev/null
+++ b/src/Admin/Transput/EditRequestInterface.php
@@ -0,0 +1,17 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+namespace FreshAdvance\NutritionFacts\Admin\Transput;
+
+use FreshAdvance\NutritionFacts\DataType\NutritionFactsInterface;
+
+interface EditRequestInterface
+{
+    public function getProductId(): string;
+
+    public function getNutritionFacts(): NutritionFactsInterface;
+}
\ No newline at end of file
diff --git a/src/Admin/services.yaml b/src/Admin/services.yaml
new file mode 100644
index 0000000..c8b2fb4
--- /dev/null
+++ b/src/Admin/services.yaml
@@ -0,0 +1,10 @@
+services:
+  _defaults:
+    public: false
+    autowire: true
+    bind:
+      OxidEsales\Eshop\Core\Request: '@=service("FreshAdvance\\NutritionFacts\\Core\\Registry").getRequest()'
+
+  FreshAdvance\NutritionFacts\Admin\Transput\EditRequestInterface:
+    class: FreshAdvance\NutritionFacts\Admin\Transput\EditRequest
+    public: true
\ No newline at end of file
diff --git a/src/Traits/ServiceContainer.php b/src/Traits/ServiceContainer.php
new file mode 100644
index 0000000..b8db842
--- /dev/null
+++ b/src/Traits/ServiceContainer.php
@@ -0,0 +1,34 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace FreshAdvance\NutritionFacts\Traits;
+
+use OxidEsales\EshopCommunity\Internal\Container\ContainerFactory;
+use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
+
+/**
+ * @deprecated not required for 7.1.x shop, as there is getService method in Base class
+ */
+trait ServiceContainer
+{
+    /**
+     * @template T
+     * @psalm-param class-string<T> $serviceName
+     *
+     * @return T
+     *
+     * @throws ServiceNotFoundException
+     */
+    protected function getServiceFromContainer(string $serviceName)
+    {
+        return ContainerFactory::getInstance()
+            ->getContainer()
+            ->get($serviceName);
+    }
+}
\ No newline at end of file
diff --git a/tests/Integration/Traits/ServiceContainerTest.php b/tests/Integration/Traits/ServiceContainerTest.php
new file mode 100644
index 0000000..de9abd3
--- /dev/null
+++ b/tests/Integration/Traits/ServiceContainerTest.php
@@ -0,0 +1,34 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace FreshAdvance\NutritionFacts\Tests\Integration\Traits;
+
+use FreshAdvance\NutritionFacts\Traits\ServiceContainer;
+use OxidEsales\Eshop\Core\Registry;
+use PHPUnit\Framework\TestCase;
+
+class ServiceContainerTest extends TestCase
+{
+    public function testGetServiceFromContainer()
+    {
+        $sut = new class {
+            use ServiceContainer;
+
+            public function getTestService(string $service)
+            {
+                return $this->getServiceFromContainer($service);
+            }
+        };
+
+        $this->assertInstanceOf(
+            Registry::class,
+            $sut->getTestService('FreshAdvance\NutritionFacts\Core\Registry')
+        );
+    }
+}
\ No newline at end of file
diff --git a/tests/Unit/Admin/Transput/EditRequestTest.php b/tests/Unit/Admin/Transput/EditRequestTest.php
new file mode 100644
index 0000000..ae3b674
--- /dev/null
+++ b/tests/Unit/Admin/Transput/EditRequestTest.php
@@ -0,0 +1,115 @@
+<?php
+
+/**
+ * Copyright © MB Arbatos Klubas. All rights reserved.
+ * See LICENSE file for license details.
+ */
+
+declare(strict_types=1);
+
+namespace Admin\Transput;
+
+use FreshAdvance\NutritionFacts\Admin\Exception\InvalidRequestParameterException;
+use FreshAdvance\NutritionFacts\Admin\Transput\EditRequest;
+use FreshAdvance\NutritionFacts\DataType\NutritionFactsInterface;
+use FreshAdvance\NutritionFacts\DataTypeFactory\NutritionFactsFactoryInterface;
+use OxidEsales\Eshop\Core\Request;
+use PHPUnit\Framework\TestCase;
+
+class EditRequestTest extends TestCase
+{
+    public function testGetProductId(): void
+    {
+        $exampleProductId = uniqid();
+        $requestStub = $this->createStub(Request::class);
+        $requestStub->method('getRequestParameter')
+            ->with(EditRequest::REQUEST_KEY_PRODUCT_ID)
+            ->willReturn($exampleProductId);
+
+        $sut = new EditRequest(
+            shopRequest: $requestStub,
+            nutritionFactsFactory: $this->createStub(NutritionFactsFactoryInterface::class)
+        );
+
+        $this->assertSame($exampleProductId, $sut->getProductId());
+    }
+
+    /** @dataProvider wrongProductIdValuesDataProvider */
+    public function testGetProductIdWithWrongDataThrowsException(mixed $value): void
+    {
+        $requestMock = $this->createStub(Request::class);
+        $requestMock->method('getRequestParameter')
+            ->with(EditRequest::REQUEST_KEY_PRODUCT_ID)
+            ->willReturn($value);
+
+        $sut = new EditRequest(
+            shopRequest: $requestMock,
+            nutritionFactsFactory: $this->createStub(NutritionFactsFactoryInterface::class)
+        );
+
+        $this->expectException(InvalidRequestParameterException::class);
+        $sut->getProductId();
+    }
+
+    public function wrongProductIdValuesDataProvider(): \Generator
+    {
+        yield 'null' => [
+            'value' => null
+        ];
+
+        yield 'array value' => [
+            'value' => [uniqid()]
+        ];
+    }
+
+    public function testGetNutritionFacts(): void
+    {
+        $exampleArray = [
+            uniqid() => uniqid(),
+            uniqid() => uniqid(),
+        ];
+
+        $sut = new EditRequest(
+            shopRequest: $requestMock = $this->createMock(Request::class),
+            nutritionFactsFactory: $factsFactoryMock = $this->createMock(NutritionFactsFactoryInterface::class)
+        );
+
+        $requestMock->method('getRequestParameter')
+            ->with(EditRequest::REQUEST_KEY_NUTRITION_FACTS)
+            ->willReturn($exampleArray);
+
+        $factsFactoryMock->method('getFromArray')
+            ->with($exampleArray)
+            ->willReturn($expectedFacts = $this->createStub(NutritionFactsInterface::class));
+
+        $this->assertSame($expectedFacts, $sut->getNutritionFacts());
+    }
+
+    /** @dataProvider wrongNutritionFactsValuesDataProvider */
+    public function testGetNutritionFactsWithWrongDataThrowsException(mixed $value): void
+    {
+        $requestMock = $this->createStub(Request::class);
+        $requestMock->method('getRequestParameter')
+            ->with(EditRequest::REQUEST_KEY_NUTRITION_FACTS)
+            ->willReturn($value);
+
+        $sut = new EditRequest(
+            shopRequest: $requestMock,
+            nutritionFactsFactory: $this->createStub(NutritionFactsFactoryInterface::class)
+        );
+
+        $this->expectException(InvalidRequestParameterException::class);
+        $sut->getNutritionFacts();
+    }
+
+    public function wrongNutritionFactsValuesDataProvider(): \Generator
+    {
+        yield 'null' => [
+            'value' => null
+        ];
+
+        yield 'string value' => [
+            'value' => uniqid()
+        ];
+    }
+}
\ No newline at end of file