Fix review

Author: nicosomb

…form/Resources/Discount/DiscountType.php …ources/DiscountType/DiscountTypeList.phpsrc/ApiPlatform/Resources/Discount/DiscountType.php renamed to src/ApiPlatform/Resources/DiscountType/DiscountTypeList.php src/ApiPlatform/Resources/Discount/DiscountType.php renamed to src/ApiPlatform/Resources/DiscountType/DiscountTypeList.php

@@ -18,7 +18,7 @@
  * @license   https://opensource.org/licenses/AFL-3.0 Academic Free License version 3.0
  */
 
-namespace PrestaShop\Module\APIResources\ApiPlatform\Resources\Discount;
+namespace PrestaShop\Module\APIResources\ApiPlatform\Resources\DiscountType;
 
 use ApiPlatform\Metadata\ApiProperty;
 use ApiPlatform\Metadata\ApiResource;
@@ -31,20 +31,20 @@
         new CQRSGetCollection(
             uriTemplate: '/discount-types',
             CQRSQuery: GetDiscountTypes::class,
-            scopes: [],
+            scopes: ['discount_read'],
             CQRSQueryMapping: [],
             ApiResourceMapping: [
                 '[type]' => '[type]',
                 '[localizedNames]' => '[names]',
                 '[localizedDescriptions]' => '[descriptions]',
-                '[isCore]' => '[isCore]',
-                '[active]' => '[active]',
+                '[core]' => '[core]',
+                '[enabled]' => '[enabled]',
             ],
         ),
     ],
     normalizationContext: ['skip_null_values' => false],
 )]
-class DiscountType
+class DiscountTypeList
 {
     #[ApiProperty(identifier: true)]
     public int $discountTypeId;
@@ -57,7 +57,7 @@ class DiscountType
     #[LocalizedValue]
     public array $descriptions;
 
-    public bool $isCore;
+    public bool $core;
 
-    public bool $active;
+    public bool $enabled;
 }

tests/Integration/ApiPlatform/DiscountTypeEndpointTest.php

@@ -0,0 +1,56 @@
+<?php
+/**
+ * Copyright since 2007 PrestaShop SA and Contributors
+ * PrestaShop is an International Registered Trademark & Property of PrestaShop SA
+ *
+ * NOTICE OF LICENSE
+ *
+ * This source file is subject to the Academic Free License version 3.0
+ * that is bundled with this package in the file LICENSE.md.
+ * It is also available through the world-wide-web at this URL:
+ * https://opensource.org/licenses/AFL-3.0
+ * If you did not receive a copy of the license and are unable to
+ * obtain it through the world-wide-web, please send an email
+ * to [email protected] so we can send you a copy immediately.
+ *
+ * @author    PrestaShop SA and Contributors <[email protected]>
+ * @copyright Since 2007 PrestaShop SA and Contributors
+ * @license   https://opensource.org/licenses/AFL-3.0 Academic Free License version 3.0
+ */
+
+namespace PsApiResourcesTest\Integration\ApiPlatform;
+
+class DiscountTypeEndpointTest extends ApiTestCase
+{
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+        self::createApiClient();
+    }
+
+    public static function getProtectedEndpoints(): iterable
+    {
+        yield 'get endpoint' => [
+            'GET',
+            '/discount-types',
+        ];
+    }
+
+    public function testGetDiscountTypes(): void
+    {
+        $discountTypes = $this->getItem('/discount-types', ['discount_read']);
+        $this->assertIsArray($discountTypes);
+
+        $this->assertGreaterThanOrEqual(1, count($discountTypes));
+
+        if (count($discountTypes) > 0) {
+            $firstType = $discountTypes[0];
+            $this->assertArrayHasKey('discountTypeId', $firstType);
+            $this->assertArrayHasKey('type', $firstType);
+            $this->assertArrayHasKey('names', $firstType);
+            $this->assertArrayHasKey('descriptions', $firstType);
+            $this->assertArrayHasKey('core', $firstType);
+            $this->assertArrayHasKey('enabled', $firstType);
+        }
+    }
+}