Skip to content

Commit 0569af2

Browse files
author
DKravtsov
committed
Added functional tests for User endpoints (part 1), updated composer dependencies.
1 parent 34b3744 commit 0569af2

File tree

22 files changed

+2139
-960
lines changed

22 files changed

+2139
-960
lines changed

‎composer.lock‎

Lines changed: 359 additions & 346 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎readme.md‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -290,6 +290,8 @@ Notes: Please see more commands in Makefile
290290
* [messenger](https://symfony.com/doc/current/messenger.html)
291291
* [composer-bin-plugin](https://github.com/bamarni/composer-bin-plugin)
292292
* [composer-normalize](https://github.com/ergebnis/composer-normalize)
293+
* [composer-unused](https://packagist.org/packages/icanhazstring/composer-unused)
294+
* [composer-require-checker](https://packagist.org/packages/maglnet/composer-require-checker)
293295
* [requirements-checker](https://github.com/symfony/requirements-checker)
294296
* [security-advisories](https://github.com/Roave/SecurityAdvisories)
295297
* [jwt-authentication-bundle](https://packagist.org/packages/lexik/jwt-authentication-bundle)

‎src/Tool/Domain/Service/Interfaces/LocalizationServiceInterface.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ interface LocalizationServiceInterface
1717
{
1818
public const DEFAULT_LANGUAGE = EnumLanguageType::LANGUAGE_EN;
1919
public const DEFAULT_LOCALE = EnumLocaleType::LOCALE_EN;
20-
public const DEFAULT_TIMEZONE = 'Europe/Kiev';
20+
public const DEFAULT_TIMEZONE = 'Europe/Kyiv';
2121

2222
/**
2323
* @return array<int, string>

‎tests/Functional/ApiKey/Transport/Controller/Api/v1/ApiKeyControllerTest.php‎

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class ApiKeyControllerTest extends WebTestCase
2525
private string $baseUrl = self::API_URL_PREFIX . '/v1/api_key';
2626

2727
/**
28-
* @testdox Test that `GET /v1/api_key` request returns `401` for non-logged user.
28+
* @testdox Test that `GET /api/v1/api_key` request returns `401` for non-logged user.
2929
*
3030
* @throws Throwable
3131
*/
@@ -59,7 +59,7 @@ public function testThatAllActionsForbiddenForNonRootUser(string $method, string
5959
}
6060

6161
/**
62-
* @testdox Test that `GET /v1/api_key/count` for the Root user returns success response.
62+
* @testdox Test that `GET /api/v1/api_key/count` for the root user returns success response.
6363
*
6464
* @throws Throwable
6565
*/
@@ -78,7 +78,7 @@ public function testThatCountActionForRootUserReturnsSuccessResponse(): void
7878
}
7979

8080
/**
81-
* @testdox Test that `GET /v1/api_key` returns `$responseCode` with login: `$login`, password: `$password`.
81+
* @testdox Test that `GET /api/v1/api_key` returns `$responseCode` with login: `$login`, password: `$password`.
8282
*
8383
* @dataProvider dataProviderTestThatFindActionWorksAsExpected
8484
*
@@ -104,7 +104,7 @@ public function testThatFindActionWorksAsExpected(string $login, string $passwor
104104
}
105105

106106
/**
107-
* @testdox Test that `POST /v1/api_key` (create api-key) for the Root user returns success response.
107+
* @testdox Test that `POST /api/v1/api_key` (create api-key) for the root user returns success response.
108108
*
109109
* @throws Throwable
110110
*/
@@ -129,7 +129,7 @@ public function testThatCreateActionForRootUserReturnsSuccessResponse(): void
129129
}
130130

131131
/**
132-
* @testdox Test that `GET /v1/api_key/{id}` for the Root user returns success response.
132+
* @testdox Test that `GET /api/v1/api_key/{id}` for the root user returns success response.
133133
*
134134
* @depends testThatCreateActionForRootUserReturnsSuccessResponse
135135
*
@@ -138,6 +138,7 @@ public function testThatCreateActionForRootUserReturnsSuccessResponse(): void
138138
public function testThatFindOneActionForRootUserReturnsSuccessResponse(): void
139139
{
140140
$client = $this->getTestClient('john-root', 'password-root');
141+
141142
$resource = static::getContainer()->get(ApiKeyResource::class);
142143
static::assertInstanceOf(ApiKeyResource::class, $resource);
143144
$apiKeyEntity = $resource->findOneBy([
@@ -157,7 +158,7 @@ public function testThatFindOneActionForRootUserReturnsSuccessResponse(): void
157158
}
158159

159160
/**
160-
* @testdox Test that `PUT /v1/api_key/{id}` for the Root user returns success response.
161+
* @testdox Test that `PUT /api/v1/api_key/{id}` for the root user returns success response.
161162
*
162163
* @depends testThatCreateActionForRootUserReturnsSuccessResponse
163164
*
@@ -166,6 +167,7 @@ public function testThatFindOneActionForRootUserReturnsSuccessResponse(): void
166167
public function testThatUpdateActionForRootUserReturnsSuccessResponse(): void
167168
{
168169
$client = $this->getTestClient('john-root', 'password-root');
170+
169171
$resource = static::getContainer()->get(ApiKeyResource::class);
170172
static::assertInstanceOf(ApiKeyResource::class, $resource);
171173
$apiKeyEntity = $resource->findOneBy([
@@ -202,7 +204,7 @@ public function testThatUpdateActionForRootUserReturnsSuccessResponse(): void
202204
}
203205

204206
/**
205-
* @testdox Test that `PATCH /v1/api_key/{id}` for the Root user returns success response.
207+
* @testdox Test that `PATCH /api/v1/api_key/{id}` for the root user returns success response.
206208
*
207209
* @depends testThatCreateActionForRootUserReturnsSuccessResponse
208210
*
@@ -211,6 +213,7 @@ public function testThatUpdateActionForRootUserReturnsSuccessResponse(): void
211213
public function testThatPatchActionForRootUserReturnsSuccessResponse(): void
212214
{
213215
$client = $this->getTestClient('john-root', 'password-root');
216+
214217
$resource = static::getContainer()->get(ApiKeyResource::class);
215218
static::assertInstanceOf(ApiKeyResource::class, $resource);
216219
$apiKeyEntity = $resource->findOneBy([
@@ -246,7 +249,7 @@ public function testThatPatchActionForRootUserReturnsSuccessResponse(): void
246249
}
247250

248251
/**
249-
* @testdox Test that `DELETE /v1/api_key/{id}` for the Root user returns success response.
252+
* @testdox Test that `DELETE /api/v1/api_key/{id}` for the root user returns success response.
250253
*
251254
* @depends testThatPatchActionForRootUserReturnsSuccessResponse
252255
*
@@ -273,7 +276,7 @@ public function testThatDeleteActionForRootUserReturnsSuccessResponse(): void
273276
}
274277

275278
/**
276-
* @testdox Test that `GET /v1/api_key/ids` for the Root user returns success response.
279+
* @testdox Test that `GET /api/v1/api_key/ids` for the root user returns success response.
277280
*
278281
* @throws Throwable
279282
*/

‎tests/Functional/User/Transport/Controller/Api/v1/Auth/GetTokenControllerTest.php‎

Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace App\Tests\Functional\User\Transport\Controller\Api\v1\Auth;
6+
7+
use App\General\Domain\Utils\JSON;
8+
use App\General\Transport\Utils\Tests\WebTestCase;
9+
use Generator;
10+
use Symfony\Component\HttpFoundation\Response;
11+
use Throwable;
12+
13+
/**
14+
* Class GetTokenControllerTest
15+
*
16+
* @package App\Tests
17+
*/
18+
class GetTokenControllerTest extends WebTestCase
19+
{
20+
private string $baseUrl = self::API_URL_PREFIX . '/v1/auth';
21+
22+
/**
23+
* @testdox Test that `POST /api/v1/auth/get_token` for the `$username` returns success response.
24+
*
25+
* @dataProvider dataProviderUsers
26+
*
27+
* @throws Throwable
28+
*/
29+
public function testThatGetTokenActionForUsersReturnsSuccessResponse(string $username, string $password): void
30+
{
31+
$client = $this->getTestClient();
32+
33+
$requestData = [
34+
'username' => $username,
35+
'password' => $password,
36+
];
37+
$client->request(method: 'POST', uri: $this->baseUrl . '/get_token', content: JSON::encode($requestData));
38+
$response = $client->getResponse();
39+
$content = $response->getContent();
40+
self::assertNotFalse($content);
41+
self::assertSame(Response::HTTP_OK, $response->getStatusCode(), "Response:\n" . $response);
42+
$responseData = JSON::decode($content, true);
43+
self::assertArrayHasKey('token', $responseData);
44+
self::assertIsString($responseData['token']);
45+
}
46+
47+
/**
48+
* @testdox Test that `POST /api/v1/auth/get_token` with wrong password returns error response.
49+
*
50+
* @throws Throwable
51+
*/
52+
public function testThatGetTokenActionForUserWithWrongPasswordReturnsErrorResponse(): void
53+
{
54+
$client = $this->getTestClient();
55+
56+
$requestData = [
57+
'username' => 'john',
58+
'password' => 'wrong-password',
59+
];
60+
$client->request(method: 'POST', uri: $this->baseUrl . '/get_token', content: JSON::encode($requestData));
61+
$response = $client->getResponse();
62+
$content = $response->getContent();
63+
self::assertNotFalse($content);
64+
self::assertSame(Response::HTTP_UNAUTHORIZED, $response->getStatusCode(), "Response:\n" . $response);
65+
$responseData = JSON::decode($content, true);
66+
self::assertArrayHasKey('code', $responseData);
67+
self::assertArrayHasKey('message', $responseData);
68+
self::assertEquals(Response::HTTP_UNAUTHORIZED, $responseData['code']);
69+
self::assertEquals('Invalid credentials.', $responseData['message']);
70+
}
71+
72+
/**
73+
* @return Generator<array{0: string, 1: string}>
74+
*/
75+
public function dataProviderUsers(): Generator
76+
{
77+
yield ['john', 'password'];
78+
yield ['john-logged', 'password-logged'];
79+
yield ['john-api', 'password-api'];
80+
yield ['john-user', 'password-user'];
81+
yield ['john-admin', 'password-admin'];
82+
yield ['john-root', 'password-root'];
83+
}
84+
}

‎tests/Functional/User/Transport/Controller/Api/v1/Profile/GroupsControllerTest.php‎

Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,65 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace App\Tests\Functional\User\Transport\Controller\Api\v1\Profile;
6+
7+
use App\General\Domain\Utils\JSON;
8+
use App\General\Transport\Utils\Tests\WebTestCase;
9+
use App\Role\Domain\Entity\Role;
10+
use App\User\Infrastructure\DataFixtures\ORM\LoadUserGroupData;
11+
use Symfony\Component\HttpFoundation\Response;
12+
use Throwable;
13+
14+
/**
15+
* Class GroupsControllerTest
16+
*
17+
* @package App\Tests
18+
*/
19+
class GroupsControllerTest extends WebTestCase
20+
{
21+
private string $baseUrl = self::API_URL_PREFIX . '/v1/profile';
22+
23+
/**
24+
* @testdox Test that `GET /api/v1/profile/groups` for the `john-logged` user returns success response.
25+
*
26+
* @throws Throwable
27+
*/
28+
public function testThatGetGroupsActionForUserReturnsSuccessResponse(): void
29+
{
30+
$client = $this->getTestClient('john-logged', 'password-logged');
31+
32+
$client->request(method: 'GET', uri: $this->baseUrl . '/groups');
33+
$response = $client->getResponse();
34+
$content = $response->getContent();
35+
self::assertNotFalse($content);
36+
self::assertSame(Response::HTTP_OK, $response->getStatusCode(), "Response:\n" . $response);
37+
$responseData = JSON::decode($content, true);
38+
self::assertIsArray($responseData);
39+
self::assertCount(1, $responseData);
40+
self::assertArrayHasKey('id', $responseData[0]);
41+
self::assertArrayHasKey('role', $responseData[0]);
42+
self::assertArrayHasKey('name', $responseData[0]);
43+
self::assertEquals($responseData[0]['id'], LoadUserGroupData::$uuids['Role-logged']);
44+
self::assertIsArray($responseData[0]['role']);
45+
self::assertArrayHasKey('id', $responseData[0]['role']);
46+
self::assertEquals(Role::ROLE_LOGGED, $responseData[0]['role']['id']);
47+
self::assertIsString($responseData[0]['name']);
48+
}
49+
50+
/**
51+
* @testdox Test that `GET /api/v1/profile/groups` for non-logged user returns error response.
52+
*
53+
* @throws Throwable
54+
*/
55+
public function testThatGetGroupsActionForNonLoggedUserReturnsErrorResponse(): void
56+
{
57+
$client = $this->getTestClient();
58+
59+
$client->request(method: 'GET', uri: $this->baseUrl . '/groups');
60+
$response = $client->getResponse();
61+
$content = $response->getContent();
62+
self::assertNotFalse($content);
63+
self::assertSame(Response::HTTP_UNAUTHORIZED, $response->getStatusCode(), "Response:\n" . $response);
64+
}
65+
}

‎tests/Functional/User/Transport/Controller/Api/v1/Profile/IndexControllerTest.php‎

Lines changed: 84 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,84 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace App\Tests\Functional\User\Transport\Controller\Api\v1\Profile;
6+
7+
use App\General\Domain\Utils\JSON;
8+
use App\General\Transport\Utils\Tests\WebTestCase;
9+
use App\Role\Application\Security\RolesService;
10+
use App\User\Application\Resource\UserResource;
11+
use App\User\Domain\Entity\User;
12+
use Symfony\Component\HttpFoundation\Response;
13+
use Throwable;
14+
15+
/**
16+
* Class IndexControllerTest
17+
*
18+
* @package App\Tests
19+
*/
20+
class IndexControllerTest extends WebTestCase
21+
{
22+
private string $baseUrl = self::API_URL_PREFIX . '/v1/profile';
23+
24+
/**
25+
* @testdox Test that `GET /api/v1/profile` for the `john-root` user returns success response.
26+
*
27+
* @throws Throwable
28+
*/
29+
public function testThatGetUserProfileActionForRootUserReturnsSuccessResponse(): void
30+
{
31+
$client = $this->getTestClient('john-root', 'password-root');
32+
$roleService = static::getContainer()->get(RolesService::class);
33+
static::assertInstanceOf(RolesService::class, $roleService);
34+
$resource = static::getContainer()->get(UserResource::class);
35+
static::assertInstanceOf(UserResource::class, $resource);
36+
$userEntity = $resource->findOneBy([
37+
'username' => 'john-root',
38+
]);
39+
self::assertInstanceOf(User::class, $userEntity);
40+
41+
$client->request(method: 'GET', uri: $this->baseUrl);
42+
$response = $client->getResponse();
43+
$content = $response->getContent();
44+
self::assertNotFalse($content);
45+
self::assertSame(Response::HTTP_OK, $response->getStatusCode(), "Response:\n" . $response);
46+
$responseData = JSON::decode($content, true);
47+
self::assertIsArray($responseData);
48+
self::assertArrayHasKey('id', $responseData);
49+
self::assertEquals($userEntity->getId(), $responseData['id']);
50+
self::assertArrayHasKey('username', $responseData);
51+
self::assertEquals($userEntity->getUsername(), $responseData['username']);
52+
self::assertArrayHasKey('firstName', $responseData);
53+
self::assertEquals($userEntity->getFirstName(), $responseData['firstName']);
54+
self::assertArrayHasKey('lastName', $responseData);
55+
self::assertEquals($userEntity->getLastName(), $responseData['lastName']);
56+
self::assertArrayHasKey('email', $responseData);
57+
self::assertEquals($userEntity->getEmail(), $responseData['email']);
58+
self::assertArrayHasKey('language', $responseData);
59+
self::assertEquals($userEntity->getLanguage(), $responseData['language']);
60+
self::assertArrayHasKey('locale', $responseData);
61+
self::assertEquals($userEntity->getLocale(), $responseData['locale']);
62+
self::assertArrayHasKey('timezone', $responseData);
63+
self::assertEquals($userEntity->getTimezone(), $responseData['timezone']);
64+
self::assertArrayHasKey('roles', $responseData);
65+
self::assertIsArray($responseData['roles']);
66+
self::assertCount(count($roleService->getInheritedRoles($userEntity->getRoles())), $responseData['roles']);
67+
}
68+
69+
/**
70+
* @testdox Test that `GET /api/v1/profile` for non-logged user returns error response.
71+
*
72+
* @throws Throwable
73+
*/
74+
public function testThatGetGetUserProfileActionForNonLoggedUserReturnsErrorResponse(): void
75+
{
76+
$client = $this->getTestClient();
77+
78+
$client->request(method: 'GET', uri: $this->baseUrl);
79+
$response = $client->getResponse();
80+
$content = $response->getContent();
81+
self::assertNotFalse($content);
82+
self::assertSame(Response::HTTP_UNAUTHORIZED, $response->getStatusCode(), "Response:\n" . $response);
83+
}
84+
}

0 commit comments

Comments
 (0)