check log directory existence

Author: Forestsoft-de

src/PhpunitMerger/Command/LogCommand.php

@@ -44,7 +44,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
     {
         $finder = new Finder();
         $finder->files()
-            ->in(realpath($input->getArgument('directory')));
+            ->in(realpath($input->getArgument('directory')))->sortByName(true);
 
         $this->document = new \DOMDocument('1.0', 'UTF-8');
         $this->document->formatOutput = true;
@@ -67,7 +67,6 @@ protected function execute(InputInterface $input, OutputInterface $output)
         foreach ($finder as $file) {
             try {
                 $xml = new \SimpleXMLElement(file_get_contents($file->getRealPath()));
-                $code = json_encode($xml);
                 $xmlArray = json_decode(json_encode($xml), true);
                 if (!empty($xmlArray)) {
                     $this->addTestSuites($baseSuite, $xmlArray);
@@ -162,21 +161,6 @@ private function addChildElements(array $tree, \DOMElement $element)
             $element->appendChild($child);
         }
     }
-    private function addAttributeValueToTestSuite(\DOMElement $element, $key, $value)
-    {
-        if (in_array($key, $this->keysToCalculate)) {
-            $currentValue = $element->hasAttribute($key) ? $element->getAttribute($key) : 0;
-            $element->setAttribute($key, (string)($currentValue + $value));
-
-            if ($element->hasAttribute('parent')) {
-                $parent = $element->getAttribute('parent');
-                if (isset($this->domElements[$parent])) {
-                    $this->addAttributeValueToTestSuite($this->domElements[$parent], $key, $value);
-                }
-            }
-        }
-    }
-
     private function calculateTopLevelStats()
     {
         /** @var \DOMElement $topNode */

tests/PhpunitMerger/Command/AbstractCommandTest.php

@@ -22,13 +22,15 @@ abstract class AbstractCommandTest extends TestCase
     public function assertOutputFileNotExists()
     {
         $filesystem = new Filesystem();
+        self::assertDirectoryExists($this->logDirectory , $this->logDirectory . " does not exists");
         $filesystem->remove($this->logDirectory . $this->outputFile);
 
         $this->assertFileNotExists($this->logDirectory . $this->outputFile);
     }
 
     public function assertOutputDirectoryNotExists()
     {
+        self::assertDirectoryExists($this->logDirectory , $this->logDirectory . " does not exists");
         $filesystem = new Filesystem();
         $filesystem->remove($this->logDirectory . dirname($this->outputFile));