Update Yaml configuration loader to parse navigation items

Author: emmadesilva

packages/framework/src/Foundation/Internal/LoadYamlConfiguration.php

@@ -8,6 +8,7 @@
 use Illuminate\Support\Arr;
 use Hyde\Foundation\Application;
 use Illuminate\Config\Repository;
+use Hyde\Framework\Features\Navigation\NavigationItem;
 use Hyde\Framework\Features\Blogging\Models\PostAuthor;
 use Hyde\Framework\Exceptions\InvalidConfigurationException;
 
@@ -50,6 +51,10 @@ protected function mergeParsedConfiguration(): void
                 $data['authors'] = $this->parseAuthors($data['authors']);
             }
 
+            if ($namespace === 'hyde' && isset($data['navigation']['custom'])) {
+                $data['navigation']['custom'] = $this->parseNavigationItems($data['navigation']['custom']);
+            }
+
             $this->mergeConfiguration($namespace, Arr::undot($data ?: []));
         }
     }
@@ -76,4 +81,15 @@ protected function parseAuthors(array $authors): array
             }
         });
     }
+
+    /**
+     * @param array<array{destination: string, label: ?string, priority: ?int}> $items Where destination is a route key or an external URI.
+     * @return array<\Hyde\Framework\Features\Navigation\NavigationItem>
+     */
+    protected function parseNavigationItems(array $items): array
+    {
+        return Arr::map($items, function (array $item): NavigationItem {
+            return NavigationItem::create($item['destination'], $item['label'], $item['priority']);
+        });
+    }
 }

packages/framework/tests/Feature/YamlConfigurationFeatureTest.php

@@ -6,6 +6,7 @@
 
 use Hyde\Testing\TestCase;
 use Illuminate\Support\Env;
+use Hyde\Framework\Features\Navigation\NavigationItem;
 use Hyde\Framework\Features\Blogging\Models\PostAuthor;
 use Hyde\Framework\Exceptions\InvalidConfigurationException;
 
@@ -456,6 +457,43 @@ public function testTypeErrorsInAuthorsYamlConfigAreRethrownMoreHelpfully()
         unlink('hyde.yml');
     }
 
+    public function testCanSetCustomNavigationItemsInTheYamlConfig()
+    {
+        $this->file('hyde.yml', <<<'YAML'
+        hyde:
+          navigation:
+            custom:
+              - destination: 'https://example.com'
+                label: 'Example'
+                priority: 100
+              - destination: 'about'
+                label: 'About Us'
+                priority: 200
+              - destination: 'contact'
+                label: 'Contact'
+                priority: 300
+        YAML);
+
+        $this->runBootstrappers();
+
+        $navigationItems = config('hyde.navigation.custom');
+
+        $this->assertCount(3, $navigationItems);
+        $this->assertContainsOnlyInstancesOf(NavigationItem::class, $navigationItems);
+
+        $this->assertSame('https://example.com', $navigationItems[0]->destination);
+        $this->assertSame('Example', $navigationItems[0]->label);
+        $this->assertSame(100, $navigationItems[0]->priority);
+
+        $this->assertSame('about', $navigationItems[1]->destination);
+        $this->assertSame('About Us', $navigationItems[1]->label);
+        $this->assertSame(200, $navigationItems[1]->priority);
+
+        $this->assertSame('contact', $navigationItems[2]->destination);
+        $this->assertSame('Contact', $navigationItems[2]->label);
+        $this->assertSame(300, $navigationItems[2]->priority);
+    }
+
     protected function runBootstrappers(?array $withMergedConfig = null): void
     {
         $this->refreshApplication();