Skip to content

Commit 4052d09

Browse files
committed
Own namespace for attributes
1 parent bcff810 commit 4052d09

File tree

2 files changed

+13
-13
lines changed

2 files changed

+13
-13
lines changed

src/Subscription/Controller/SubscriberAttributeDefinitionController.php

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
use Symfony\Component\HttpFoundation\Response;
2121
use Symfony\Component\Routing\Attribute\Route;
2222

23-
#[Route('/subscribers/attributes', name: 'subscriber_attribute_definition_')]
23+
#[Route('/attributes', name: 'subscriber_attribute_definition_')]
2424
class SubscriberAttributeDefinitionController extends BaseController
2525
{
2626
private AttributeDefinitionManager $definitionManager;
@@ -43,7 +43,7 @@ public function __construct(
4343

4444
#[Route('', name: 'create', methods: ['POST'])]
4545
#[OA\Post(
46-
path: '/api/v2/subscribers/attributes',
46+
path: '/api/v2/attributes',
4747
description: '🚧 **Status: Beta** – This method is under development. Avoid using in production. ' .
4848
'Returns created subscriber attribute definition.',
4949
summary: 'Create a subscriber attribute definition.',
@@ -96,7 +96,7 @@ public function create(Request $request): JsonResponse
9696

9797
#[Route('/{definitionId}', name: 'update', requirements: ['definitionId' => '\d+'], methods: ['PUT'])]
9898
#[OA\Put(
99-
path: '/api/v2/subscribers/attributes/{definitionId}',
99+
path: '/api/v2/attributes/{definitionId}',
100100
description: '🚧 **Status: Beta** – This method is under development. Avoid using in production. ' .
101101
'Returns updated subscriber attribute definition.',
102102
summary: 'Update a subscriber attribute definition.',
@@ -164,7 +164,7 @@ public function update(
164164

165165
#[Route('/{definitionId}', name: 'delete', requirements: ['definitionId' => '\d+'], methods: ['DELETE'])]
166166
#[OA\Delete(
167-
path: '/api/v2/subscribers/attributes/{definitionId}',
167+
path: '/api/v2/attributes/{definitionId}',
168168
description: '🚧 **Status: Beta** – This method is under development. Avoid using in production. ' .
169169
'Deletes a single subscriber attribute definition.',
170170
summary: 'Deletes an attribute definition.',
@@ -219,7 +219,7 @@ public function delete(
219219

220220
#[Route('', name: 'get_list', methods: ['GET'])]
221221
#[OA\Get(
222-
path: '/api/v2/subscribers/attributes',
222+
path: '/api/v2/attributes',
223223
description: '🚧 **Status: Beta** – This method is under development. Avoid using in production. ' .
224224
'Returns a JSON list of all subscriber attribute definitions.',
225225
summary: 'Gets a list of all subscriber attribute definitions.',
@@ -286,7 +286,7 @@ public function getPaginated(Request $request): JsonResponse
286286

287287
#[Route('/{definitionId}', name: 'get_one', requirements: ['definitionId' => '\d+'], methods: ['GET'])]
288288
#[OA\Get(
289-
path: '/api/v2/subscribers/attributes/{definitionId}',
289+
path: '/api/v2/attributes/{definitionId}',
290290
description: '🚧 **Status: Beta** – This method is under development. Avoid using in production. ' .
291291
'Returns a single attribute with specified ID.',
292292
summary: 'Gets attribute with specified ID.',

tests/Integration/Subscription/Controller/SubscriberAttributeDefinitionControllerTest.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,21 +23,21 @@ public function testControllerIsAvailableViaContainer()
2323

2424
public function testGetAttributesWithoutSessionKeyReturnsForbidden()
2525
{
26-
self::getClient()->request('GET', '/api/v2/subscribers/attributes');
26+
self::getClient()->request('GET', '/api/v2/attributes');
2727
$this->assertHttpForbidden();
2828
}
2929

3030
public function testGetAttributesWithSessionKeyReturnsOk()
3131
{
3232
$this->loadFixtures([AdministratorFixture::class, AdministratorTokenFixture::class]);
33-
$this->authenticatedJsonRequest('GET', '/api/v2/subscribers/attributes');
33+
$this->authenticatedJsonRequest('GET', '/api/v2/attributes');
3434
$this->assertHttpOkay();
3535
}
3636

3737
public function testGetAttributeWithInvalidIdReturnsNotFound()
3838
{
3939
$this->loadFixtures([AdministratorFixture::class, AdministratorTokenFixture::class]);
40-
$this->authenticatedJsonRequest('GET', '/api/v2/subscribers/attributes/999');
40+
$this->authenticatedJsonRequest('GET', '/api/v2/attributes/999');
4141
$this->assertHttpNotFound();
4242
}
4343

@@ -53,7 +53,7 @@ public function testCreateAttributeDefinition()
5353
'required' => true,
5454
]);
5555

56-
$this->authenticatedJsonRequest('POST', '/api/v2/subscribers/attributes', [], [], [], $payload);
56+
$this->authenticatedJsonRequest('POST', '/api/v2/attributes', [], [], [], $payload);
5757

5858
$this->assertHttpCreated();
5959

@@ -77,7 +77,7 @@ public function testUpdateAttributeDefinition()
7777
'required' => false,
7878
]);
7979

80-
$this->authenticatedJsonRequest('PUT', '/api/v2/subscribers/attributes/1', [], [], [], $payload);
80+
$this->authenticatedJsonRequest('PUT', '/api/v2/attributes/1', [], [], [], $payload);
8181
$this->assertHttpOkay();
8282
$response = $this->getDecodedJsonResponseContent();
8383
self::assertSame('Updated Country', $response['name']);
@@ -91,7 +91,7 @@ public function testDeleteAttributeDefinition()
9191
SubscriberAttributeDefinitionFixture::class,
9292
]);
9393

94-
$this->authenticatedJsonRequest('DELETE', '/api/v2/subscribers/attributes/1');
94+
$this->authenticatedJsonRequest('DELETE', '/api/v2/attributes/1');
9595
$this->assertHttpNoContent();
9696

9797
$repo = self::getContainer()->get(SubscriberAttributeDefinitionRepository::class);
@@ -108,7 +108,7 @@ public function testCreateAttributeDefinitionMissingNameReturnsValidationError()
108108
'required' => false
109109
]);
110110

111-
$this->authenticatedJsonRequest('POST', '/api/v2/subscribers/attributes', [], [], [], $payload);
111+
$this->authenticatedJsonRequest('POST', '/api/v2/attributes', [], [], [], $payload);
112112
$this->assertHttpUnprocessableEntity();
113113
}
114114
}

0 commit comments

Comments
 (0)