test: compatibility with PHPUnit 10 (api-platform#5701)

* test: compatibility with PHPUnit 10

* fix

* another fix

* one more fix

* fix

* try again

* again

* fixed locally

* clean composer.json

Author: dunglas

.github/workflows/ci.yml

@@ -620,7 +620,8 @@ jobs:
       - name: Clear test app cache
         run: tests/Fixtures/app/console cache:clear --ansi
       - name: Run PHPUnit tests
-        run: vendor/bin/simple-phpunit
+        # The PHPUnit Bridge doesn't support PHPUnit 10 yet: https://github.com/symfony/symfony/issues/49069
+        run: vendor/bin/phpunit -c phpunit10.xml.dist
 
   behat-symfony-next:
     name: Behat (PHP ${{ matrix.php }}) (Symfony dev)

.gitignore

@@ -2,6 +2,7 @@
 /.php-cs-fixer.php
 /.php-cs-fixer.cache
 /.phpunit.result.cache
+/.phpunit.cache/
 /build/
 /composer.lock
 /composer.phar

CONTRIBUTING.md

@@ -124,7 +124,7 @@ Both `simple-phpunit` and `behat` are development dependencies and should be ava
 
 To launch unit tests:
 
-    vendor/bin/simple-phpunit --stop-on-failure -vvv
+    vendor/bin/simple-phpunit --stop-on-defect -vvv
 
 If you want coverage, you will need the `pcov` PHP extension and run:
 

composer.json

@@ -28,7 +28,7 @@
     },
     "require-dev": {
         "behat/behat": "^3.1",
-        "behat/mink": "^1.9@dev",
+        "behat/mink": "^1.9",
         "doctrine/cache": "^1.11 || ^2.1",
         "doctrine/common": "^3.2.2",
         "doctrine/data-fixtures": "^1.2.2",

phpunit.xml.dist

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd" backupGlobals="false" bootstrap="tests/Fixtures/app/bootstrap.php" colors="true">
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd" backupGlobals="false" bootstrap="tests/Fixtures/app/bootstrap.php" colors="true">
     <php>
         <ini name="error_reporting" value="-1" />
         <ini name="memory_limit" value="-1" />

phpunit10.xml.dist

@@ -0,0 +1,38 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd" backupGlobals="false" bootstrap="tests/Fixtures/app/bootstrap.php" colors="true" cacheDirectory=".phpunit.cache">
+  <php>
+    <ini name="error_reporting" value="-1"/>
+    <ini name="memory_limit" value="-1"/>
+    <server name="SYMFONY_DEPRECATIONS_HELPER" value="max[total]=0&amp;ignoreFile=./tests/.ignored-deprecations"/>
+    <!-- This is necessary for GitHub Actions to work properly -->
+    <server name="SYMFONY_PHPUNIT_DIR" value="vendor/bin/.phpunit"/>
+    <server name="SYMFONY_PHPUNIT_REMOVE" value="symfony/yaml"/>
+    <server name="KERNEL_DIR" value="tests/Fixtures/app/"/>
+    <server name="KERNEL_CLASS" value="AppKernel"/>
+    <env name="APP_ENV" value="test"/>
+  </php>
+
+  <testsuites>
+    <testsuite name="Project Test Suite">
+      <directory>tests</directory>
+    </testsuite>
+  </testsuites>
+
+  <groups>
+    <exclude>
+      <group>mongodb</group>
+    </exclude>
+  </groups>
+
+  <source>
+    <include>
+      <directory>.</directory>
+    </include>
+    <exclude>
+      <directory>features</directory>
+      <directory>tests</directory>
+      <directory>vendor</directory>
+      <file>.php-cs-fixer.dist.php</file>
+    </exclude>
+  </source>
+</phpunit>

src/JsonSchema/Tests/SchemaTest.php

@@ -44,7 +44,7 @@ public function testCollectionJsonSchemaVersion(string $version, string $ref): v
         $this->assertSame('Foo', $schema->getItemsDefinitionKey());
     }
 
-    public function versionProvider(): iterable
+    public static function versionProvider(): iterable
     {
         yield [Schema::VERSION_JSON_SCHEMA, '#/definitions/Foo'];
         yield [Schema::VERSION_SWAGGER, '#/definitions/Foo'];
@@ -85,7 +85,7 @@ public function testDefinitions(string $version, array $baseDefinitions): void
         $this->assertArrayNotHasKey('components', $schema->getArrayCopy(false));
     }
 
-    public function definitionsDataProvider(): iterable
+    public static function definitionsDataProvider(): iterable
     {
         yield [Schema::VERSION_OPENAPI,  ['foo' => []]];
         yield [Schema::VERSION_JSON_SCHEMA,  ['foo' => []]];

src/JsonSchema/Tests/TypeFactoryTest.php

@@ -41,7 +41,7 @@ public function testGetType(array $schema, Type $type): void
         $this->assertEquals($schema, $typeFactory->getType($type, 'json', null, null, new Schema(Schema::VERSION_OPENAPI)));
     }
 
