Merge pull request #131 from php-school/reset-menu-after-reset-progress

Reset menu after reset progress

Author: AydinHassan

app/config.php

@@ -246,10 +246,7 @@
     SyntaxHighlighter::class => factory(PsxFactory::class),
     PsxFactory::class => object(),
     ResetProgress::class => function (ContainerInterface $c) {
-        return new ResetProgress(
-            $c->get(UserStateSerializer::class),
-            $c->get(OutputInterface::class)
-        );
+        return new ResetProgress($c->get(UserStateSerializer::class));
     },
     ResultRendererFactory::class => object(),
     ResultsRenderer::class => function (ContainerInterface $c) {

composer.json

@@ -23,9 +23,9 @@
         "symfony/process": "^2.3|^3.0",
         "symfony/filesystem": "^2.3|^3.0",
         "fzaninotto/faker": "^1.5",
-        "aydin-hassan/cli-md-renderer": "^2.0",
+        "aydin-hassan/cli-md-renderer": "^2.1",
         "php-school/psx": "~1.0",
-        "php-school/cli-menu": "^1.2.0",
+        "php-school/cli-menu": "^2.0",
         "beberlei/assert": "^2.4",
         "psr/http-message": "^1.0",
         "zendframework/zend-diactoros": "^1.1.3",

composer.lock

@@ -18,20 +18,13 @@ class ResetProgress
      * @var UserStateSerializer
      */
     private $userStateSerializer;
-    
-    /**
-     * @var OutputInterface
-     */
-    private $output;
 
     /**
      * @param UserStateSerializer $userStateSerializer
-     * @param OutputInterface $output
      */
-    public function __construct(UserStateSerializer $userStateSerializer, OutputInterface $output)
+    public function __construct(UserStateSerializer $userStateSerializer)
     {
         $this->userStateSerializer = $userStateSerializer;
-        $this->output = $output;
     }
 
     /**
@@ -40,6 +33,17 @@ public function __construct(UserStateSerializer $userStateSerializer, OutputInte
     public function __invoke(CliMenu $menu)
     {
         $this->userStateSerializer->serialize(new UserState);
-        $this->output->writeLine("Status Reset!");
+
+        $items = $menu
+            ->getParent()
+            ->getItems();
+
+        foreach ($items as $item) {
+            $item->hideItemExtra();
+        }
+
+        $confirm = $menu->confirm('Status Reset!');
+        $confirm->getStyle()->setBg('magenta')->setFg('black');
+        $confirm->display('OK');
     }
 }

src/MenuItem/ResetProgress.php

@@ -3,6 +3,10 @@
 namespace PhpSchool\PhpWorkshopTest\MenuItem;
 
 use PhpSchool\CliMenu\CliMenu;
+use PhpSchool\CliMenu\Dialogue\Confirm;
+use PhpSchool\CliMenu\MenuItem\MenuItemInterface;
+use PhpSchool\CliMenu\MenuStyle;
+use PhpSchool\CliMenu\Terminal\TerminalInterface;
 use PhpSchool\PhpWorkshop\MenuItem\ResetProgress;
 use PhpSchool\PhpWorkshop\Output\OutputInterface;
 use PhpSchool\PhpWorkshop\UserState;
@@ -16,22 +20,53 @@
  */
 class ResetProgressTest extends PHPUnit_Framework_TestCase
 {
-    public function testResetProgress()
+    public function testResetProgressDisablesParentMenuItems()
     {
-        $menu = $this->createMock(CliMenu::class);
+        $item1 = $this->createMock(MenuItemInterface::class);
+        $item2 = $this->createMock(MenuItemInterface::class);
+
+        $item1
+            ->expects($this->once())
+            ->method('hideItemExtra');
+
+        $item2
+            ->expects($this->once())
+            ->method('hideItemExtra');
+
+        $terminal = $this->createMock(TerminalInterface::class);
+
+        $menu = new CliMenu('Menu', [$item1, $item2], $terminal);
+
+        $confirm = $this->createMock(Confirm::class);
+        $confirm
+            ->expects($this->once())
+            ->method('getStyle')
+            ->willReturn(new MenuStyle($terminal));
+
+        $confirm
+            ->expects($this->once())
+            ->method('display')
+            ->with('OK');
+
+        $subMenu = $this->getMockBuilder(CliMenu::class)
+            ->setMethods(['confirm'])
+            ->setConstructorArgs(['Sub Menu', [], $terminal])
+            ->getMock();
+
+        $subMenu->setParent($menu);
+
+        $subMenu
+            ->expects($this->once())
+            ->method('confirm')
+            ->willReturn($confirm);
+
         $userStateSerializer = $this->createMock(UserStateSerializer::class);
         $userStateSerializer
             ->expects($this->once())
             ->method('serialize')
             ->with($this->isInstanceOf(UserState::class));
 
-        $output = $this->createMock(OutputInterface::class);
-        $output
-            ->expects($this->once())
-            ->method('writeLine')
-            ->with(("Status Reset!"));
-        
-        $resetProgress = new ResetProgress($userStateSerializer, $output);
-        $resetProgress->__invoke($menu);
+        $resetProgress = new ResetProgress($userStateSerializer);
+        $resetProgress->__invoke($subMenu);
     }
 }