Merge pull request #7 from php-etl/feature/logger

Added logger management + updated tests

Author: gplanchat

.github/workflows/infection.yaml

@@ -7,7 +7,7 @@ jobs:
             -   uses: actions/checkout@v2
             -   uses: shivammathur/setup-php@v2
                 with:
-                    php-version: '8.0'
+                    php-version: '8.1'
                     tools: composer:v2
                     coverage: pcov
             -   uses: actions/cache@v2
@@ -19,7 +19,7 @@ jobs:
             -   uses: php-actions/composer@v5
                 with:
                     args: --prefer-dist
-                    php_version: 8.0
+                    php_version: 8.1
 
             -   name: Infection
                 run: |

.github/workflows/phpstan-5.yaml

@@ -14,7 +14,7 @@ jobs:
       - uses: php-actions/composer@v5
         with:
           args: --prefer-dist
-          php_version: 8.0
+          php_version: 8.1
 
       - name: PHPStan
         uses: php-actions/phpstan@v2

.github/workflows/phpstan-7.yaml

@@ -14,7 +14,7 @@ jobs:
       - uses: php-actions/composer@v5
         with:
           args: --prefer-dist
-          php_version: 8.0
+          php_version: 8.1
 
       - name: PHPStan
         uses: php-actions/phpstan@v2

.github/workflows/phpstan-8.yaml

@@ -14,7 +14,7 @@ jobs:
       - uses: php-actions/composer@v5
         with:
           args: --prefer-dist
-          php_version: 8.0
+          php_version: 8.1
 
       - name: PHPStan
         uses: php-actions/phpstan@v2

.github/workflows/quality.yaml

@@ -17,7 +17,7 @@ jobs:
             -   uses: actions/checkout@v2
             -   uses: shivammathur/setup-php@v2
                 with:
-                    php-version: '8.0'
+                    php-version: '8.1'
                     tools: composer:v2
                     coverage: pcov
             -   uses: actions/cache@v2
@@ -29,7 +29,7 @@ jobs:
             -   uses: php-actions/composer@v5
                 with:
                     args: --prefer-dist
-                    php_version: 8.0
+                    php_version: 8.1
 
             -   name: Run tests & generate Coverage
                 run: bin/phpunit --configuration=phpunit.xml tests --coverage-html var/coverage --whitelist=src
@@ -52,7 +52,7 @@ jobs:
             -   uses: php-actions/composer@v5
                 with:
                     args: --prefer-dist
-                    php_version: 8.0
+                    php_version: 8.1
 
             -   name: PHPStan
                 uses: php-actions/phpstan@v2

composer.lock

@@ -127,6 +127,7 @@ class: new Node\Name\FullyQualified('Kiboko\\Component\\Flow\\SQL\\Extractor'),
                     : new Node\Expr\Array_(attributes: [
                         'kind' => Node\Expr\Array_::KIND_SHORT
                     ]),
+                new Node\Arg(value: $this->logger ?? new Node\Expr\New_(new Node\Name\FullyQualified('Psr\\Log\\NullLogger'))),
             ],
         );
     }

src/Builder/Extractor.php

@@ -130,6 +130,7 @@ class: new Node\Name\FullyQualified('Kiboko\\Component\\Flow\\SQL\\Loader'),
                     : new Node\Expr\Array_(attributes: [
                         'kind' => Node\Expr\Array_::KIND_SHORT
                     ]),
+                new Node\Arg(value: $this->logger ?? new Node\Expr\New_(new Node\Name\FullyQualified('Psr\\Log\\NullLogger'))),
             ],
         );
     }

src/Builder/Loader.php

@@ -190,6 +190,7 @@ class: new Node\Name\FullyQualified('Psr\Log\NullLogger')
                 new Node\Arg(
                     value: $this->compileAfterQueries()
                 ),
+                new Node\Arg(value: $this->logger ?? new Node\Expr\New_(new Node\Name\FullyQualified('Psr\\Log\\NullLogger'))),
             ],
         );
     }

src/Builder/Lookup.php

@@ -457,11 +457,11 @@ public function testExtractor(): void
         $this->assertBuildsExtractorExtractsExactly(
             [
                 [
-                    'id' => '1',
+                    'id' => 1,
                     'value' => 'Lorem ipsum dolor',
                 ],
                 [
-                    'id' => '2',
+                    'id' => 2,
                     'value' => 'Sit amet consecutir',
                 ],
             ],