Bump to PHPUnit 10

eclipxe13 · eclipxe13 · commit 4e9089a4218c · 2025-04-13T22:59:42.000-06:00
diff --git a/composer.json b/composer.json
@@ -26,7 +26,7 @@
     },
     "require-dev": {
         "ext-json": "*",
-        "phpunit/phpunit": "^9.5"
+        "phpunit/phpunit": "^10.5.45"
     },
     "prefer-stable": true,
     "autoload": {
diff --git a/docs/CHANGELOG.md b/docs/CHANGELOG.md
@@ -18,6 +18,7 @@ Esto incluye cambios al código relacionados con tipos, calidad de código y eli
 
 Los siguientes cambios aplican al entorno de desarrollo.
 
+- Se actualizan las pruebas a PHPUnit 10.
 - Se agrega la herramienta `composer-normalize` al proceso de construcción.
 - Se actualiza el estándar de código para `phpcs` y `php-cs-fixer`.
 
diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,18 +1,18 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-    xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
-    cacheResultFile="build/phpunit.result.cache"
-    bootstrap="tests/bootstrap.php"
-    colors="true"
-    >
-    <testsuites>
-        <testsuite name="default">
-            <directory>tests</directory>
-        </testsuite>
-    </testsuites>
-    <coverage>
-        <include>
-            <directory suffix=".php">./src/</directory>
-        </include>
-    </coverage>
+  xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+  cacheResultFile="build/phpunit.result.cache"
+  bootstrap="tests/bootstrap.php"
+  colors="true"
+  >
+  <testsuites>
+    <testsuite name="default">
+      <directory>tests</directory>
+    </testsuite>
+  </testsuites>
+  <source>
+    <include>
+      <directory suffix=".php">./src/</directory>
+    </include>
+  </source>
 </phpunit>
diff --git a/tests/Unit/DOMDocumentsTestCase.php b/tests/Unit/DOMDocumentsTestCase.php
@@ -9,52 +9,52 @@
 
 abstract class DOMDocumentsTestCase extends TestCase
 {
-    public function documentCfdi40(): DOMDocument
+    public static function documentCfdi40(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('cfdi40-real.xml'));
+        $document->load(static::filePath('cfdi40-real.xml'));
         return $document;
     }
 
-    public function documentCfdi33(): DOMDocument
+    public static function documentCfdi33(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('cfdi33-real.xml'));
+        $document->load(static::filePath('cfdi33-real.xml'));
         return $document;
     }
 
-    public function documentCfdi32(): DOMDocument
+    public static function documentCfdi32(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('cfdi32-real.xml'));
+        $document->load(static::filePath('cfdi32-real.xml'));
         return $document;
     }
 
-    public function documentRet20Mexican(): DOMDocument
+    public static function documentRet20Mexican(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('ret20-mexican-fake.xml'));
+        $document->load(static::filePath('ret20-mexican-fake.xml'));
         return $document;
     }
 
-    public function documentRet20Foreign(): DOMDocument
+    public static function documentRet20Foreign(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('ret20-foreign-fake.xml'));
+        $document->load(static::filePath('ret20-foreign-fake.xml'));
         return $document;
     }
 
-    public function documentRet10Mexican(): DOMDocument
+    public static function documentRet10Mexican(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('ret10-mexican-fake.xml'));
+        $document->load(static::filePath('ret10-mexican-fake.xml'));
         return $document;
     }
 
-    public function documentRet10Foreign(): DOMDocument
+    public static function documentRet10Foreign(): DOMDocument
     {
         $document = new DOMDocument();
-        $document->load($this->filePath('ret10-foreign-fake.xml'));
+        $document->load(static::filePath('ret10-foreign-fake.xml'));
         return $document;
     }
 }
diff --git a/tests/Unit/DiscoverExtractorTest.php b/tests/Unit/DiscoverExtractorTest.php
@@ -59,16 +59,16 @@ public function testThrowExceptionOnUnmatchedDocument(): void
     }
 
     /** @return array<string, array{DOMDocument, string}> */
-    public function providerExpressionOnValidDocuments(): array
+    public static function providerExpressionOnValidDocuments(): array
     {
         return [
-            'Cfdi40' => [$this->documentCfdi40(), 'CFDI40'],
-            'Cfdi33' => [$this->documentCfdi33(), 'CFDI33'],
-            'Cfdi32' => [$this->documentCfdi32(), 'CFDI32'],
-            'Ret20Mexican' => [$this->documentRet20Mexican(), 'RET20'],
-            'Ret20Foreign' => [$this->documentRet20Foreign(), 'RET20'],
-            'Ret10Mexican' => [$this->documentRet10Mexican(), 'RET10'],
-            'Ret10Foreign' => [$this->documentRet10Foreign(), 'RET10'],
+            'Cfdi40' => [self::documentCfdi40(), 'CFDI40'],
+            'Cfdi33' => [self::documentCfdi33(), 'CFDI33'],
+            'Cfdi32' => [self::documentCfdi32(), 'CFDI32'],
+            'Ret20Mexican' => [self::documentRet20Mexican(), 'RET20'],
+            'Ret20Foreign' => [self::documentRet20Foreign(), 'RET20'],
+            'Ret10Mexican' => [self::documentRet10Mexican(), 'RET10'],
+            'Ret10Foreign' => [self::documentRet10Foreign(), 'RET10'],
         ];
     }
 
