Skip to content

Commit 2344253

Browse files
committed
Minor tweaks
Signed-off-by: alexmerlin <[email protected]>
1 parent 5cd03b9 commit 2344253

File tree

7 files changed

+73
-8
lines changed

7 files changed

+73
-8
lines changed

src/Admin/src/InputFilter/Input/StatusInput.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
namespace Admin\Admin\InputFilter\Input;
66

77
use Core\Admin\Enum\AdminStatusEnum;
8+
use Core\App\Message;
89
use Laminas\Filter\StringTrim;
910
use Laminas\Filter\StripTags;
1011
use Laminas\InputFilter\Input;
@@ -25,6 +26,7 @@ public function __construct(?string $name = null, bool $isRequired = true)
2526
$this->getValidatorChain()
2627
->attachByName(InArray::class, [
2728
'haystack' => AdminStatusEnum::values(),
29+
'message' => Message::invalidValue('status'),
2830
], true);
2931
}
3032
}
Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
namespace Core\App\Fixture;
6+
7+
use Core\User\Entity\User;
8+
use Core\User\Entity\UserDetail;
9+
use Core\User\Entity\UserRole;
10+
use Core\User\Enum\UserRoleEnum;
11+
use Core\User\Enum\UserStatusEnum;
12+
use Doctrine\Common\DataFixtures\DependentFixtureInterface;
13+
use Doctrine\Common\DataFixtures\FixtureInterface;
14+
use Doctrine\Persistence\ObjectManager;
15+
16+
use function assert;
17+
18+
class UserLoader implements FixtureInterface, DependentFixtureInterface
19+
{
20+
public function load(ObjectManager $manager): void
21+
{
22+
$userRoleRepository = $manager->getRepository(UserRole::class);
23+
24+
$guestRole = $userRoleRepository->findOneBy([
25+
'name' => UserRoleEnum::Guest,
26+
]);
27+
assert($guestRole instanceof UserRole);
28+
29+
$userRole = $userRoleRepository->findOneBy([
30+
'name' => UserRoleEnum::User,
31+
]);
32+
assert($userRole instanceof UserRole);
33+
34+
$user = (new User())
35+
->setIdentity('[email protected]')
36+
->usePassword('dotkernel')
37+
->setStatus(UserStatusEnum::Active)
38+
->setHash(User::generateHash())
39+
->addRole($guestRole)
40+
->addRole($userRole);
41+
$manager->persist($user);
42+
43+
$userDetail = (new UserDetail())
44+
->setUser($user)
45+
->setFirstName('Test')
46+
->setLastName('Account')
47+
->setEmail('[email protected]');
48+
$manager->persist($userDetail);
49+
50+
$manager->flush();
51+
}
52+
53+
public function getDependencies(): array
54+
{
55+
return [
56+
UserRoleLoader::class,
57+
];
58+
}
59+
}

src/Core/src/App/src/Fixture/UserRoleLoader.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,9 @@ public function load(ObjectManager $manager): void
1616
$manager->persist(
1717
(new UserRole())->setName(UserRoleEnum::User)
1818
);
19+
$manager->persist(
20+
(new UserRole())->setName(UserRoleEnum::Guest)
21+
);
1922

2023
$manager->flush();
2124
}

src/Core/src/App/src/Message.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,11 @@ public static function mailNotSentTo(string $email): string
8787
return sprintf(self::MAIL_NOT_SENT_TO, $email);
8888
}
8989

90+
public static function mailSentUserActivation(string $email): string
91+
{
92+
return sprintf(self::MAIL_SENT_USER_ACTIVATION, $email);
93+
}
94+
9095
public static function restrictionDeprecation(string $first, string $second): string
9196
{
9297
return sprintf(self::RESTRICTION_DEPRECATION, $first, $second);

src/User/src/InputFilter/Input/StatusInput.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
namespace Admin\User\InputFilter\Input;
66

7+
use Core\App\Message;
78
use Core\User\Enum\UserStatusEnum;
89
use Laminas\Filter\StringTrim;
910
use Laminas\Filter\StripTags;
@@ -25,6 +26,7 @@ public function __construct(?string $name = null, bool $isRequired = true)
2526
$this->getValidatorChain()
2627
->attachByName(InArray::class, [
2728
'haystack' => UserStatusEnum::values(),
29+
'message' => Message::invalidValue('status'),
2830
], true);
2931
}
3032
}

test/Unit/Admin/InputFilter/AdminInputFilterTest.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -481,10 +481,7 @@ public function testWillValidateStatus(): void
481481
$this->assertArrayHasKey('status', $messages);
482482
$this->assertIsArray($messages['status']);
483483
$this->assertArrayHasKey('notInArray', $messages['status']);
484-
$this->assertSame(
485-
'The input was not found in the haystack',
486-
$messages['status']['notInArray']
487-
);
484+
$this->assertSame(Message::invalidValue('status'), $messages['status']['notInArray']);
488485
}
489486

490487
public function testWillValidateRoles(): void

test/Unit/Admin/InputFilter/EditAdminInputFilterTest.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -544,10 +544,7 @@ public function testWillValidateStatus(): void
544544
$this->assertArrayHasKey('status', $messages);
545545
$this->assertIsArray($messages['status']);
546546
$this->assertArrayHasKey('notInArray', $messages['status']);
547-
$this->assertSame(
548-
'The input was not found in the haystack',
549-
$messages['status']['notInArray']
550-
);
547+
$this->assertSame(Message::invalidValue('status'), $messages['status']['notInArray']);
551548
}
552549

553550
public function testWillValidateRoles(): void

0 commit comments

Comments
 (0)