Skip to content

Commit 456cb16

Browse files
committed
Added readme and fixed logger class name
1 parent 1c1619b commit 456cb16

File tree

7 files changed

+64
-30
lines changed

7 files changed

+64
-30
lines changed

‎CHANGELOG.md‎

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

‎README.md‎

Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
# ETL Adapter: Logger
2+
3+
[![Minimum PHP Version](https://img.shields.io/badge/php-%3E%3D%207.4-8892BF.svg)](https://php.net/)
4+
5+
## Description
6+
7+
ETL Adapter that provides PSR Logger support for ETL.
8+
9+
## Loader - PsrLogger
10+
11+
Load each row into PsrLoggerInterface implementation.
12+
13+
```php
14+
<?php
15+
16+
$logger = new TestLogger();
17+
18+
$loader = new PsrLoggerLoader($logger, 'row log', LogLevel::ERROR);
19+
20+
$loader->load(new Rows(
21+
Row::create(
22+
new Row\Entry\IntegerEntry('id', 12345),
23+
Row\Entry\StringEntry::lowercase('name', 'Norbert')
24+
)
25+
));
26+
27+
$this->assertTrue($logger->hasErrorRecords());
28+
$this->assertTrue($logger->hasError('row log'));
29+
```
30+
31+
## Development
32+
33+
In order to install dependencies please, launch following commands:
34+
35+
```bash
36+
composer install
37+
```
38+
39+
## Run Tests
40+
41+
In order to execute full test suite, please launch following command:
42+
43+
```bash
44+
composer build
45+
```
46+
47+
It's recommended to use [pcov](https://pecl.php.net/package/pcov) for code coverage however you can also use
48+
xdebug by setting `XDEBUG_MODE=coverage` env variable.

‎infection.json‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,14 +18,14 @@
1818
},
1919
"mutators": {
2020
"@default": true,
21-
"DecrementInteger": {
21+
"MethodCallRemoval": {
2222
"ignore": [
23-
"Flow\\ETL\\Extractor\\MemoryExtractor::extract"
23+
"Flow\\ETL\\Adapter\\Logger\\Logger\\DumpLogger::log"
2424
]
2525
},
26-
"IncrementInteger": {
26+
"ArrayItemRemoval": {
2727
"ignore": [
28-
"Flow\\ETL\\Extractor\\MemoryExtractor::extract"
28+
"Flow\\ETL\\Adapter\\Logger\\Logger\\DumpLogger::log"
2929
]
3030
}
3131
},

‎src/Flow/ETL/Adapter/Logger/Logger/DumpLogger.php‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,14 @@
88

99
final class DumpLogger extends AbstractLogger
1010
{
11-
public function log($level, $message, array $context = array())
11+
public function log($level, $message, array $context = []) : void
1212
{
1313
if (\class_exists('\\Symfony\\Component\\VarDumper\\VarDumper')) {
14+
/** @psalm-suppress UndefinedClass */
1415
\Symfony\Component\VarDumper\VarDumper::dump([$message => $context]);
1516
} else {
1617
/** @psalm-suppress ForbiddenCode */
17-
var_dump([$message => $context]);
18+
\var_dump([$message => $context]);
1819
}
1920
}
20-
}
21+
}

src/Flow/ETL/Adapter/Logger/PSRLoggerLoader.php renamed to src/Flow/ETL/Adapter/Logger/PsrLoggerLoader.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
use Psr\Log\LoggerInterface;
1111
use Psr\Log\LogLevel;
1212

13-
final class PSRLoggerLoader implements Loader
13+
final class PsrLoggerLoader implements Loader
1414
{
1515
private LoggerInterface $logger;
1616

‎tests/Flow/ETL/Tests/Unit/Adapter/Logger/Logger/DumpLoggerTest.php‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@ public function test_logger() : void
1616
\ob_start();
1717
$logger->error('error', ['id' => 1]);
1818
$output = \ob_get_contents();
19-
\ob_end_clean();;
19+
\ob_end_clean();
2020

21-
$this->assertStringContainsString(<<<OUTPUT
21+
$this->assertStringContainsString(
22+
<<<'OUTPUT'
2223
array(1) {
2324
["error"]=>
2425
array(1) {
@@ -30,4 +31,4 @@ public function test_logger() : void
3031
$output
3132
);
3233
}
33-
}
34+
}

tests/Flow/ETL/Tests/Unit/Adapter/Logger/PSRLoggerLoaderTest.php renamed to tests/Flow/ETL/Tests/Unit/Adapter/Logger/PsrLoggerLoaderTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,20 +4,20 @@
44

55
namespace Flow\ETL\Tests\Unit\Adapter\Logger;
66

7-
use Flow\ETL\Adapter\Logger\PSRLoggerLoader;
7+
use Flow\ETL\Adapter\Logger\PsrLoggerLoader;
88
use Flow\ETL\Row;
99
use Flow\ETL\Rows;
1010
use PHPUnit\Framework\TestCase;
1111
use Psr\Log\LogLevel;
1212
use Psr\Log\Test\TestLogger;
1313

14-
final class PSRLoggerLoaderTest extends TestCase
14+
final class PsrLoggerLoaderTest extends TestCase
1515
{
1616
public function test_psr_logger_loader() : void
1717
{
1818
$logger = new TestLogger();
1919

20-
$loader = new PSRLoggerLoader($logger, 'row log', LogLevel::ERROR);
20+
$loader = new PsrLoggerLoader($logger, 'row log', LogLevel::ERROR);
2121

2222
$loader->load(new Rows(
2323
Row::create(

0 commit comments

Comments
 (0)