diff --git a/tests/Unit/Extractors/Comprobante32Test.php b/tests/Unit/Extractors/Comprobante32Test.php
@@ -34,11 +34,11 @@ public function testExtractCfdi32(): void
     }
 
     /** @return array<string, array{DOMDocument}> */
-    public function providerCfdiDifferentVersions(): array
+    public static function providerCfdiDifferentVersions(): array
     {
         return [
-            'CFDI 4.0' => [$this->documentCfdi40()],
-            'CFDI 3.3' => [$this->documentCfdi33()],
+            'CFDI 4.0' => [self::documentCfdi40()],
+            'CFDI 3.3' => [self::documentCfdi33()],
         ];
     }
 
diff --git a/tests/Unit/Extractors/Comprobante33Test.php b/tests/Unit/Extractors/Comprobante33Test.php
@@ -34,11 +34,11 @@ public function testExtractCfdi33(): void
     }
 
     /** @return array<string, array{DOMDocument}> */
-    public function providerCfdiDifferentVersions(): array
+    public static function providerCfdiDifferentVersions(): array
     {
         return [
-            'CFDI 4.0' => [$this->documentCfdi40()],
-            'CFDI 3.2' => [$this->documentCfdi32()],
+            'CFDI 4.0' => [self::documentCfdi40()],
+            'CFDI 3.2' => [self::documentCfdi32()],
         ];
     }
 
diff --git a/tests/Unit/Extractors/Comprobante40Test.php b/tests/Unit/Extractors/Comprobante40Test.php
@@ -34,11 +34,11 @@ public function testExtractCfdi40(): void
     }
 
     /** @return array<string, array{DOMDocument}> */
-    public function providerCfdiDifferentVersions(): array
+    public static function providerCfdiDifferentVersions(): array
     {
         return [
-            'CFDI 3.3' => [$this->documentCfdi33()],
-            'CFDI 3.2' => [$this->documentCfdi32()],
+            'CFDI 3.3' => [self::documentCfdi33()],
+            'CFDI 3.2' => [self::documentCfdi32()],
         ];
     }
 
diff --git a/tests/Unit/Extractors/Retenciones10Test.php b/tests/Unit/Extractors/Retenciones10Test.php
@@ -44,11 +44,11 @@ public function testExtractRetenciones10Mexican(): void
     }
 
     /** @return array<string, array{DOMDocument}> */
-    public function providerCfdiDifferentVersions(): array
+    public static function providerCfdiDifferentVersions(): array
     {
         return [
-            'RET 2.0 Mexican' => [$this->documentRet20Mexican()],
-            'RET 2.0 Foreign' => [$this->documentRet20Foreign()],
+            'RET 2.0 Mexican' => [self::documentRet20Mexican()],
+            'RET 2.0 Foreign' => [self::documentRet20Foreign()],
         ];
     }
 
diff --git a/tests/Unit/Extractors/Retenciones20Test.php b/tests/Unit/Extractors/Retenciones20Test.php
@@ -52,11 +52,11 @@ public function testExtractRetenciones20Mexican(): void
     }
 
     /** @return array<string, array{DOMDocument}> */
-    public function providerCfdiDifferentVersions(): array
+    public static function providerCfdiDifferentVersions(): array
     {
         return [
-            'RET 1.0 Mexican' => [$this->documentRet10Mexican()],
-            'RET 1.0 Foreign' => [$this->documentRet10Foreign()],
+            'RET 1.0 Mexican' => [self::documentRet10Mexican()],
+            'RET 1.0 Foreign' => [self::documentRet10Foreign()],
         ];
     }
 

Original file line number	Diff line number	Diff line change
`@@ -9,52 +9,52 @@`
`9`	`9`
`10`	`10`	`abstract class DOMDocumentsTestCase extends TestCase`
`11`	`11`	`{`
`12`		`- public function documentCfdi40(): DOMDocument`
	`12`	`+ public static function documentCfdi40(): DOMDocument`
`13`	`13`	`{`
`14`	`14`	`$document = new DOMDocument();`
`15`		`- $document->load($this->filePath('cfdi40-real.xml'));`
	`15`	`+ $document->load(static::filePath('cfdi40-real.xml'));`
`16`	`16`	`return $document;`
`17`	`17`	`}`
`18`	`18`
`19`		`- public function documentCfdi33(): DOMDocument`
	`19`	`+ public static function documentCfdi33(): DOMDocument`
`20`	`20`	`{`
`21`	`21`	`$document = new DOMDocument();`
`22`		`- $document->load($this->filePath('cfdi33-real.xml'));`
	`22`	`+ $document->load(static::filePath('cfdi33-real.xml'));`
