Merge pull request #30 from silarhi/app-updates

Drop php 8.0 and 8.1 support

Author: guillaume-sainthillier

.laminas-ci.json

@@ -15,13 +15,13 @@
         }
     ],
     "require": {
-        "php": ">= 8.0"
+        "php": ">=8.2"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.66",
-        "phpunit/phpunit": "^9.6.19",
-        "phpstan/phpstan": "^1.12",
-        "rector/rector": "^1.2"
+        "phpstan/phpstan": "^2",
+        "phpunit/phpunit": "^10.5.5",
+        "rector/rector": "^2"
     },
     "autoload": {
         "psr-4": {

composer.json

@@ -1,17 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="false"
-         backupStaticAttributes="false"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         backupGlobals="false"
          bootstrap="vendor/autoload.php"
          colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false">
-  <coverage>
-    <include>
-      <directory>./src</directory>
-    </include>
-  </coverage>
+         processIsolation="false"
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+         cacheDirectory=".phpunit.cache"
+         backupStaticProperties="false"
+>
   <testsuites>
     <testsuite name="Test Suite">
       <directory>./tests</directory>
@@ -21,4 +17,9 @@
     <ini name="error_reporting" value="32767"/>
     <ini name="date.timezone" value="Europe/Paris"/>
   </php>
+  <source>
+    <include>
+      <directory>./src</directory>
+    </include>
+  </source>
 </phpunit>

phpunit.xml.dist

@@ -23,5 +23,5 @@
     // uncomment to reach your current PHP version
     ->withPhpSets()
     ->withSets([
-        PHPUnitSetList::PHPUNIT_90,
+        PHPUnitSetList::PHPUNIT_100,
     ]);

rector.php

@@ -15,6 +15,6 @@
 
 use Exception;
 
-class ParseException extends Exception
+final class ParseException extends Exception
 {
 }

src/Exceptions/ParseException.php

@@ -13,7 +13,7 @@
 
 namespace Silarhi\Caf\Model;
 
-class PaymentSlip
+final class PaymentSlip
 {
     /** @var PaymentSlipLine[] */
     private array $lines = [];

src/Model/PaymentSlip.php

@@ -15,7 +15,7 @@
 
 use DateTimeInterface;
 
-class PaymentSlipLine
+final class PaymentSlipLine
 {
     private ?string $reference = null;
 

src/Model/PaymentSlipLine.php

@@ -26,7 +26,7 @@
 /**
  * Parser for LA44ZZ file
  */
-class PaymentSlipParser
+final class PaymentSlipParser
 {
     private const TABLE_CONTENT_REGEX = '/-{121}[\S\s]*-{121}\s*?([\s|\S]*)\s*-{121}[\S\s]*-{121}/U';
 

src/Parser/PaymentSlipParser.php

@@ -17,7 +17,7 @@
 use Silarhi\Caf\Exceptions\ParseException;
 use Silarhi\Caf\Parser\PaymentSlipParser;
 
-class PaymentSlipParserTest extends TestCase
+final class PaymentSlipParserTest extends TestCase
 {
     public function testEmptyInput(): void
     {
@@ -52,7 +52,7 @@ public function testParsing(): void
         $content = file_get_contents(__DIR__ . '/../fixtures/LA44ZZ/caf_LA44.txt');
         $this->assertNotFalse($content);
         $result = $parser->parse($content);
-        $this->assertNotNull($result);
+        $this->assertNotCount(0, $result->getLines());
     }
 
     public function testParsing2(): void
@@ -61,7 +61,7 @@ public function testParsing2(): void
         $content = file_get_contents(__DIR__ . '/../fixtures/LA44ZZ/caf_LA44_2.txt');
         $this->assertNotFalse($content);
         $result = $parser->parse($content);
-        $this->assertNotNull($result);
+        $this->assertNotCount(0, $result->getLines());
     }
 
     public function testParsing3(): void
@@ -70,6 +70,6 @@ public function testParsing3(): void
         $content = file_get_contents(__DIR__ . '/../fixtures/LA44ZZ/caf_LA44_3.txt');
         $this->assertNotFalse($content);
         $result = $parser->parse($content);
-        $this->assertNotNull($result);
+        $this->assertNotCount(0, $result->getLines());
     }
 }