Skip to content

Commit 22eb5da

Browse files
authored
Fix test with time and add ext-json to require-dev section of composer.json (#79)
1 parent 18a2934 commit 22eb5da

2 files changed

Lines changed: 18 additions & 10 deletions

File tree

composer.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,11 @@
2525
"yiisoft/var-dumper": "^1.0"
2626
},
2727
"require-dev": {
28+
"ext-json": "*",
2829
"phpunit/phpunit": "^9.5",
29-
"roave/infection-static-analysis-plugin": "^1.16",
30+
"roave/infection-static-analysis-plugin": "^1.18",
3031
"spatie/phpunit-watcher": "^1.23",
31-
"vimeo/psalm": "^4.18"
32+
"vimeo/psalm": "^4.22"
3233
},
3334
"provide": {
3435
"psr/log-implementation": "1.0.0"

tests/Message/FormatterTest.php

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -136,14 +136,21 @@ public function testFormatWithTimeCommaSeparated(): void
136136
public function testFormatWithSetFormatAndSetPrefix(): void
137137
{
138138
$this->formatter->setFormat(static fn (Message $message) => "({$message->level()}) {$message->message()}");
139-
$this->formatter->setPrefix(static function (Message $message) {
140-
$category = strtoupper($message->context('category'));
141-
$time = date('H:i:s', $message->context('time'));
142-
return "{$category}: ({$time})";
143-
});
144-
$message = new Message(LogLevel::INFO, 'message', ['category' => 'app', 'time' => 1508160390]);
145-
$expected = 'APP: (13:26:30)(info) message';
146-
$this->assertSame($expected, $this->formatter->format($message, []));
139+
$this->formatter->setPrefix(
140+
static function (Message $message) {
141+
$category = strtoupper($message->context('category'));
142+
$time = date('H:i:s', $message->context('time'));
143+
return "{$category}: ({$time})";
144+
}
145+
);
146+
147+
$time = 1508160390;
148+
$message = new Message(LogLevel::INFO, 'message', ['category' => 'app', 'time' => $time]);
149+
150+
$this->assertSame(
151+
'APP: (' . date('H:i:s', $time) . ')(info) message',
152+
$this->formatter->format($message, [])
153+
);
147154
}
148155

149156
public function testFormatWithContextAndSetFormat(): void

0 commit comments

Comments
 (0)