Skip to content

Commit c832eb3

Browse files
committed
test(integration): align FieldDefinitionApiController integration test with policy enums
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent a0489db commit c832eb3

1 file changed

Lines changed: 7 additions & 10 deletions

File tree

tests/php/ControllerIntegration/Controller/FieldDefinitionApiControllerTest.php

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,9 @@
1212
use OCA\ProfileFields\AppInfo\Application;
1313
use OCA\ProfileFields\Controller\FieldDefinitionApiController;
1414
use OCA\ProfileFields\Db\FieldDefinitionMapper;
15+
use OCA\ProfileFields\Enum\FieldEditPolicy;
16+
use OCA\ProfileFields\Enum\FieldExposurePolicy;
1517
use OCA\ProfileFields\Enum\FieldType;
16-
use OCA\ProfileFields\Enum\FieldVisibility;
1718
use OCA\ProfileFields\Migration\Version1000Date20260309120000;
1819
use OCA\ProfileFields\Service\FieldDefinitionService;
1920
use OCP\AppFramework\App;
@@ -81,12 +82,10 @@ public function testDefinitionCrudFlow(): void {
8182
$fieldKey,
8283
'Performance score',
8384
FieldType::NUMBER->value,
84-
false,
85-
false,
86-
true,
87-
FieldVisibility::USERS->value,
85+
FieldEditPolicy::USERS->value,
86+
FieldExposurePolicy::PRIVATE->value,
8887
5,
89-
true,
88+
false,
9089
);
9190

9291
$this->assertSame(Http::STATUS_CREATED, $createResponse->getStatus());
@@ -105,10 +104,8 @@ public function testDefinitionCrudFlow(): void {
105104
$createResponse->getData()['id'],
106105
'Performance score updated',
107106
FieldType::NUMBER->value,
108-
true,
109-
false,
110-
true,
111-
FieldVisibility::PUBLIC->value,
107+
FieldEditPolicy::ADMINS->value,
108+
FieldExposurePolicy::PUBLIC->value,
112109
6,
113110
true,
114111
);

0 commit comments

Comments
 (0)