Fix config validation is keys are used defining the "root_sass" paths

SymfonyCasts · Nov 23, 2023 · 32a8ca3 · 32a8ca3
1 parent b942232
commit 32a8ca3
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/DependencyInjection/SymfonycastsSassExtension.php b/src/DependencyInjection/SymfonycastsSassExtension.php
@@ -61,7 +61,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                         ->end()
                     ->validate()
                         ->ifTrue(static function (array $paths): bool {
-                            if (1 === \count($paths)) {
+                            if (1 === \count($paths) || !array_is_list($paths)) {
                                 return false;
                             }
 

diff --git a/tests/ConfigurationTest.php b/tests/ConfigurationTest.php
@@ -47,6 +47,18 @@ public function testMultipleSassRootPaths(): void
         ]);
     }
 
+    public function testMultipleSassRootPathsWithIdentifier(): void
+    {
+        $this->assertConfigurationIsValid([
+            'symfonycasts_sass' => [
+                'root_sass' => [
+                    'website' => '%kernel.project_dir%/assets/scss/app.scss',
+                    'admin' => '%kernel.project_dir%/assets/admin/scss/app.scss',
+                ],
+            ],
+        ]);
+    }
+
     public function testMultipleSassRootPathsWithSameFilename(): void
     {
         $this->assertConfigurationIsInvalid([