Merge pull request #102 from laminas/renovate/lock-file-maintenance

Removed internal redundant type checks, updated `vimeo/psalm`

psalm-baseline.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="5.1.0@4defa177c89397c5e14737a80fe4896584130674">
+<files psalm-version="5.4.0@62db5d4f6a7ae0a20f7cc5a4952d730272fc0863">
   <file src="src/Aggregate/AggregateHydrator.php">
     <DocblockTypeContradiction occurrences="1">
       <code>null === $this-&gt;eventManager</code>

src/HydratorPluginManagerFactory.php

@@ -46,9 +46,7 @@ public function __invoke(ContainerInterface $container, string $name, ?array $op
             ));
         }
 
-        /** @psalm-var ServiceManagerConfiguration $options */
-        $options       = is_array($options) ? $options : [];
-        $pluginManager = new HydratorPluginManager($container, $options);
+        $pluginManager = new HydratorPluginManager($container, $options ?? []);
 
         // If this is in a laminas-mvc application, the ServiceListener will inject
         // merged configuration during bootstrap.

test/Strategy/CollectionStrategyTest.php

@@ -9,11 +9,9 @@
 use Laminas\Hydrator\HydratorInterface;
 use Laminas\Hydrator\ReflectionHydrator;
 use Laminas\Hydrator\Strategy\CollectionStrategy;
-use Laminas\Hydrator\Strategy\StrategyInterface;
 use LaminasTest\Hydrator\TestAsset;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
-use ReflectionClass;
 use stdClass;
 use Throwable;
 use TypeError;
@@ -35,17 +33,6 @@
  */
 class CollectionStrategyTest extends TestCase
 {
-    public function testImplementsStrategyInterface(): void
-    {
-        $reflection = new ReflectionClass(CollectionStrategy::class);
-
-        self::assertTrue($reflection->implementsInterface(StrategyInterface::class), sprintf(
-            'Failed to assert that "%s" implements "%s"',
-            CollectionStrategy::class,
-            StrategyInterface::class
-        ));
-    }
-
     /**
      * @dataProvider providerInvalidObjectClassName
      * @param class-string<Throwable> $expectedExceptionType

test/Strategy/HydratorStrategyTest.php

@@ -45,10 +45,9 @@ public function testConstructorRejectsInvalidObjectClassName(
         );
     }
 
-    /** @return array<string, array{0: mixed, class-string<Throwable>, string}> */
+    /** @return non-empty-array<non-empty-string, array{mixed, class-string<Throwable>, string}> */
     public function providerInvalidObjectClassName(): array
     {
-        // @codingStandardsIgnoreStart
         return [
             'array'                     => [[], TypeError::class, 'type string'],
             'boolean-false'             => [false, TypeError::class, 'type string'],
@@ -58,9 +57,12 @@ public function providerInvalidObjectClassName(): array
             'null'                      => [null, TypeError::class, 'type string'],
             'object'                    => [new stdClass(), TypeError::class, 'type string'],
             'resource'                  => [fopen(__FILE__, 'r'), TypeError::class, 'type string'],
-            'string-non-existent-class' => ['FooBarBaz9000', InvalidArgumentException::class, 'class name needs to be the name of an existing class'],
+            'string-non-existent-class' => [
+                'FooBarBaz9000',
+                InvalidArgumentException::class,
+                'class name needs to be the name of an existing class',
+            ],
         ];
-        // @codingStandardsIgnoreEnd
     }
 
     /**