Update dependency slam/php-cs-fixer-extensions to ^3.15.0 (#94)

* Update dependency slam/php-cs-fixer-extensions to ^3.15.0

| datasource | package                      | from   | to     |
| ---------- | ---------------------------- | ------ | ------ |
| packagist  | slam/php-cs-fixer-extensions | 3.14.0 | 3.15.0 |

* CS Fix

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Filippo Tessarotto <zoeslam@gmail.com>

Author: renovate[bot]

.php-cs-fixer.php

@@ -1,6 +1,9 @@
 <?php
 
-$config = new SlamCsFixer\Config();
+$config = new SlamCsFixer\Config([
+    'php_unit_data_provider_name' => false,
+    'php_unit_data_provider_return_type' => false,
+]);
 $config->getFinder()
     ->in(__DIR__ . '/src')
     ->in(__DIR__ . '/tests')

Dockerfile

@@ -1,4 +1,4 @@
-FROM php:8.5
+FROM php:8.4
 
 ADD --chmod=0755 https://github.com/mlocati/docker-php-extension-installer/releases/latest/download/install-php-extensions /usr/local/bin/
 

composer.json

@@ -33,7 +33,7 @@
         "phpstan/phpstan-deprecation-rules": "^2.0.4",
         "phpstan/phpstan-phpunit": "^2.0.16",
         "phpunit/phpunit": "^11.5.55",
-        "slam/php-cs-fixer-extensions": "^3.14.0"
+        "slam/php-cs-fixer-extensions": "^3.15.0"
     },
     "conflict": {
         "laminas/laminas-filter": "<2.41",

phpunit.xml

@@ -1,8 +1,9 @@
 <?xml version="1.0"?>
-<phpunit
-        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-        colors="true"
-        xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
+    colors="true"
+    failOnAllIssues="true"
+    displayDetailsOnAllIssues="true"
 >
   <coverage>
     <report>

src/Rules/Laminas/ServiceManagerGetMethodCallRule.php

@@ -98,8 +98,8 @@ public function processNode(Node $node, Scope $scope): array
     /** @phpstan-assert-if-true ObjectType $type */
     private function isTypeInstanceOfContainer(Type $type): bool
     {
-        return (new ObjectType(ServiceLocatorInterface::class))->isSuperTypeOf($type)->yes()
-            || (new ObjectType(InteropContainerInterface::class))->isSuperTypeOf($type)->yes()
-            || (new ObjectType(PsrContainerInterface::class))->isSuperTypeOf($type)->yes();
+        return new ObjectType(ServiceLocatorInterface::class)->isSuperTypeOf($type)->yes()
+            || new ObjectType(InteropContainerInterface::class)->isSuperTypeOf($type)->yes()
+            || new ObjectType(PsrContainerInterface::class)->isSuperTypeOf($type)->yes();
     }
 }

tests/LaminasIntegration/IntegrationTest.php

@@ -5,12 +5,18 @@
 namespace LaminasPhpStan\Tests\LaminasIntegration;
 
 use PHPStan\Testing\LevelsTestCase;
+use PHPUnit\Framework\Attributes\CoversNothing;
+use PHPUnit\Framework\Attributes\DataProvider;
 
-/**
- * @coversNothing
- */
+#[CoversNothing]
 final class IntegrationTest extends LevelsTestCase
 {
+    #[DataProvider('dataTopics')]
+    public function testLevels(string $topic): void
+    {
+        parent::testLevels($topic);
+    }
+
     public static function dataTopics(): array
     {
         return [

tests/Rules/Laminas/ServiceManagerGetMethodCallRuleTest.php

@@ -8,17 +8,19 @@
 use Laminas\ServiceManager\ServiceManager;
 use LaminasPhpStan\Rules\Laminas\ServiceManagerGetMethodCallRule;
 use LaminasPhpStan\ServiceManagerLoader;
+use LaminasPhpStan\UnmappedAliasServiceLocatorProxy;
 use PhpParser\Node\Expr\MethodCall;
 use PHPStan\Rules\Rule;
 use PHPStan\Testing\RuleTestCase;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\DataProvider;
 use Psr\Container\ContainerInterface as PsrContainerInterface;
 
 /**
- * @covers \LaminasPhpStan\Rules\Laminas\ServiceManagerGetMethodCallRule
- * @covers \LaminasPhpStan\UnmappedAliasServiceLocatorProxy
- *
  * @extends RuleTestCase<ServiceManagerGetMethodCallRule>
  */
+#[CoversClass(ServiceManagerGetMethodCallRule::class)]
+#[CoversClass(UnmappedAliasServiceLocatorProxy::class)]
 final class ServiceManagerGetMethodCallRuleTest extends RuleTestCase
 {
     private ServiceManagerLoader $serviceManagerLoader;
@@ -34,7 +36,7 @@ protected function getRule(): Rule
         return new ServiceManagerGetMethodCallRule($this->createReflectionProvider(), $this->serviceManagerLoader);
     }
 
-    /** @dataProvider provideRuleCases */
+    #[DataProvider('provideRuleCases')]
     public function testRule(string $filename, string $containerClassname): void
     {
         $this->analyse([__DIR__ . '/ServiceManagerGetMethodCallRule/' . $filename], [

tests/Type/Laminas/ObjectServiceManagerTypeTest.php

@@ -5,12 +5,11 @@
 namespace LaminasPhpStan\Tests\Type\Laminas;
 
 use LaminasPhpStan\Type\Laminas\ObjectServiceManagerType;
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 use stdClass;
 
-/**
- * @covers \LaminasPhpStan\Type\Laminas\ObjectServiceManagerType
- */
+#[CoversClass(ObjectServiceManagerType::class)]
 final class ObjectServiceManagerTypeTest extends TestCase
 {
     public function testTrivialUsage(): void

tests/Type/Laminas/PluginMethodReflectionTest.php

@@ -9,11 +9,10 @@
 use PHPStan\Testing\PHPStanTestCase;
 use PHPStan\TrinaryLogic;
 use PHPStan\Type\ObjectType;
+use PHPUnit\Framework\Attributes\CoversClass;
 use stdClass;
 
-/**
- * @covers \LaminasPhpStan\Type\Laminas\PluginMethodReflection
- */
+#[CoversClass(PluginMethodReflection::class)]
 final class PluginMethodReflectionTest extends PHPStanTestCase
 {
     public function testTrivialUsage(): void

tests/Type/Laminas/ServiceManagerLoaderTest.php

@@ -10,11 +10,10 @@
 use LaminasPhpStan\TestAsset\BarService;
 use LaminasPhpStan\TestAsset\FooService;
 use PHPStan\ShouldNotHappenException;
+use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\TestCase;
 
-/**
- * @covers \LaminasPhpStan\ServiceManagerLoader
- */
+#[CoversClass(ServiceManagerLoader::class)]
 final class ServiceManagerLoaderTest extends TestCase
 {
     public function testLoaderMustBeAValidFile(): void