Merge pull request #12 from run-as-root/feature/update-unit-tests

Update Unit tests to make them compatible with phpunit 10

Author: vpodorozh

.gitignore

@@ -4,3 +4,4 @@
 vendor/
 composer.lock
 .phpunit.result.cache
+.phpunit.cache/

lib/Test/Unit/Map/ProxiedConstructArgsToDiConfigMapperTest.php

@@ -27,7 +27,7 @@ public function test_map(array $diConfig, string $instanceClassName, array $prox
         $this->assertEquals($expected, $result);
     }
 
-    public function mapDataProvider(): array
+    public static function mapDataProvider(): array
     {
         return [
             'case1' => [
@@ -108,4 +108,4 @@ public function mapDataProvider(): array
 
         ];
     }
-}
+}

lib/Test/Unit/Service/GetProxiedConstructArgsConfigServiceTest.php

@@ -31,7 +31,17 @@ public function test_get(): void
 
         $this->proxyValidator->expects($this->exactly(2))
             ->method('validate')
-            ->withConsecutive(['Some\Class'], ['Some\OtherClass'])
+            ->willReturnCallback(function () use (&$i) {
+                switch ($i) {
+                    case 0:
+                        return ['Some\Class'];
+                    case 1:
+                        return ['Some\OtherClass'];
+                }
+                $i++;
+
+                return [];
+            })
             ->willReturnOnConsecutiveCalls(true, false);
 
         $this->assertEquals(
@@ -56,7 +66,17 @@ public function test_get_with_no_eligible_classes(): void
 
         $this->proxyValidator->expects($this->exactly(2))
             ->method('validate')
-            ->withConsecutive(['Some\Class'], ['Some\OtherClass'])
+            ->willReturnCallback(function () use (&$i) {
+                switch ($i) {
+                    case 0:
+                        return ['Some\Class'];
+                    case 1:
+                        return ['Some\OtherClass'];
+                }
+                $i++;
+
+                return [];
+            })
             ->willThrowException(new ClassIsNotEligibleForProxyException());
 
         $this->assertEquals([], $this->sut->get($constructConfig));
@@ -66,4 +86,4 @@ public function testGetWithNoConstructConfig(): void
     {
         $this->assertEquals([], $this->sut->get([]));
     }
-}
+}

phpunit.xml

@@ -1,32 +1,21 @@
 <?xml version="1.0" encoding="UTF-8"?>
-
-<phpunit bootstrap = "vendor/autoload.php"
-         backupGlobals               = "false"
-         backupStaticAttributes      = "false"
-         colors                      = "true"
-         convertErrorsToExceptions   = "true"
-         convertNoticesToExceptions  = "true"
-         convertWarningsToExceptions = "true"
-         processIsolation            = "false"
-         stopOnFailure               = "false">
-
-    <testsuites>
-        <testsuite name="run-as-root/magento-cli-auto-proxy">
-            <directory>lib/Test/Unit</directory>
-        </testsuite>
-    </testsuites>
-
-    <coverage cacheDirectory=".phpunit.cache/code-coverage" processUncoveredFiles="true">
-        <include>
-            <directory suffix=".php">lib</directory>
-        </include>
-        <report>
-            <clover outputFile="clover.xml" />
-        </report>
-    </coverage>
-
-    <php>
-        <env name="APP_ENV" value="testing"/>
-    </php>
-
-</phpunit>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" bootstrap="vendor/autoload.php" backupGlobals="false" colors="true" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
+  <testsuites>
+    <testsuite name="run-as-root/magento-cli-auto-proxy">
+      <directory>lib/Test/Unit</directory>
+    </testsuite>
+  </testsuites>
+  <coverage>
+    <report>
+      <clover outputFile="clover.xml"/>
+    </report>
+  </coverage>
+  <php>
+    <env name="APP_ENV" value="testing"/>
+  </php>
+  <source>
+    <include>
+      <directory suffix=".php">lib</directory>
+    </include>
+  </source>
+</phpunit>