Adds test for propagating styles to SplitItem + SubMenu

Author: jtreminio

src/Builder/CliMenuBuilder.php

@@ -555,9 +555,12 @@ public function build() : CliMenu
      * Pass styles from current menu to sub-menu
      * only if sub-menu style has not be customized
      */
-    private function propagateStyles(CliMenu $menu)
+    private function propagateStyles(CliMenu $menu, array $items = [])
     {
-        foreach ($menu->getItems() as $item) {
+        $currentItems = !empty($items) ? $items : $menu->getItems();
+
+        foreach ($currentItems as $item) {
+            // Apply current style to children, if they are not customized
             if ($item instanceof MenuMenuItem) {
                 $subMenu = $item->getSubMenu();
 
@@ -566,6 +569,11 @@ private function propagateStyles(CliMenu $menu)
 
                 $this->propagateStyles($subMenu);
             }
+
+            // Apply styles to SplitItem children using current $menu
+            if ($item instanceof SplitItem) {
+                $this->propagateStyles($menu, $item->getItems());
+            }
         }
     }
 }

test/Builder/CliMenuBuilderTest.php

@@ -11,6 +11,7 @@
 use PhpSchool\CliMenu\MenuItem\MenuMenuItem;
 use PhpSchool\CliMenu\MenuItem\RadioItem;
 use PhpSchool\CliMenu\MenuItem\SelectableItem;
+use PhpSchool\CliMenu\MenuItem\SplitItem;
 use PhpSchool\CliMenu\MenuItem\StaticItem;
 use PhpSchool\Terminal\Terminal;
 use PHPUnit\Framework\TestCase;
@@ -606,6 +607,37 @@ public function testSubMenuInheritsParentsStyle() : void
         self::assertEquals($menu->getStyle(), $subMenu2->getStyle());
     }
 
+    public function testSplitItemSubMenuInheritsParentsStyle() : void
+    {
+        $terminal = self::createMock(Terminal::class);
+        $terminal
+            ->expects($this->any())
+            ->method('getWidth')
+            ->will($this->returnValue(200));
+
+        $menu = (new CliMenuBuilder($terminal))
+            ->setBackgroundColour('green')
+            ->addSplitItem(function (SplitItemBuilder $b) {
+                $b
+                    ->addItem('Item 1', function () {})
+                    ->addSubMenu('Submenu 1', function (CliMenuBuilder $b) {
+                        $b->addItem('Item 2', function () {});
+                    })
+                ;
+            })
+            ->build();
+
+        /** @var SplitItem $splitItem */
+        $splitItem = $menu->getItems()[0];
+        /** @var SelectableItem $selectableItem1 */
+        $selectableItem1 = $splitItem->getItems()[0];
+        /** @var CliMenu $subMenu */
+        $subMenu = $splitItem->getItems()[1]->getSubMenu();
+
+        self::assertSame('green', $subMenu->getStyle()->getBg());
+        self::assertEquals($menu->getStyle(), $subMenu->getStyle());
+    }
+
     public function testSubMenuIgnoresParentsStyleIfCustomAndPassesToChildren() : void
     {
         $terminal = self::createMock(Terminal::class);