Add tests

Author: GromNaN

Diff for: src/Configuration.php

@@ -9,6 +9,14 @@
 
 final readonly class Configuration
 {
+    /**
+     * @var list<class-string<\ArrayAccess<string, mixed>>>
+     */
+    public array $arrayAccessClasses;
+
+    /**
+     * @param list<class-string<\ArrayAccess<string, mixed>>> $arrayAccessClasses classes with unknown properties, implemeting ArrayAccess
+     */
     public function __construct(
         /**
          * Class prefix used to prefix the class name of the generated mappers.
@@ -39,17 +47,16 @@ public function __construct(
          * Does the mapper should throw an exception if the target is read-only.
          */
         public bool $allowReadOnlyTargetToPopulate = false,
-        /**
-         * Classes with unknown properties, implemeting ArrayAccess.
-         *
-         * @var list<class-string<\ArrayAccess<string, mixed>>> $arrayAccessClasses
-         */
-        public array $arrayAccessClasses = [],
+        array $arrayAccessClasses = [],
     ) {
-        $this->arrayAccessClasses[] = \ArrayObject::class;
+        $arrayAccessClasses[] = \ArrayObject::class;
+
+        // Classes provided by the mongodb extension and mongodb/mongodb package
         if (class_exists(Document::class, false)) {
-            $this->arrayAccessClasses[] = Document::class;
-            $this->arrayAccessClasses[] = BSONDocument::class;
+            $arrayAccessClasses[] = Document::class;
+            $arrayAccessClasses[] = BSONDocument::class;
         }
+
+        $this->arrayAccessClasses = $arrayAccessClasses;
     }
 }

Diff for: src/Symfony/Bundle/DependencyInjection/AutoMapperExtension.php

@@ -65,6 +65,7 @@ public function load(array $configs, ContainerBuilder $container): void
             ->setArgument('$autoRegister', $config['auto_register'])
             ->setArgument('$mapPrivateProperties', $config['map_private_properties'])
             ->setArgument('$allowReadOnlyTargetToPopulate', $config['allow_readonly_target_to_populate'])
+            ->setArgument('$arrayAccessClasses', $config['array_access_classes'])
         ;
 
         $container->registerForAutoconfiguration(PropertyTransformerInterface::class)->addTag('automapper.property_transformer');

Diff for: src/Symfony/Bundle/DependencyInjection/Configuration.php

@@ -31,6 +31,14 @@ public function getConfigTreeBuilder(): TreeBuilder
                 ->booleanNode('check_attributes')->defaultTrue()->end()
                 ->booleanNode('auto_register')->defaultTrue()->end()
                 ->booleanNode('map_private_properties')->defaultTrue()->end()
+                ->arrayNode('array_access_classes')
+                    ->scalarPrototype()
+                        ->validate()
+                            ->ifTrue(fn ($className) => !is_subclass_of($className, \ArrayAccess::class))
+                            ->thenInvalid('Class %s does not implement "ArrayAccess"')
+                        ->end()
+                    ->end()
+                ->end()
                 ->booleanNode('allow_readonly_target_to_populate')->defaultFalse()->end()
                 ->arrayNode('normalizer')
                     ->children()

Diff for: tests/AutoMapperTest.php

@@ -210,6 +210,41 @@ public function testAutoMapperStdObjectToStdObject(): void
         self::assertEquals($user, $userStd);
     }
 
+    public function testAutoMapperFromArrayObject(): void
+    {
+        $this->buildAutoMapper(mapPrivatePropertiesAndMethod: true);
+
+        $user = new \ArrayObject(['id' => 1]);
+
+        /** @var Fixtures\UserDTO $userDto */
+        $userDto = $this->autoMapper->map($user, Fixtures\UserDTO::class);
+
+        self::assertInstanceOf(Fixtures\UserDTO::class, $userDto);
+        self::assertEquals(1, $userDto->id);
+    }
+
+    public function testAutoMapperToArrayObject(): void
+    {
+        $userDto = new Fixtures\UserDTO();
+        $userDto->id = 1;
+
+        $user = $this->autoMapper->map($userDto, \ArrayObject::class);
+
+        self::assertInstanceOf(\ArrayObject::class, $user);
+        self::assertEquals(1, $user['id']);
+    }
+
+    public function testAutoMapperArrayObjectToArrayObject(): void
+    {
+        $user = new \ArrayObject(['id' => 1, 'nested' => new \ArrayObject(['id' => 2])]);
+        $userStd = $this->autoMapper->map($user, \ArrayObject::class);
+
+        self::assertInstanceOf(\ArrayObject::class, $userStd);
+        self::assertNotSame($user, $userStd);
+        self::assertNotSame($user['nested'], $userStd['nested']);
+        self::assertEquals($user, $userStd);
+    }
+
     public function testNotReadable(): void
     {
         $this->buildAutoMapper(classPrefix: 'CustomDateTime_');

Diff for: tools/phpstan/phpstan.neon

@@ -2,11 +2,11 @@ parameters:
 	level: max
 	paths:
 		- ../../src/
-
-	tmpDir: cache
 	scanFiles:
 		- stubs/mongodb.php
 
+	tmpDir: cache
+
 	ignoreErrors:
 		- "#Instantiated class fromIteratorToArray not found#"
 		- "#Instantiated class toArray not found#"