Add failing test case for constructor + readonly + list + constructor_arguments

castor.php

@@ -15,15 +15,15 @@ function qa_cs_check()
     ]);
 }
 
-#[AsTask('cs:fix', namespace: 'qa', description: 'Fix all coding standards')]
+#[AsTask('cs:fix', namespace: 'qa', description: 'Fix all coding standards', aliases: ['cs'])]
 function qa_cs_fix()
 {
     php_cs_fixer(['fix', '--config', __DIR__ . '/.php-cs-fixer.php'], '3.50', [
         'kubawerlos/php-cs-fixer-custom-fixers' => '^3.21',
     ]);
 }
 
-#[AsTask('phpstan', namespace: 'qa', description: 'Run PHPStan for static analysis')]
+#[AsTask('phpstan', namespace: 'qa', description: 'Run PHPStan for static analysis', aliases: ['phpstan'])]
 function qa_phpstan()
 {
     phpstan(['analyse', '--configuration', __DIR__ . '/phpstan.neon', '--memory-limit=-1'], '1.11.1');

tests/AutoMapperTest.php

@@ -519,6 +519,30 @@ public function testConstructorAndRelationMissing2(): void
         self::assertSame(30, $userDto->age);
     }
 
+    public function testConstructorWithRelationAndReadonlyAndList(): void
+    {
+        $entity = new Fixtures\ConstructorWithRelationAndReadonlyAndList\Entity(
+            'my id',
+            ['fr', 'en'],
+            ['page1' => ['foo'], 'page2' => ['bar']],
+        );
+
+        /** @var Fixtures\ConstructorWithRelationAndReadonlyAndList\Target $target */
+        $target = $this->autoMapper->map($entity, Fixtures\ConstructorWithRelationAndReadonlyAndList\Target::class, [
+            MapperContext::CONSTRUCTOR_ARGUMENTS => [
+                Fixtures\ConstructorWithRelationAndReadonlyAndList\Target::class => [
+                    'entity' => $entity,
+                ],
+            ],
+        ]);
+
+        $this->assertInstanceOf(Fixtures\ConstructorWithRelationAndReadonlyAndList\Target::class, $target);
+        $this->assertInstanceOf(Fixtures\ConstructorWithRelationAndReadonlyAndList\Entity::class, $target->getEntity());
+        $this->assertSame('my id', $target->id);
+        $this->assertSame(['fr', 'en'], $target->locales);
+        $this->assertSame(['page1' => ['foo'], 'page2' => ['bar']], $target->pages);
+    }
+
     public function testConstructorAndRelationMissingAndContext(): void
     {
         $user = ['name' => 'foo'];

tests/Fixtures/ConstructorWithRelationAndReadonlyAndList/Entity.php

@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AutoMapper\Tests\Fixtures\ConstructorWithRelationAndReadonlyAndList;
+
+class Entity
+{
+    public function __construct(
+        public string $id,
+        public array $locales,
+        public array $pages,
+    ) {
+    }
+}

tests/Fixtures/ConstructorWithRelationAndReadonlyAndList/Target.php

@@ -0,0 +1,24 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AutoMapper\Tests\Fixtures\ConstructorWithRelationAndReadonlyAndList;
+
+class Target
+{
+    /**
+     * @param list<string> $locales
+     */
+    public function __construct(
+        private readonly Entity $entity,
+        public readonly string $id,
+        public array $locales,
+        public array $pages,
+    ) {
+    }
+
+    public function getEntity(): Entity
+    {
+        return $this->entity;
+    }
+}