Add containerMenu and containerMenuTag properties to HasContainerMenu…

… trait. (#186)
php-forge · Dec 31, 2023 · fbecc51 · fbecc51
1 parent bcf5a84
commit fbecc51
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 14 deletions.
diff --git a/src/Attribute/Custom/HasContainerMenu.php b/src/Attribute/Custom/HasContainerMenu.php
@@ -12,7 +12,9 @@
  */
 trait HasContainerMenu
 {
+    protected bool $containerMenu = true;
     protected array $containerMenuAttributes = [];
+    protected string $containerMenuTag = 'nav';
 
     /**
      * Enable or disable the container menu tag.

diff --git a/tests/Attribute/Custom/HasContainerMenuTest.php b/tests/Attribute/Custom/HasContainerMenuTest.php
@@ -15,9 +15,6 @@ public function testClass(): void
         $instance = new class () {
             use HasContainerMenu;
 
-            protected bool $containerMenu = true;
-            protected string $containerMenuTag = 'div';
-
             public function getContainerMenuClass(): string
             {
                 return $this->containerMenuAttributes['class'] ?? '';
@@ -26,26 +23,23 @@ public function getContainerMenuClass(): string
 
         $this->assertEmpty($instance->getContainerMenuClass());
 
-        $instance = $instance->containerMenuClass('test-class');
+        $instance = $instance->containerMenuClass('class');
 
-        $this->assertSame('test-class', $instance->getContainerMenuClass());
+        $this->assertSame('class', $instance->getContainerMenuClass());
 
-        $instance = $instance->containerMenuClass('test-class-1');
+        $instance = $instance->containerMenuClass('class-1');
 
-        $this->assertSame('test-class test-class-1', $instance->getContainerMenuClass());
+        $this->assertSame('class class-1', $instance->getContainerMenuClass());
 
-        $instance = $instance->containerMenuClass('test-override-class', true);
+        $instance = $instance->containerMenuClass('override-class', true);
 
-        $this->assertSame('test-override-class', $instance->getContainerMenuClass());
+        $this->assertSame('override-class', $instance->getContainerMenuClass());
     }
 
     public function testException(): void
     {
         $instance = new class () {
             use HasContainerMenu;
-
-            protected bool $containerMenu = true;
-            protected string $containerMenuTag = '';
         };
 
         $this->expectException(InvalidArgumentException::class);
@@ -58,8 +52,6 @@ public function testImmutability(): void
     {
         $instance = new class () {
             use HasContainerMenu;
-
-            protected string $containerMenuTag = 'div';
         };
 
         $this->assertNotSame($instance, $instance->containerMenu(true));