Update CLI result renderer to look similar to CGI

Author: AydinHassan

src/ResultRenderer/CliResultRenderer.php

@@ -43,8 +43,22 @@ public function render(ResultsRenderer $renderer)
         });
 
         $output = '';
+        if (count($results)) {
+            $output .= $renderer->center("Some executions of your solution produced incorrect output!\n");
+        }
+
+        /** @var FailureInterface $request **/
         foreach ($results as $key => $request) {
-            $output .= $renderer->renderResult($request) . "\n";
+            $output .= $renderer->lineBreak();
+            $output .= "\n";
+            $output .= $renderer->style(sprintf('Execution %d', $key + 1), ['bold', 'underline', 'blue']);
+            $output .= ' ' . $renderer->style(' FAILED ', ['bg_red', 'bold']) . "\n\n";
+
+            $output .= $request->getArgs()->isEmpty()
+                ? "Arguments: None\n"
+                : sprintf("Arguments: \"%s\"\n", $request->getArgs()->implode('", "'));
+
+            $output .= "\n" . $renderer->renderResult($request) . "\n";
         }
 
         return $output;

test/ResultRenderer/CliResultRendererTest.php

@@ -44,7 +44,41 @@ function (RequestFailure $failure) use ($failureRenderer) {
         $result = new CliResult([$failure]);
         $renderer = new CliResultRenderer($result, new RequestRenderer);
 
-        $expected = "REQUEST FAILURE\n\n";
+        $expected  = "Some executions of your solution produced incorrect output!\n";
+        $expected .= "\e[33m──────────────────────────────────────────────────\e[0m\n";
+        $expected .= "\e[34m\e[4m\e[1mExecution 1\e[0m\e[0m\e[0m \e[1m\e[41m FAILED \e[0m\e[0m\n";
+        $expected .= "\n";
+        $expected .= "Arguments: None\n\nREQUEST FAILURE\n\n";
+
+        $this->assertSame($expected, $renderer->render($this->getRenderer()));
+    }
+
+    public function testRenderWithFailedRequestWithMultipleArgs()
+    {
+        $failureRenderer = $this->prophesize(RequestFailureRenderer::class);
+        $failureRenderer->render($this->getRenderer())->willReturn("REQUEST FAILURE\n");
+
+        $this->getResultRendererFactory()->registerRenderer(
+            RequestFailure::class,
+            RequestFailureRenderer::class,
+            function (RequestFailure $failure) use ($failureRenderer) {
+                return $failureRenderer->reveal();
+            }
+        );
+
+        $failure = new RequestFailure(
+            new ArrayObject(['one', 'two', 'three']),
+            'EXPECTED OUTPUT',
+            'ACTUAL OUTPUT'
+        );
+        $result = new CliResult([$failure]);
+        $renderer = new CliResultRenderer($result, new RequestRenderer);
+
+        $expected  = "Some executions of your solution produced incorrect output!\n";
+        $expected .= "\e[33m──────────────────────────────────────────────────\e[0m\n";
+        $expected .= "\e[34m\e[4m\e[1mExecution 1\e[0m\e[0m\e[0m \e[1m\e[41m FAILED \e[0m\e[0m\n";
+        $expected .= "\n";
+        $expected .= "Arguments: \"one\", \"two\", \"three\"\n\nREQUEST FAILURE\n\n";
 
         $this->assertSame($expected, $renderer->render($this->getRenderer()));
     }