Simplify syntax

Author: kylekatarnls

.codeclimate.yml

@@ -26,3 +26,4 @@ ratings:
   - "**.rb"
 exclude_paths:
 - config/
+- tests/

src/Phug/DevTool/Command/CheckCommand.php

@@ -32,20 +32,17 @@ protected function execute(InputInterface $input, OutputInterface $output)
             '--coverage-clover' => $coverageFilePath,
         ];
 
-        if ($input->getOption('coverage-text')) {
-            $args[] = '--coverage-text';
-        }
-
-        if ($path = $input->getOption('coverage-clover')) {
-            $args['--coverage-clover'] = $path;
-        }
-
-        if ($path = $input->getOption('coverage-html')) {
-            $args['--coverage-html'] = $path;
-        }
+        $passTrough = [
+            'coverage-text',
+            'coverage-clover',
+            'coverage-html',
+            'group',
+        ];
 
-        if ($group = $input->getOption('group')) {
-            $args['--group'] = $group;
+        foreach ($passTrough as $option) {
+            if ($value = $input->getOption($option)) {
+                $args[$option] = $value;
+            }
         }
 
         if (($code = $app->runCommand('unit-tests:run', $output, $args)) !== 0) {

tests/Phug/DevTool/Command/UnitTestsRunCommandTest.php

@@ -50,13 +50,11 @@ public function testExecute()
         }
         $coverageHtml = $app.DIRECTORY_SEPARATOR.'coverage';
         $coverageClover = $app.DIRECTORY_SEPARATOR.'coverage.xml';
-        $coverageHtmlShellArg = escapeshellarg(addslashes($coverageHtml));
-        $coverageCloverShellArg = escapeshellarg(addslashes($coverageClover));
         $input = new StringInput(
             'unit-tests:run'.
             ' --coverage-text'.
-            ' --coverage-html='.$coverageHtmlShellArg.
-            ' --coverage-clover='.$coverageCloverShellArg
+            ' --coverage-html='.escapeshellarg(addslashes($coverageHtml)).
+            ' --coverage-clover='.escapeshellarg(addslashes($coverageClover))
         );
         $buffer = new NullOutput();
         $app = new Application();