Rebrand documentation intelligence to code intelligence

caendesilva · caendesilva · commit a22875d757b7 · 2024-03-15T14:32:07.000+01:00
diff --git a/monorepo/CodeIntelligence/CodeIntelligence.php b/monorepo/CodeIntelligence/CodeIntelligence.php
@@ -34,7 +34,7 @@
     $this->info('Generating documentation intelligence...');
     $this->line();
 
-    $generator = new DocumentationIntelligence();
+    $generator = new CodeIntelligence();
 
     task('discover pages', fn () => $generator->discoverPages());
     task('assemble model', fn () => $generator->assembleModel());
@@ -71,7 +71,7 @@
     return 0;
 });
 
-class DocumentationIntelligence
+class CodeIntelligence
 {
     protected HydeKernel $kernel;
 
diff --git a/monorepo/CodeIntelligence/CodeIntelligence/.gitignore b/monorepo/CodeIntelligence/CodeIntelligence/.gitignore
diff --git a/monorepo/CodeIntelligence/README.md b/monorepo/CodeIntelligence/README.md
@@ -1,6 +1,6 @@
-# Internal Documentation Intelligence
+# Internal Code and Documentation Intelligence
 
-This internal monorepo module contains tools to analyse the documentation to improve its quality.
+This internal monorepo module contains tools to analyse the documentation and codebase to improve its quality.
 All code here is internal and should not be used outside the monorepo and no support will be provided for it.
 
 ## Usage
@@ -9,5 +9,5 @@ As the resulting data models and dashboards can get quite large, they are exclud
 To generate them, run the following command:
 
 ```bash
-php monorepo/DocumentationIntelligence/DocumentationIntelligence.php
+php monorepo/CodeIntelligence/CodeIntelligence.php
 ```
diff --git a/monorepo/CodeIntelligence/dashboard-template.php b/monorepo/CodeIntelligence/dashboard-template.php
@@ -3,7 +3,7 @@
 <head>
     <meta charset="utf-8">
     <meta name="viewport" content="width=device-width, initial-scale=1">
-    <title>HydePHP Documentation Intelligence Dashboard</title>
+    <title>HydePHP Code Intelligence Dashboard</title>
     <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.2.3/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-rbsA2VBKQhggwzxH7pPCaAqO46MgnOM80zW1RWuH61DGLwZJEdK2Kadq2F9CUG65" crossorigin="anonymous">
     <style>
         .w-fit {
@@ -16,9 +16,9 @@
 <header class="container">
     <div class="row">
         <div class="col-12 py-4 text-center">
-            <h1>HydePHP Documentation Intelligence Dashboard</h1>
+            <h1>HydePHP Code Intelligence Dashboard</h1>
             <p class="lead">
-                This internal monorepo module contains tools to analyse the documentation to improve its quality.
+                This internal monorepo module contains tools to analyse the codebase and documentation to improve its quality.
             </p>
         </div>
     </div>
diff --git a/packages/framework/tests/Unit/Views/NavigationLinkViewTest.php b/packages/framework/tests/Unit/Views/NavigationLinkViewTest.php
@@ -4,7 +4,11 @@
 
 namespace Hyde\Framework\Testing\Unit\Views;
 
-use Hyde\Foundation\Facades\Routes;
+use Hyde\Pages\InMemoryPage;
+use Hyde\Support\Models\Route;
+use Hyde\Testing\TestsBladeViews;
+use Hyde\Testing\Support\TestView;
+use Illuminate\View\ComponentAttributeBag;
 use Hyde\Framework\Features\Navigation\NavItem;
 use Hyde\Testing\TestCase;
 
@@ -13,39 +17,67 @@
  */
 class NavigationLinkViewTest extends TestCase
 {
+    use TestsBladeViews;
+
     protected function setUp(): void
     {
         parent::setUp();
         $this->mockRoute();
         $this->mockPage();
     }
 
-    protected function render(?NavItem $item = null): string
+    protected function test(): TestView
     {
-        return view('hyde::components.navigation.navigation-link', [
-            'item' => $item ?? NavItem::forLink('foo.html', 'Foo'),
-        ])->render();
+        return $this->test(view('hyde::components.navigation.navigation-link', [
+            'item' => NavItem::forRoute(new Route(new InMemoryPage('foo')), 'Foo'),
+            'attributes' => new ComponentAttributeBag(),
+        ]));
     }
 
     public function testComponentLinksToRouteDestination()
     {
-        $this->assertStringContainsString('href="foo.html"', $this->render());
+        $this->test()->assertAttributeIs('href', 'foo.html');
+    }
+
+    public function testComponentResolvesRelativeLinksForRoutes()
+    {
+        $this->mockCurrentPage('foo/bar');
+        $this->test()->assertAttributeIs('href', '../foo.html');
     }
 
     public function testComponentUsesTitle()
     {
-        $this->assertStringContainsString('Foo', $this->render());
+        $this->test()->assertTextIs('Foo');
+    }
+
+    public function testComponentDoesNotHaveCurrentAttributesWhenCurrentRouteDoesNotMatch()
+    {
+        $this->test()
+            ->assertDontSee('current')
+            ->assertDoesNotHaveAttribute('aria-current');
     }
 
     public function testComponentIsCurrentWhenCurrentRouteMatches()
     {
-        $this->mockRoute(Routes::get('index'));
-        $this->assertStringContainsString('current', $this->render(NavItem::forRoute(Routes::get('index'), 'Home')));
+        $this->mockCurrentPage('foo')
+            ->test()
+            ->assertSee('current')
+            ->assertHasAttribute('aria-current')
+            ->assertAttributeIs('aria-current="page"');
+    }
+
+    public function testComponentDoesNotHaveActiveClassWhenNotActive()
+    {
+        $this->test()
+            ->assertSee('navigation-link ')
+            ->assertDontSee('navigation-link-active');
     }
 
-    public function testComponentHasAriaCurrentWhenCurrentRouteMatches()
+    public function testComponentHasActiveClassWhenActive()
     {
-        $this->mockRoute(Routes::get('index'));
-        $this->assertStringContainsString('aria-current="page"', $this->render(NavItem::forRoute(Routes::get('index'), 'Home')));
+        $this->mockCurrentPage('foo')
+            ->test()
+            ->assertSee('navigation-link ')
+            ->assertSee('navigation-link-active');
     }
 }
