feat: add multiple arguments support for ARRAY

ci/phpstan/config.neon

@@ -5,12 +5,15 @@ includes:
 parameters:
     level: max
     paths:
+        - ../../ci
         - ../../src
         - ../../tests
 
-    checkMissingIterableValueType: false
     reportUnmatchedIgnoredErrors: false
+
     ignoreErrors:
+        - identifier: missingType.iterableValue
+
         - '#Parameter \#1 \$phpArray of method MartinGeorgiev\\Doctrine\\DBAL\\Types\\BaseArray::convertToDatabaseValue\(\) expects array\|null, string given.#'
         - '#Parameter \#1 \$postgresArray of method MartinGeorgiev\\Doctrine\\DBAL\\Types\\BaseArray::convertToPHPValue\(\) expects string\|null, int given.#'
         - '#Property MartinGeorgiev\\Doctrine\\ORM\\Query\\AST\\Functions\\Cast::\$sourceType \(Doctrine\\ORM\\Query\\AST\\Node\) does not accept Doctrine\\ORM\\Query\\AST\\Node\|string#'

composer.json

@@ -60,23 +60,39 @@
     },
 
     "scripts": {
+        "deptrac": [
+            "deptrac analyze --config-file=./ci/deptrac/config.yml --cache-file=./ci/deptrac/.cache --no-interaction --no-progress"
+        ],
+        "php-cs-fixer": [
+            "PHP_CS_FIXER_IGNORE_ENV=1 php-cs-fixer fix --config=./ci/php-cs-fixer/config.php --show-progress=none --no-interaction --diff -v"
+        ],
+        "phpstan": [
+            "phpstan analyse --configuration=./ci/phpstan/config.neon"
+        ],
+        "phpunit": [
+            "XDEBUG_MODE=coverage phpunit --configuration=./ci/phpunit/config.xml"
+        ],
+        "rector": [
+            "rector --config=./ci/rector/config.php --ansi --no-progress-bar"
+        ],
+
         "check-code-style": [
-            "php-cs-fixer fix --config='./ci/php-cs-fixer/config.php' --show-progress=none --dry-run --no-interaction --diff -v",
-            "rector --config='./ci/rector/config.php' --ansi --no-progress-bar --dry-run"
+            "@php-cs-fixer --dry-run",
+            "@rector --dry-run"
         ],
         "fix-code-style": [
-            "rector --config='./ci/rector/config.php' --ansi --no-progress-bar",
-            "php-cs-fixer fix --config='./ci/php-cs-fixer/config.php' --show-progress=none --no-interaction --diff -v"
+            "@rector",
+            "@php-cs-fixer"
         ],
         "run-static-analysis": [
-            "phpstan analyse --configuration='./ci/phpstan/config.neon'",
-            "deptrac analyze --config-file='./ci/deptrac/config.yml' --cache-file='./ci/deptrac/.cache' --no-interaction --no-progress"
+            "@phpstan",
+            "@deptrac"
         ],
         "run-tests": [
-            "phpunit --configuration='./ci/phpunit/config.xml'"
+            "@phpunit"
         ],
         "run-tests-with-clover": [
-            "phpunit --configuration='./ci/phpunit/config.xml' --coverage-clover './build/logs/clover.xml'"
+            "@phpunit --coverage-clover=./var/logs/clover.xml"
         ]
     },
 

src/MartinGeorgiev/Doctrine/ORM/Query/AST/Functions/Arr.php

@@ -4,6 +4,8 @@
 
 namespace MartinGeorgiev\Doctrine\ORM\Query\AST\Functions;
 
+use MartinGeorgiev\Doctrine\ORM\Query\AST\Functions\Exception\InvalidArgumentForVariadicFunctionException;
+
 /**
  * Implementation of PostgreSQL ARRAY[].
  *
@@ -12,11 +14,20 @@
  *
  * @author Martin Georgiev <martin.georgiev@gmail.com>
  */
-class Arr extends BaseFunction
+class Arr extends BaseVariadicFunction
 {
+    protected string $commonNodeMapping = 'StringPrimary';
+
     protected function customizeFunction(): void
     {
         $this->setFunctionPrototype('ARRAY[%s]');
-        $this->addNodeMapping('StringPrimary');
+    }
+
+    protected function validateArguments(array $arguments): void
+    {
+        $argumentCount = \count($arguments);
+        if ($argumentCount === 0) {
+            throw InvalidArgumentForVariadicFunctionException::atLeast('ARRAY', 1);
+        }
     }
 }

tests/MartinGeorgiev/Doctrine/ORM/Query/AST/Functions/ArrTest.php

@@ -0,0 +1,42 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\MartinGeorgiev\Doctrine\ORM\Query\AST\Functions;
+
+use Fixtures\MartinGeorgiev\Doctrine\Entity\ContainsArrays;
+use MartinGeorgiev\Doctrine\ORM\Query\AST\Functions\Arr;
+
+class ArrTest extends TestCase
+{
+    protected function getStringFunctions(): array
+    {
+        return [
+            'ARRAY' => Arr::class,
+        ];
+    }
+
+    protected function getExpectedSqlStatements(): array
+    {
+        return [
+            // Multiple literal values
+            "SELECT ARRAY['foo', 'bar', 'baz'] AS sclr_0 FROM ContainsArrays c0_",
+            // Column references
+            'SELECT ARRAY[c0_.array1] AS sclr_0 FROM ContainsArrays c0_',
+            // Mix of column references and literals
+            "SELECT ARRAY[c0_.array1, 'test-value', c0_.array2] AS sclr_0 FROM ContainsArrays c0_",
+        ];
+    }
+
+    protected function getDqlStatements(): array
+    {
+        return [
+            // Multiple literal values
+            \sprintf("SELECT ARRAY('foo', 'bar', 'baz') FROM %s e", ContainsArrays::class),
+            // Column references
+            \sprintf('SELECT ARRAY(e.array1) FROM %s e', ContainsArrays::class),
+            // Mix of column references and literals
+            \sprintf("SELECT ARRAY(e.array1, 'test-value', e.array2) FROM %s e", ContainsArrays::class),
+        ];
+    }
+}