[#.x] - reworking tests - moving dispatch tests to own file

Author: niden

tests/Unit/Domain/Services/User/UserServiceDispatchTest.php

@@ -0,0 +1,83 @@
+<?php
+
+/**
+ * This file is part of the Phalcon API.
+ *
+ * (c) Phalcon Team <[email protected]>
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace Phalcon\Api\Tests\Unit\Domain\Services\User;
+
+use Phalcon\Api\Domain\Components\Cache\Cache;
+use Phalcon\Api\Domain\Components\Container;
+use Phalcon\Api\Domain\Components\DataSource\TransportRepository;
+use Phalcon\Api\Domain\Components\Enums\Http\RoutesEnum;
+use Phalcon\Api\Domain\Components\Env\EnvManager;
+use Phalcon\Api\Tests\AbstractUnitTestCase;
+use Phalcon\Api\Tests\Fixtures\Domain\Migrations\UsersMigration;
+use PHPUnit\Framework\Attributes\BackupGlobals;
+
+#[BackupGlobals(true)]
+final class UserServiceDispatchTest extends AbstractUnitTestCase
+{
+    public function testDispatchGet(): void
+    {
+        /** @var EnvManager $env */
+        $env = $this->container->getShared(Container::ENV);
+        /** @var Cache $cache */
+        $cache = $this->container->getShared(Container::CACHE);
+        /** @var TransportRepository $transport */
+        $transport = $this->container->get(Container::REPOSITORY_TRANSPORT);
+
+        $migration  = new UsersMigration($this->getConnection());
+        $dbUser     = $this->getNewUser($migration);
+        $userId     = $dbUser['usr_id'];
+        $token      = $this->getUserToken($dbUser);
+        $domainUser = $transport->newUser($dbUser);
+
+        /**
+         * Store the token in the cache
+         */
+        $cache->storeTokenInCache($env, $domainUser, $token);
+
+        $time    = $_SERVER['REQUEST_TIME_FLOAT'] ?? time();
+        $_SERVER = [
+            'REQUEST_METHOD'     => 'GET',
+            'REQUEST_TIME_FLOAT' => $time,
+            'HTTP_AUTHORIZATION' => 'Bearer ' . $token,
+            'REQUEST_URI'        => RoutesEnum::userGet->endpoint(),
+        ];
+
+        $_GET = [
+            'id' => $userId,
+        ];
+
+        ob_start();
+        require_once $env->appPath('public/index.php');
+        $response = ob_get_clean();
+
+        $contents = json_decode($response, true);
+
+        restore_error_handler();
+
+        $this->assertArrayHasKey('data', $contents);
+        $this->assertArrayHasKey('errors', $contents);
+
+        $data   = $contents['data'];
+        $errors = $contents['errors'];
+
+        $expected = [];
+        $actual   = $errors;
+        $this->assertSame($expected, $actual);
+
+        $user     = $transport->newUser($dbUser);
+        $expected = $user->toArray();
+        $actual   = $data;
+        $this->assertSame($expected, $actual);
+    }
+}

tests/Unit/Domain/Services/User/UserServiceTest.php renamed to tests/Unit/Domain/Services/User/UserServiceGetTest.php

@@ -15,64 +15,14 @@
 
 use PayloadInterop\DomainStatus;
 use Phalcon\Api\Domain\Components\Container;
-use Phalcon\Api\Domain\Components\DataSource\TransportRepository;
-use Phalcon\Api\Domain\Components\Env\EnvManager;
 use Phalcon\Api\Domain\Services\User\UserGetService;
 use Phalcon\Api\Tests\AbstractUnitTestCase;
 use Phalcon\Api\Tests\Fixtures\Domain\Migrations\UsersMigration;
 use PHPUnit\Framework\Attributes\BackupGlobals;
 
 #[BackupGlobals(true)]
-final class UserServiceTest extends AbstractUnitTestCase
+final class UserServiceGetTest extends AbstractUnitTestCase
 {
-    public function testDispatch(): void
-    {
-        /** @var EnvManager $env */
-        $env = $this->container->getShared(Container::ENV);
-        /** @var TransportRepository $transport */
-        $transport = $this->container->get(Container::REPOSITORY_TRANSPORT);
-
-        $migration = new UsersMigration($this->getConnection());
-        $dbUser    = $this->getNewUser($migration);
-        $userId    = $dbUser['usr_id'];
-        $token     = $this->getUserToken($dbUser);
-
-        $time    = $_SERVER['REQUEST_TIME_FLOAT'] ?? time();
-        $_SERVER = [
-            'REQUEST_METHOD'     => 'GET',
-            'REQUEST_TIME_FLOAT' => $time,
-            'HTTP_AUTHORIZATION' => 'Bearer ' . $token,
-            'REQUEST_URI'        => '/user',
-        ];
-
-        $_GET = [
-            'userId' => $userId,
-        ];
-
-        ob_start();
-        require_once $env->appPath('public/index.php');
-        $response = ob_get_clean();
-
-        $contents = json_decode($response, true);
-
-        restore_error_handler();
-
-        $this->assertArrayHasKey('data', $contents);
-        $this->assertArrayHasKey('errors', $contents);
-
-        $data   = $contents['data'];
-        $errors = $contents['errors'];
-
-        $expected = [];
-        $actual   = $errors;
-        $this->assertSame($expected, $actual);
-
-        $user     = $transport->newUser($dbUser);
-        $expected = $user->toArray();
-        $actual   = $data;
-        $this->assertSame($expected, $actual);
-    }
-
     public function testServiceEmptyUserId(): void
     {
         /** @var UserGetService $service */
@@ -103,7 +53,7 @@ public function testServiceWithUserId(): void
 
         $payload = $service->__invoke(
             [
-                'userId' => $userId,
+                'id' => $userId,
             ]
         );
 
@@ -194,7 +144,7 @@ public function testServiceWrongUserId(): void
 
         $payload = $service->__invoke(
             [
-                'userId' => 999999,
+                'id' => 999999,
             ]
         );