`23`	`23`	`return $document;`
`24`	`24`	`}`
`25`	`25`
`26`		`- public function documentCfdi32(): DOMDocument`
	`26`	`+ public static function documentCfdi32(): DOMDocument`
`27`	`27`	`{`
`28`	`28`	`$document = new DOMDocument();`
`29`		`- $document->load($this->filePath('cfdi32-real.xml'));`
	`29`	`+ $document->load(static::filePath('cfdi32-real.xml'));`
`30`	`30`	`return $document;`
`31`	`31`	`}`
`32`	`32`
`33`		`- public function documentRet20Mexican(): DOMDocument`
	`33`	`+ public static function documentRet20Mexican(): DOMDocument`
`34`	`34`	`{`
`35`	`35`	`$document = new DOMDocument();`
`36`		`- $document->load($this->filePath('ret20-mexican-fake.xml'));`
	`36`	`+ $document->load(static::filePath('ret20-mexican-fake.xml'));`
`37`	`37`	`return $document;`
`38`	`38`	`}`
`39`	`39`
`40`		`- public function documentRet20Foreign(): DOMDocument`
	`40`	`+ public static function documentRet20Foreign(): DOMDocument`
`41`	`41`	`{`
`42`	`42`	`$document = new DOMDocument();`
`43`		`- $document->load($this->filePath('ret20-foreign-fake.xml'));`
	`43`	`+ $document->load(static::filePath('ret20-foreign-fake.xml'));`
`44`	`44`	`return $document;`
`45`	`45`	`}`
`46`	`46`
`47`		`- public function documentRet10Mexican(): DOMDocument`
	`47`	`+ public static function documentRet10Mexican(): DOMDocument`
`48`	`48`	`{`
`49`	`49`	`$document = new DOMDocument();`
`50`		`- $document->load($this->filePath('ret10-mexican-fake.xml'));`
	`50`	`+ $document->load(static::filePath('ret10-mexican-fake.xml'));`
`51`	`51`	`return $document;`
`52`	`52`	`}`
`53`	`53`
`54`		`- public function documentRet10Foreign(): DOMDocument`
	`54`	`+ public static function documentRet10Foreign(): DOMDocument`
`55`	`55`	`{`
`56`	`56`	`$document = new DOMDocument();`
`57`		`- $document->load($this->filePath('ret10-foreign-fake.xml'));`
	`57`	`+ $document->load(static::filePath('ret10-foreign-fake.xml'));`
`58`	`58`	`return $document;`
`59`	`59`	`}`
`60`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,11 +34,11 @@ public function testExtractCfdi32(): void`
`34`	`34`	`}`
`35`	`35`
`36`	`36`	`/** @return array<string, array{DOMDocument}> */`
`37`		`- public function providerCfdiDifferentVersions(): array`
	`37`	`+ public static function providerCfdiDifferentVersions(): array`
`38`	`38`	`{`
`39`	`39`	`return [`
`40`		`- 'CFDI 4.0' => [$this->documentCfdi40()],`
`41`		`- 'CFDI 3.3' => [$this->documentCfdi33()],`
	`40`	`+ 'CFDI 4.0' => [self::documentCfdi40()],`
	`41`	`+ 'CFDI 3.3' => [self::documentCfdi33()],`
`42`	`42`	`];`
`43`	`43`	`}`
`44`	`44`
Original file line number	Diff line number	Diff line change
`@@ -44,11 +44,11 @@ public function testExtractRetenciones10Mexican(): void`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`/** @return array<string, array{DOMDocument}> */`
`47`		`- public function providerCfdiDifferentVersions(): array`
	`47`	`+ public static function providerCfdiDifferentVersions(): array`
`48`	`48`	`{`
`49`	`49`	`return [`
`50`		`- 'RET 2.0 Mexican' => [$this->documentRet20Mexican()],`
`51`		`- 'RET 2.0 Foreign' => [$this->documentRet20Foreign()],`
	`50`	`+ 'RET 2.0 Mexican' => [self::documentRet20Mexican()],`
	`51`	`+ 'RET 2.0 Foreign' => [self::documentRet20Foreign()],`
`52`	`52`	`];`
`53`	`53`	`}`
`54`	`54`
Original file line number	Diff line number	Diff line change
`@@ -52,11 +52,11 @@ public function testExtractRetenciones20Mexican(): void`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`/** @return array<string, array{DOMDocument}> */`
`55`		`- public function providerCfdiDifferentVersions(): array`
	`55`	`+ public static function providerCfdiDifferentVersions(): array`
`56`	`56`	`{`
`57`	`57`	`return [`
`58`		`- 'RET 1.0 Mexican' => [$this->documentRet10Mexican()],`
`59`		`- 'RET 1.0 Foreign' => [$this->documentRet10Foreign()],`
	`58`	`+ 'RET 1.0 Mexican' => [self::documentRet10Mexican()],`
	`59`	`+ 'RET 1.0 Foreign' => [self::documentRet10Foreign()],`
`60`	`60`	`];`
`61`	`61`	`}`
`62`	`62`