Skip to content

Commit 83ae4b8

Browse files
committed
add parameters to repository
1 parent 50b84a2 commit 83ae4b8

3 files changed

Lines changed: 20 additions & 15 deletions

File tree

src/Debug/Repository/Logger.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,14 @@
99

1010
interface Logger
1111
{
12-
public function log(string $message, int $created, string $level = 'error'): void;
12+
/**
13+
* @param array<string, mixed> $parameters
14+
*/
15+
public function log(string $message, int $created, string $level = 'error', array $parameters = []): void;
1316

1417

1518
/**
16-
* @return list<array{message: string, created: int}>
19+
* @return list<array{message: string, created: int, parameters: array<string, mixed>}>>
1720
*/
1821
public function getList(string $level = 'error'): array;
1922
}

src/Debug/Repository/LoggerSettings.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public function setup(?int $limit = null): void
3232
}
3333

3434

35-
public function log(string $message, int $created, string $level = 'error'): void
35+
public function log(string $message, int $created, string $level = 'error', array $parameters = []): void
3636
{
3737
$list = $this->settings->load(self::Key . $level);
3838

@@ -44,6 +44,7 @@ public function log(string $message, int $created, string $level = 'error'): voi
4444
$list[] = [
4545
'message' => $message,
4646
'created' => $created,
47+
'parameters' => $parameters,
4748
];
4849

4950
$this->settings->set(self::Key . $level, count($list) > $this->limit ? array_slice($list, 1, $this->limit) : $list, ['type' => 'array']);
@@ -61,6 +62,7 @@ public function getList(string $level = 'error'): array
6162
$output[] = [
6263
'message' => isset($item['message']) && is_scalar($item['message']) ? (string) $item['message'] : '',
6364
'created' => isset($item['created']) && is_scalar($item['created']) ? (int) $item['created'] : 0,
65+
'parameters' => isset($item['parameters']) && is_array($item['parameters']) ? $item['parameters'] : [],
6466
];
6567
}
6668

tests/Debug/Repository/LoggerSettingsTest.phpt

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class LoggerSettingsTest extends TestCase
2828
$settings->shouldReceive('set')->with('static:log_level', [
2929
['message' => 'test1', 'created' => 5],
3030
['message' => 'test2', 'created' => 6],
31-
['message' => 'test3', 'created' => 7]
31+
['message' => 'test3', 'created' => 7, 'parameters' => []]
3232
], ['type' => 'array'])->once();
3333

3434
$repository->log('test3', 7, 'level');
@@ -49,7 +49,7 @@ class LoggerSettingsTest extends TestCase
4949
$settings->shouldReceive('set')->with('static:log_level', [
5050
['message' => 'test2', 'created' => 6],
5151
['message' => 'test3', 'created' => 7],
52-
['message' => 'test4', 'created' => 8]
52+
['message' => 'test4', 'created' => 8, 'parameters' => []]
5353
], ['type' => 'array'])->once();
5454

5555
$repository->log('test4', 8, 'level');
@@ -64,7 +64,7 @@ class LoggerSettingsTest extends TestCase
6464
$repository->setup(3);
6565
$settings->shouldReceive('load')->with('static:log_level')->once()->andReturnNull();
6666
$settings->shouldReceive('set')->with('static:log_level', [
67-
['message' => 'test4', 'created' => 8]
67+
['message' => 'test4', 'created' => 8, 'parameters' => []]
6868
], ['type' => 'array'])->once();
6969

7070
$repository->log('test4', 8, 'level');
@@ -77,21 +77,21 @@ class LoggerSettingsTest extends TestCase
7777
{
7878
$repository = new LoggerSettings($settings = Mockery::mock(Settings::class));
7979
$settings->shouldReceive('load')->with('static:log_level')->once()->andReturn([
80-
['message' => 'test1', 'created' => 5],
81-
['message' => 'test2', 'created' => 6],
82-
['message' => 'test3', 'created' => 7],
80+
['message' => 'test1', 'created' => 5, 'parameters' => ['test1' => 'test1']],
81+
['message' => 'test2', 'created' => 6, 'parameters' => ['test2' => 'test2']],
82+
['message' => 'test3', 'created' => 7, 'parameters' => ['test3' => 'test3']],
8383
[],
8484
['message' => []],
8585
['created' => []]
8686
]);
8787

8888
Assert::same([
89-
['message' => 'test1', 'created' => 5],
90-
['message' => 'test2', 'created' => 6],
91-
['message' => 'test3', 'created' => 7],
92-
['message' => '', 'created' => 0],
93-
['message' => '', 'created' => 0],
94-
['message' => '', 'created' => 0],
89+
['message' => 'test1', 'created' => 5, 'parameters' => ['test1' => 'test1']],
90+
['message' => 'test2', 'created' => 6, 'parameters' => ['test2' => 'test2']],
91+
['message' => 'test3', 'created' => 7, 'parameters' => ['test3' => 'test3']],
92+
['message' => '', 'created' => 0, 'parameters' => []],
93+
['message' => '', 'created' => 0, 'parameters' => []],
94+
['message' => '', 'created' => 0, 'parameters' => []],
9595
], $repository->getList('level'));
9696
}
9797

0 commit comments

Comments
 (0)