refactor file container and add more tests

Author: eugene-matvejev

src/Container/File.php renamed to src/Container/FileContainer.php

@@ -4,7 +4,7 @@
 
 use EM\CssCompiler\Exception\FileException;
 
-class File
+class FileContainer
 {
     const TYPE_UNKNOWN    = 'unknown';
     const TYPE_COMPASS    = 'compass';

src/Processor/Processor.php

@@ -3,7 +3,7 @@
 namespace EM\CssCompiler\Processor;
 
 use Composer\IO\IOInterface;
-use EM\CssCompiler\Container\File;
+use EM\CssCompiler\Container\FileContainer;
 use EM\CssCompiler\Exception\CompilerException;
 use Leafo\ScssPhp\Compiler as SASSCompiler;
 use lessc as LESSCompiler;
@@ -28,7 +28,7 @@ class Processor
      */
     private $io;
     /**
-     * @var File[]
+     * @var FileContainer[]
      */
     private $files = [];
     /**
@@ -70,14 +70,14 @@ public function attachFiles($inputPath, $outputPath)
                 $this->attachFiles("$inputPath/$file", $outputPath);
             }
         } else if (is_file($inputPath)) {
-            $this->files[] = new File($inputPath, $outputPath);
+            $this->files[] = new FileContainer($inputPath, $outputPath);
         } else {
             throw new \Exception("file doesn't exists");
         }
     }
 
     /**
-     * @return File[]
+     * @return FileContainer[]
      */
     public function getFiles()
     {
@@ -146,14 +146,14 @@ public function processFiles($formatter)
      * @return File
      * @throws CompilerException
      */
-    public function processFile(File $file)
+    public function processFile(FileContainer $file)
     {
         switch ($file->getType()) {
-            case File::TYPE_COMPASS:
-            case File::TYPE_SCSS:
-            case File::TYPE_SASS:
+            case FileContainer::TYPE_COMPASS:
+            case FileContainer::TYPE_SCSS:
+            case FileContainer::TYPE_SASS:
                 return $file->setParsedContent($this->sass->compile($file->getSourceContent()));
-            case File::TYPE_LESS:
+            case FileContainer::TYPE_LESS:
                 return $file->setParsedContent($this->less->compile($file->getSourceContent()));
         }
 

tests/phpunit/ProcessorTest.php

@@ -0,0 +1,134 @@
+<?php
+
+namespace EM\Tests\PHPUnit;
+
+use EM\CssCompiler\ScriptHandler;
+use EM\Tests\Environment\IntegrationTestSuite;
+
+/**
+ * @see ScriptHandler
+ */
+class ScriptHandlerTest extends IntegrationTestSuite
+{
+    /*** *************************** CONFIGURATION VALIDATION *************************** ***/
+    /**
+     * @see ScriptHandler::validateConfiguration
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateConfigurationExpectedExceptionOnNotExistingKey()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [[]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateConfiguration
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateConfigurationExpectedExceptionOnNotArray()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [[ScriptHandler::CONFIG_MAIN_KEY]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateConfiguration
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateConfigurationExpectedExceptionOptionIsNotArray()
+    {
+        $arr = [
+            ScriptHandler::CONFIG_MAIN_KEY => [
+                'string'
+            ]
+        ];
+        $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [$arr]);
+    }
+
+    /**
+     * @see ScriptHandler::validateConfiguration
+     * @test
+     */
+    function validateConfigurationOnValid()
+    {
+        $arr = [
+            ScriptHandler::CONFIG_MAIN_KEY => [
+                [
+                    ScriptHandler::OPTION_KEY_INPUT  => ['string'],
+                    ScriptHandler::OPTION_KEY_OUTPUT => 'string'
+                ]
+            ]
+        ];
+        $result = $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [$arr]);
+        $this->assertTrue($result);
+    }
+
+    /*** *************************** OPTIONS VALIDATION *************************** ***/
+    /**
+     * @see ScriptHandler::validateOptions
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateOptionsExpectedExceptionOnMissingInput()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateOptions', [[ScriptHandler::OPTION_KEY_OUTPUT]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateOptions
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateOptionsExpectedExceptionOnMissingOutput()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateOptions', [[ScriptHandler::OPTION_KEY_INPUT]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateOptions
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateOptionsExpectedExceptionOnInputNotArray()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateOptions', [[
+            ScriptHandler::OPTION_KEY_INPUT  => 'string',
+            ScriptHandler::OPTION_KEY_OUTPUT => 'string'
+        ]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateOptions
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateOptionsExpectedExceptionOnOutputNotString()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateOptions', [[
+            ScriptHandler::OPTION_KEY_INPUT  => ['string'],
+            ScriptHandler::OPTION_KEY_OUTPUT => ['string']
+        ]]);
+    }
+
+    /**
+     * @see ScriptHandler::validateOptions
+     * @test
+     */
+    function validateOptionsOnValid()
+    {
+        $result = $this->invokeMethod(new ScriptHandler(), 'validateOptions', [[
+            ScriptHandler::OPTION_KEY_INPUT  => ['string'],
+            ScriptHandler::OPTION_KEY_OUTPUT => 'string'
+        ]]);
+
+        $this->assertTrue($result);
+    }
+}