Skip to content

Commit 025b2e8

Browse files
committed
return a created resource
1 parent 20cbd74 commit 025b2e8

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

src/Domain/Subscription/Service/Manager/SubscriberBlacklistManager.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,12 @@ public function getBlacklistInfo(string $email): ?UserBlacklist
3131
{
3232
return $this->userBlacklistRepository->findBlacklistInfoByEmail($email);
3333
}
34-
// 095-38-25-55
35-
public function addEmailToBlacklist(string $email, ?string $reasonData = null): void
34+
35+
public function addEmailToBlacklist(string $email, ?string $reasonData = null): UserBlacklist
3636
{
3737
$existing = $this->subscriberRepository->isEmailBlacklisted($email);
3838
if ($existing) {
39-
return;
39+
return $this->getBlacklistInfo($email);
4040
}
4141

4242
$blacklistEntry = new UserBlacklist();
@@ -54,6 +54,8 @@ public function addEmailToBlacklist(string $email, ?string $reasonData = null):
5454
}
5555

5656
$this->entityManager->flush();
57+
58+
return $blacklistEntry;
5759
}
5860

5961
public function removeEmailFromBlacklist(string $email): void

tests/Unit/Domain/Subscription/Service/Manager/SubscriberBlacklistManagerTest.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,11 @@ public function testAddEmailToBlacklistDoesNotAddIfAlreadyBlacklisted(): void
7474
7575
->willReturn(true);
7676

77+
$this->userBlacklistRepository
78+
->expects($this->once())
79+
->method('findBlacklistInfoByEmail')
80+
->willReturn($this->createMock(UserBlacklist::class));
81+
7782
$this->entityManager
7883
->expects($this->never())
7984
->method('persist');

0 commit comments

Comments
 (0)