-    public function typeProvider(): iterable
+    public static function typeProvider(): iterable
     {
         yield [['type' => 'integer'], new Type(Type::BUILTIN_TYPE_INT)];
         yield [['nullable' => true, 'type' => 'integer'], new Type(Type::BUILTIN_TYPE_INT, true)];
@@ -169,7 +169,7 @@ public function testGetTypeWithJsonSchemaSyntax(array $schema, Type $type): void
         $this->assertEquals($schema, $typeFactory->getType($type, 'json', null, null, new Schema(Schema::VERSION_JSON_SCHEMA)));
     }
 
-    public function jsonSchemaTypeProvider(): iterable
+    public static function jsonSchemaTypeProvider(): iterable
     {
         yield [['type' => 'integer'], new Type(Type::BUILTIN_TYPE_INT)];
         yield [['type' => ['integer', 'null']], new Type(Type::BUILTIN_TYPE_INT, true)];
@@ -290,7 +290,7 @@ public function testGetTypeWithOpenAPIV2Syntax(array $schema, Type $type): void
         $this->assertEquals($schema, $typeFactory->getType($type, 'json', null, null, new Schema(Schema::VERSION_SWAGGER)));
     }
 
-    public function openAPIV2TypeProvider(): iterable
+    public static function openAPIV2TypeProvider(): iterable
     {
         yield [['type' => 'integer'], new Type(Type::BUILTIN_TYPE_INT)];
         yield [['type' => 'integer'], new Type(Type::BUILTIN_TYPE_INT, true)];
@@ -412,20 +412,21 @@ public function testGetClassType(): void
     }
 
     /** @dataProvider classTypeWithNullabilityDataProvider */
-    public function testGetClassTypeWithNullability(array $expected, SchemaFactoryInterface $schemaFactory, Schema $schema): void
+    public function testGetClassTypeWithNullability(array $expected, callable $schemaFactoryFactory, Schema $schema): void
     {
         $typeFactory = new TypeFactory();
-        $typeFactory->setSchemaFactory($schemaFactory);
+        $typeFactory->setSchemaFactory($schemaFactoryFactory($this));
 
         self::assertEquals(
             $expected,
             $typeFactory->getType(new Type(Type::BUILTIN_TYPE_OBJECT, true, Dummy::class), 'jsonld', true, ['foo' => 'bar'], $schema)
         );
     }
 
-    public function classTypeWithNullabilityDataProvider(): iterable
+    public static function classTypeWithNullabilityDataProvider(): iterable
     {
-        $schemaFactory = $this->createSchemaFactoryMock($schema = new Schema());
+        $schema = new Schema();
+        $schemaFactoryFactory = fn (self $that): SchemaFactoryInterface => $that->createSchemaFactoryMock($schema);
 
         yield 'JSON-Schema version' => [
             [
@@ -434,11 +435,12 @@ public function classTypeWithNullabilityDataProvider(): iterable
                     ['type' => 'null'],
                 ],
             ],
-            $schemaFactory,
+            $schemaFactoryFactory,
             $schema,
         ];
 
-        $schemaFactory = $this->createSchemaFactoryMock($schema = new Schema(Schema::VERSION_OPENAPI));
+        $schema = new Schema(Schema::VERSION_OPENAPI);
+        $schemaFactoryFactory = fn (self $that): SchemaFactoryInterface => $that->createSchemaFactoryMock($schema);
 
         yield 'OpenAPI < 3.1 version' => [
             [
@@ -447,7 +449,7 @@ public function classTypeWithNullabilityDataProvider(): iterable
                 ],
                 'nullable' => true,
             ],
-            $schemaFactory,
+            $schemaFactoryFactory,
             $schema,
         ];
     }

src/Metadata/Tests/Extractor/PropertyMetadataCompatibilityTest.php

@@ -93,7 +93,7 @@ public function testValidMetadata(string $extractorClass, PropertyAdapterInterfa
         $this->assertEquals($this->buildApiProperty(), $property);
     }
 
-    public function getExtractors(): array
+    public static function getExtractors(): array
     {
         return [
             [XmlPropertyExtractor::class, new XmlPropertyAdapter()],

src/Metadata/Tests/Extractor/ResourceMetadataCompatibilityTest.php

@@ -515,7 +515,7 @@ public function testValidMetadata(string $extractorClass, ResourceAdapterInterfa
         $this->assertEquals(new ResourceMetadataCollection(self::RESOURCE_CLASS, $this->buildApiResources()), $collection);
     }
 
-    public function getExtractors(): array
+    public static function getExtractors(): array
     {
         return [
             [XmlResourceExtractor::class, new XmlResourceAdapter()],