chore(api): list integration test

tleon · Feb 19, 2024 · 81aacfc · 81aacfc
1 parent 34f1b21
commit 81aacfc
Show file tree

Hide file tree

Showing 5 changed files with 70 additions and 18 deletions.
diff --git a/src/ApiPlatform/Resources/ApiClient.php b/src/ApiPlatform/Resources/ApiClient.php
@@ -39,8 +39,6 @@
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSDelete;
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSGet;
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSPartialUpdate;
-use PrestaShopBundle\ApiPlatform\Processor\CommandProcessor;
-use PrestaShopBundle\ApiPlatform\Provider\QueryProvider;
 
 #[ApiResource(
     operations: [
@@ -67,7 +65,6 @@
                     ],
                 ],
             ],
-            provider: QueryProvider::class,
             CQRSQuery: GetApiClientForEditing::class,
             scopes: ['api_client_read']
         ),
@@ -95,20 +92,17 @@
                 ],
             ],
             output: false,
-            provider: QueryProvider::class,
             CQRSQuery: DeleteApiClientCommand::class,
             scopes: ['api_client_write']
         ),
         new CQRSCreate(
             uriTemplate: '/api-client',
-            processor: CommandProcessor::class,
             CQRSCommand: AddApiClientCommand::class,
             scopes: ['api_client_write']
         ),
         new CQRSPartialUpdate(
             uriTemplate: '/api-client/{apiClientId}',
             read: false,
-            processor: CommandProcessor::class,
             CQRSCommand: EditApiClientCommand::class,
             CQRSQuery: GetApiClientForEditing::class,
             scopes: ['api_client_write']

diff --git a/src/ApiPlatform/Resources/CartRule.php b/src/ApiPlatform/Resources/CartRule.php
@@ -31,13 +31,11 @@
 use ApiPlatform\Metadata\ApiResource;
 use PrestaShop\PrestaShop\Core\Domain\CartRule\Command\EditCartRuleCommand;
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSPartialUpdate;
-use PrestaShopBundle\ApiPlatform\Processor\CommandProcessor;
 
 #[ApiResource(
     operations: [
         new CQRSPartialUpdate(
             uriTemplate: '/cart-rule',
-            processor: CommandProcessor::class,
             CQRSCommand: EditCartRuleCommand::class
         ),
     ],

diff --git a/src/ApiPlatform/Resources/FoundProduct.php b/src/ApiPlatform/Resources/FoundProduct.php
@@ -32,7 +32,6 @@
 use ApiPlatform\Metadata\ApiResource;
 use PrestaShop\PrestaShop\Core\Domain\Product\Query\SearchProducts;
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSQueryCollection;
-use PrestaShopBundle\ApiPlatform\Provider\QueryProvider;
 
 #[ApiResource(
     operations: [
@@ -74,7 +73,6 @@
                     ],
                 ],
             ],
-            provider: QueryProvider::class,
             CQRSQuery: SearchProducts::class
         ),
     ],

diff --git a/src/ApiPlatform/Resources/Hook.php b/src/ApiPlatform/Resources/Hook.php
@@ -36,11 +36,9 @@
 use PrestaShop\PrestaShop\Core\Domain\Hook\Query\GetHook;
 use PrestaShop\PrestaShop\Core\Domain\Hook\Query\GetHookStatus;
 use PrestaShopBundle\ApiPlatform\Metadata\CQRSGet;
-use PrestaShopBundle\ApiPlatform\Metadata\CQRSPartialUpdate;
+use PrestaShopBundle\ApiPlatform\Metadata\CQRSUpdate;
 use PrestaShopBundle\ApiPlatform\Metadata\DQBPaginatedList;
-use PrestaShopBundle\ApiPlatform\Processor\CommandProcessor;
 use PrestaShopBundle\ApiPlatform\Provider\QueryListProvider;
-use PrestaShopBundle\ApiPlatform\Provider\QueryProvider;
 
 #[ApiResource(
     operations: [
@@ -68,25 +66,21 @@
                 ],
             ],
             exceptionToStatus: [HookNotFoundException::class => 404],
-            provider: QueryProvider::class,
             CQRSQuery: GetHookStatus::class,
             scopes: ['hook_read']
         ),
-        new CQRSPartialUpdate(
+        new CQRSUpdate(
             uriTemplate: '/hook-status',
-            processor: CommandProcessor::class,
             CQRSCommand: UpdateHookStatusCommand::class,
             scopes: ['hook_write']
         ),
         new CQRSGet(
             uriTemplate: '/hooks/{id}',
             requirements: ['id' => '\d+'],
             exceptionToStatus: [HookNotFoundException::class => 404],
-            provider: QueryProvider::class,
             CQRSQuery: GetHook::class,
             scopes: ['hook_read']
         ),
-        // PR module + inte module + convertir en custom ce qu'il reste en natif
         new DQBPaginatedList(
             uriTemplate: '/hooks',
             provider: QueryListProvider::class,

diff --git a/tests/Integration/ApiPlatform/ListHooksTest.php b/tests/Integration/ApiPlatform/ListHooksTest.php
@@ -0,0 +1,68 @@
+<?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 Open Software License (OSL 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/OSL-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.
+ *
+ * DISCLAIMER
+ *
+ * Do not edit or add to this file if you wish to upgrade PrestaShop to newer
+ * versions in the future. If you wish to customize PrestaShop for your
+ * needs please refer to https://devdocs.prestashop.com/ for more information.
+ *
+ * @author    PrestaShop SA and Contributors <[email protected]>
+ * @copyright Since 2007 PrestaShop SA and Contributors
+ * @license   https://opensource.org/licenses/OSL-3.0 Open Software License (OSL 3.0)
+ */
+
+declare(strict_types=1);
+
+namespace PsApiResourcesTest\Integration\ApiPlatform;
+
+use Tests\Resources\DatabaseDump;
+
+class ListHooksTest extends ApiTestCase
+{
+    public static function setUpBeforeClass(): void
+    {
+        parent::setUpBeforeClass();
+        DatabaseDump::restoreTables(['hook']);
+    }
+
+    public static function tearDownAfterClass(): void
+    {
+        parent::tearDownAfterClass();
+        DatabaseDump::restoreTables(['hook']);
+    }
+
+    public function testListHooks(): void
+    {
+        $bearerToken = $this->getBearerToken([
+            'hook_read',
+            'hook_write',
+        ]);
+
+        $client = static::createClient();
+        $client->
+        $response = static::createClient()->request('GET', '/api/hooks', ['auth_bearer' => $bearerToken]);
+        self::assertResponseStatusCodeSame(200);
+        self::assertCount(50, json_decode($response->getContent())->items);
+
+        $response = static::createClient()->request('GET', '/api/hooks', ['auth_bearer' => $bearerToken, 'query' => ['limit' => 10]]);
+        self::assertResponseStatusCodeSame(200);
+        self::assertCount(10, json_decode($response->getContent())->items);
+
+        static::createClient()->request('GET', '/api/hooks');
+        self::assertResponseStatusCodeSame(401);
+    }
+}