Merge pull request #63 from robertboloc/feature-5

Detect Ascii art that does not fit in the menu

Author: mikeymike

src/CliMenuBuilder.php

@@ -180,7 +180,11 @@ public function addAsciiArt($art, $position = AsciiArtItem::POSITION_CENTER)
         Assertion::string($art);
         Assertion::string($position);
 
-        $this->addMenuItem(new AsciiArtItem($art, $position));
+        $asciiArtItem = new AsciiArtItem($art, $position);
+
+        if ($asciiArtItem->getArtLength() <= $this->getMenuStyle()->getContentWidth()) {
+            $this->addMenuItem($asciiArtItem);
+        }
 
         return $this;
     }

src/MenuItem/AsciiArtItem.php

@@ -116,6 +116,16 @@ public function getText()
         return $this->text;
     }
 
+    /**
+     * Return the length of the art
+     *
+     * @return int
+     */
+    public function getArtLength()
+    {
+        return $this->artLength;
+    }
+
     /**
      * Whether or not the menu item is showing the menustyle extra value
      *

test/CliMenuBuilderTest.php

@@ -247,6 +247,18 @@ public function testAsciiArtWithSpecificPosition()
         $this->checkItems($menu, $expected);
     }
 
+    public function testAddAsciiArtDetectsArtThatDoesNotFitAndSkipsIt()
+    {
+        $builder = new CliMenuBuilder;
+        $builder->setWidth(1);
+        $builder->addAsciiArt("//\n//", AsciiArtItem::POSITION_LEFT);
+        $menu = $builder->build();
+
+        foreach ($menu->getItems() as $menuItem) {
+            $this->assertNotInstanceOf(AsciiArtItem::class, $menuItem);
+        }
+    }
+
     public function testEndThrowsExceptionIfNoParentBuilder()
     {
         $builder = new CliMenuBuilder;

test/MenuItem/AsciiArtItemTest.php

@@ -50,6 +50,12 @@ public function testGetText()
         $this->assertEquals('////\\\\', $item->getText());
     }
 
+    public function testGetArtLength()
+    {
+        $item = new AsciiArtItem("//\n//\n///");
+        $this->assertEquals(3, $item->getArtLength());
+    }
+
     public function testGetRowsLeftAligned()
     {
         $menuStyle = $this->createMock(MenuStyle::class);