Skip to content

Commit 51ccc8a

Browse files
committed
fix: Fix integration tests
Birthdate was missing in editable fields while being editable, and the order was checked while having no importance. Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
1 parent 173a338 commit 51ccc8a

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

build/integration/features/bootstrap/Provisioning.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -267,6 +267,9 @@ public function userHasEditableFields($user, $fields) {
267267

268268
$expectedFields = $fields->getRows();
269269
$expectedFields = $this->simplifyArray($expectedFields);
270+
/* Sort both arrays as order is not important */
271+
sort($fieldsArray);
272+
sort($expectedFields);
270273
Assert::assertEquals($expectedFields, $fieldsArray);
271274
}
272275

build/integration/features/provisioning-v1.feature

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ Feature: provisioning
7979
| role |
8080
| headline |
8181
| biography |
82+
| birthdate |
8283
| profile_enabled |
8384
| pronouns |
8485
Given As an "brand-new-user"
@@ -96,6 +97,7 @@ Feature: provisioning
9697
| role |
9798
| headline |
9899
| biography |
100+
| birthdate |
99101
| profile_enabled |
100102
| pronouns |
101103
Then user "self" has editable fields
@@ -112,6 +114,7 @@ Feature: provisioning
112114
| role |
113115
| headline |
114116
| biography |
117+
| birthdate |
115118
| profile_enabled |
116119
| pronouns |
117120

0 commit comments

Comments
 (0)