Skip to content

Commit 0b724fa

Browse files
SergiuBota1SergiuBota1
committed
fixed tests for logger
Signed-off-by: bota <[email protected]>
1 parent 014827c commit 0b724fa

File tree

5 files changed

+34
-37
lines changed

5 files changed

+34
-37
lines changed

‎composer.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
"dotkernel/dot-dependency-injection": "^1.2",
5353
"dotkernel/dot-errorhandler": "4.2.1",
5454
"dotkernel/dot-mail": "^5.3.0",
55+
"laminas/laminas-authentication": "2.18.0",
5556
"laminas/laminas-component-installer": "^3.5",
5657
"laminas/laminas-config-aggregator": "^1.18",
5758
"laminas/laminas-diactoros": "^3.0",

‎src/Core/src/App/src/InputFilter/AbstractInputFilter.php‎

Lines changed: 0 additions & 15 deletions
This file was deleted.

‎src/Core/src/App/src/Service/MailService.php‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
use Core\App\Message;
88
use Core\User\Entity\User;
99
use Dot\DependencyInjection\Attribute\Inject;
10-
use Dot\Log\LoggerInterface;
10+
use Dot\Log\Logger;
1111
use Dot\Mail\Exception\MailException;
1212
use Symfony\Component\Mailer\Exception\TransportExceptionInterface;
1313

@@ -25,7 +25,7 @@ class MailService
2525
)]
2626
public function __construct(
2727
protected \Dot\Mail\Service\MailService $mailService,
28-
protected LoggerInterface $logger,
28+
protected Logger $logger,
2929
private readonly array $config,
3030
) {
3131
}
@@ -46,7 +46,7 @@ public function sendActivationMail(User $user, string $body): bool
4646
try {
4747
return $this->mailService->send()->isValid();
4848
} catch (MailException | TransportExceptionInterface $exception) {
49-
$this->logger->err($exception->getMessage());
49+
$this->logger->error($exception->getMessage());
5050
throw new MailException(sprintf(Message::MAIL_NOT_SENT_TO, $user->getEmail()));
5151
}
5252
}
@@ -65,7 +65,7 @@ public function sendResetPasswordRequestedMail(User $user, string $body): bool
6565
try {
6666
return $this->mailService->send()->isValid();
6767
} catch (MailException | TransportExceptionInterface $exception) {
68-
$this->logger->err($exception->getMessage());
68+
$this->logger->error($exception->getMessage());
6969
throw new MailException(sprintf(Message::MAIL_NOT_SENT_TO, $user->getEmail()));
7070
}
7171
}
@@ -84,7 +84,7 @@ public function sendResetPasswordCompletedMail(User $user, string $body): bool
8484
try {
8585
return $this->mailService->send()->isValid();
8686
} catch (MailException | TransportExceptionInterface $exception) {
87-
$this->logger->err($exception->getMessage());
87+
$this->logger->error($exception->getMessage());
8888
throw new MailException(sprintf(Message::MAIL_NOT_SENT_TO, $user->getEmail()));
8989
}
9090
}
@@ -103,7 +103,7 @@ public function sendRecoverIdentityMail(User $user, string $body): bool
103103
try {
104104
return $this->mailService->send()->isValid();
105105
} catch (MailException | TransportExceptionInterface $exception) {
106-
$this->logger->err($exception->getMessage());
106+
$this->logger->error($exception->getMessage());
107107
throw new MailException(sprintf(Message::MAIL_NOT_SENT_TO, $user->getEmail()));
108108
}
109109
}
@@ -120,7 +120,7 @@ public function sendWelcomeMail(User $user, string $body): bool
120120
try {
121121
return $this->mailService->send()->isValid();
122122
} catch (MailException | TransportExceptionInterface $exception) {
123-
$this->logger->err($exception->getMessage());
123+
$this->logger->error($exception->getMessage());
124124
throw new MailException(sprintf(Message::MAIL_NOT_SENT_TO, $user->getEmail()));
125125
}
126126
}

‎test/App/Message/ExampleMessageHandlerTest.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ public function setUp(): void
4141
$this->logger = new Logger([
4242
'writers' => [
4343
'FileWriter' => [
44-
'name' => 'null',
45-
'priority' => Logger::ALERT,
44+
'name' => 'null',
45+
'level' => Logger::ALERT,
4646
],
4747
],
4848
]);

‎test/Swoole/Delegators/TCPServerDelegatorTest.php‎

Lines changed: 24 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@
44

55
namespace QueueTest\Swoole\Delegators;
66

7-
use Dot\Log\LoggerInterface;
7+
use Dot\Log\Logger;
88
use PHPUnit\Framework\Attributes\RunInSeparateProcess;
99
use PHPUnit\Framework\MockObject\Exception;
1010
use PHPUnit\Framework\TestCase;
11+
use Psr\Container\ContainerExceptionInterface;
1112
use Psr\Container\ContainerInterface;
1213
use Queue\Swoole\Delegators\TCPServerDelegator;
1314
use Swoole\Server;
@@ -16,22 +17,40 @@
1617

1718
class TCPServerDelegatorTest extends TestCase
1819
{
20+
protected Logger $logger;
21+
22+
/**
23+
* @throws ContainerExceptionInterface
24+
*/
25+
public function setUp(): void
26+
{
27+
parent::setUp();
28+
29+
$this->logger = new Logger([
30+
'writers' => [
31+
'FileWriter' => [
32+
'name' => 'null',
33+
'level' => Logger::ALERT,
34+
],
35+
],
36+
]);
37+
}
38+
1939
/**
2040
* @throws Exception
2141
*/
2242
#[RunInSeparateProcess]
2343
public function testInvokeRegistersAllCallbacks(): void
2444
{
25-
$logger = $this->createMock(LoggerInterface::class);
26-
$bus = $this->createMock(MessageBusInterface::class);
45+
$bus = $this->createMock(MessageBusInterface::class);
2746

2847
$server = new DummySwooleServer();
2948
$callback = fn (): Server => $server;
3049

3150
$container = $this->createMock(ContainerInterface::class);
3251
$container->method('get')->willReturnMap([
3352
[MessageBusInterface::class, $bus],
34-
['dot-log.queue-log', $logger],
53+
['dot-log.queue-log', $this->logger],
3554
]);
3655

3756
$delegator = new TCPServerDelegator();
@@ -61,21 +80,13 @@ public function testReceiveCallbackDispatchesMessagesAndLogs(): void
6180
return new Envelope($message);
6281
});
6382

64-
$logger = $this->createMock(LoggerInterface::class);
65-
$logger->expects($this->once())
66-
->method('notice')
67-
->with(
68-
$this->equalTo("Request received on receive"),
69-
$this->arrayHasKey('fd')
70-
);
71-
7283
$server = new DummySwooleServer();
7384
$callback = fn (): Server => $server;
7485

7586
$container = $this->createMock(ContainerInterface::class);
7687
$container->method('get')->willReturnMap([
7788
[MessageBusInterface::class, $bus],
78-
['dot-log.queue-log', $logger],
89+
['dot-log.queue-log', $this->logger],
7990
]);
8091

8192
$delegator = new TCPServerDelegator();

0 commit comments

Comments
 (0)