heiglandreas / JUnitDiff

src/Command/DiffCommand.php

Indentation +6 added lines, -6 removed lines

@@ -37,12 +37,12 @@
     protected function configure()
     {
         $this->setName('diff')
-             ->setDescription('Shows a diff between two JUnit log files')
-             ->setDefinition(array(
-                 new InputOption('input1', '1', InputOption::VALUE_REQUIRED, 'First input file'),
-                 new InputOption('input2', '2', InputOption::VALUE_REQUIRED, 'Second input file')
-             ))
-             ->setHelp('');
+                ->setDescription('Shows a diff between two JUnit log files')
+                ->setDefinition(array(
+                    new InputOption('input1', '1', InputOption::VALUE_REQUIRED, 'First input file'),
+                    new InputOption('input2', '2', InputOption::VALUE_REQUIRED, 'Second input file')
+                ))
+                ->setHelp('');
     }
 
 

Spacing +4 added lines, -4 removed lines

@@ -62,7 +62,7 @@ 
         try {
             $file1  = $input->getOption('input1');
             $array1 = $parser->parseFile($file1);
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
             $output->writeln('<bg=red;fg=white>  ' . $e->getMessage() . '  </>');
             return;
         }
@@ -78,11 +78,11 @@ 
         $array = $this->merge($array1, $array2);
 
         foreach ($array as $key => $value) {
-            if (! isset($value['base'])) {
+            if (!isset($value['base'])) {
                 $output->writeln('<bg=green;fg=black>+</> ' . $key);
                 continue;
             }
-            if (! isset($value['current'])) {
+            if (!isset($value['current'])) {
                 $output->writeln('<bg=red;fg=yellow>-</> ' . $key);
                 continue;
             }
@@ -133,7 +133,7 @@ 
             $merged[$key]['base'] = $value;
         }
 
-        foreach($array2 as $key => $value) {
+        foreach ($array2 as $key => $value) {
             $merged[$key]['current'] = $value;
         }
 

src/JUnitMerger.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
             $this->mergeResult->addBase($key, $value);
         }
 
-        foreach($array2 as $key => $value) {
+        foreach ($array2 as $key => $value) {
             $this->mergeResult->addCurrent($key, $value);
         }
 

src/Command/CompareCommand.php

Indentation +6 added lines, -6 removed lines

@@ -47,12 +47,12 @@
     protected function configure()
     {
         $this->setName('compare')
-             ->setDescription('Compares two JUnit log files')
-             ->setDefinition(array(
-                 new InputArgument('base', InputArgument::REQUIRED, 'Base file for the comparison'),
-                 new InputArgument('current', InputArgument::REQUIRED, 'Current file to compare against the base file')
-             ))
-             ->setHelp('');
+                ->setDescription('Compares two JUnit log files')
+                ->setDefinition(array(
+                    new InputArgument('base', InputArgument::REQUIRED, 'Base file for the comparison'),
+                    new InputArgument('current', InputArgument::REQUIRED, 'Current file to compare against the base file')
+                ))
+                ->setHelp('');
     }
 
 

src/Writer/Summary.php

Doc Comments +4 added lines

@@ -41,6 +41,10 @@
 
     protected $file2;
 
+    /**
+     * @param string $file1
+     * @param string $file2
+     */
     public function __construct(StyleInterface $style, $file1, $file2)
     {
         $this->style = $style;

src/Writer/Standard.php

Doc Comments +3 added lines

@@ -40,6 +40,9 @@
 
     private $verbosity;
 
+    /**
+     * @param integer $verbosity
+     */
     public function __construct(StyleInterface $style, $verbosity = Output::VERBOSITY_NORMAL)
     {
         $this->style = $style;

Spacing +4 added lines, -4 removed lines

@@ -51,11 +51,11 @@ 
         $mergeResult->sort();
 
         foreach ($mergeResult as $key => $value) {
-            if (! isset($value['base'])) {
+            if (!isset($value['base'])) {
                 $this->writeAddedTest($key);
                 continue;
             }
-            if (! isset($value['current'])) {
+            if (!isset($value['current'])) {
                 $this->writeRemovedTest($key);
                 continue;
             }
@@ -112,7 +112,7 @@ 
 
     private function addVerboseInformationToChangedTest($current)
     {
-        if (! $current['message']) {
+        if (!$current['message']) {
             return;
         }
 
@@ -129,7 +129,7 @@ 
 
     private function addVeryVerboseInformationToChangedTest($current)
     {
-        if (! $current['info']) {
+        if (!$current['info']) {
             return;
         }
 

src/JUnitParser.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
     public function getLastLibXmlError()
     {
         $error = libxml_get_last_error();
-        if (! $error instanceof \LibXMLError) {
+        if (!$error instanceof \LibXMLError) {
             return false;
         }
 
@@ -54,7 +54,7 @@ 
      */
     public function parseFile($filename)
     {
-        if (! is_readable($filename)) {
+        if (!is_readable($filename)) {
             throw new \UnexpectedValueException(sprintf(
                 'File %s is not readable',
                 basename($filename)
@@ -78,7 +78,7 @@ 
         foreach ($items as $item) {
 
             $class = $item->getAttribute('class');
-            if (! $class) {
+            if (!$class) {
                 $class = explode('::', $item->parentNode->getAttribute('name'))[0];
             }
 
@@ -99,14 +99,14 @@ 
             switch ($type) {
                 case 'error':
                 case 'failure':
-                    if (! $element->hasAttributes()) {
+                    if (!$element->hasAttributes()) {
                         break;
                     }
                     $message = $element->attributes->getNamedItem('message');
                     if ($message instanceof \DOMAttr) {
                         $message = $message->value;
                     }
-                    $ftype   = $element->attributes->getNamedItem('type');
+                    $ftype = $element->attributes->getNamedItem('type');
                     if ($ftype instanceof \DOMAttr) {
                         $ftype = $ftype->value;
                     }