Add more tests

.travis.yml

@@ -55,9 +55,9 @@ php:
   - 5.5
 
 script:
-  - bin/setup assets:compile-less
-  - bin/setup assets:build-javascript
   - bin/setup system:upgrade -y -v -f
   - php hudson/cleanupSubdefs.php
+  - bin/developer assets:compile-less
+  - bin/developer assets:build-javascript
   - sh tests/js/run.sh
   - bin/phpunit

lib/Alchemy/Phrasea/Command/Developer/JavascriptBuilder.php

@@ -18,7 +18,7 @@ use Symfony\Component\Process\ProcessBuilder;
 use Alchemy\Phrasea\Exception\RuntimeException;
 
 /**
- * This command builds less file
+ * This command builds javascript files
  */
 class JavascriptBuilder extends Command
 {

lib/Alchemy/Phrasea/Command/Developer/LessCompiler.php

@@ -55,7 +55,7 @@ class LessCompiler extends Command
         try {
             $this->container['phraseanet.less-builder']->build($files);
         } catch (RuntimeException $e) {
-            $output->writeln(sprintf('<error>Could not build less files %s</error>', implode(', ', $e->getMessage())));
+            $output->writeln(sprintf('<error>Could not build less files %s</error>', $e->getMessage()));
 
             return 1;
         }

lib/Alchemy/Phrasea/Command/Plugin/AbstractPluginCommand.php

@@ -44,12 +44,12 @@ abstract class AbstractPluginCommand extends Command
         );
 
         $output->write('Building Assets...');
-        
+
         try {
             $this->container['phraseanet.less-builder']->build($files);
             $output->writeln(" <comment>OK</comment>");
         } catch (RuntimeException $e) {
-            $output->writeln(sprintf('<error>Could not build less files %s</error>', implode(', ', $e->getMessage())));
+            $output->writeln(sprintf('<error>Could not build less files %s</error>', $e->getMessage()));
         }
     }
 }

lib/Alchemy/Phrasea/Core/Provider/LessCompilerServiceProvider.php

@@ -27,4 +27,4 @@ class LessCompilerServiceProvider implements ServiceProviderInterface
     public function boot(Application $app)
     {
     }
-}
+}

lib/Alchemy/Phrasea/Utilities/Less/Builder.php

@@ -35,6 +35,8 @@ class Builder
      * Build LESS files
      *
      * @param array $files
+     *
+     * @throws RuntimeException
      */
     public function build($files)
     {

lib/Alchemy/Phrasea/Utilities/Less/Compiler.php

@@ -15,7 +15,6 @@ use Alchemy\Phrasea\Application;
 use Alchemy\BinaryDriver\BinaryInterface;
 use Alchemy\BinaryDriver\Exception\ExecutionFailureException;
 use Alchemy\Phrasea\Exception\RuntimeException;
-use Doctrine\Common\Collections\ArrayCollection;
 use Symfony\Component\Filesystem\Filesystem;
 
 class Compiler
@@ -52,19 +51,19 @@ class Compiler
             $files = new \ArrayObject(is_array($files) ? $files : array($files));
         }
 
-        $files = new ArrayCollection((array) $files);
+        $files = (array) $files;
 
-        if ($files->forAll(function($file) {
+        foreach($files as $file) {
-            return is_file($file);
+            if (false === is_file($file)) {
-        })) {
+                throw new RuntimeException($file . ' does not exists.');
-            throw new RuntimeException($files . ' does not exists.');
+            }
         }
 
         if (!is_writable(dirname($target))) {
             throw new RuntimeException(realpath(dirname($target)) . ' is not writable.');
         }
 
-        $commands = $files->toArray();
+        $commands = $files;
         array_unshift($commands, '--compile');
 
         try {

tests/Alchemy/Tests/Phrasea/Plugin/Fixtures/PluginDir/TestPlugin/account.less

@@ -1 +1 @@
-@import "my-plugin/account.less";
+@import "my-plugin/account.less";

tests/Alchemy/Tests/Phrasea/Plugin/Fixtures/PluginDir/TestPlugin/login.less

@@ -1 +1 @@
-@import "my-plugin/login.less";
+@import "my-plugin/login.less";

tests/Alchemy/Tests/Phrasea/Plugin/Fixtures/PluginDirInstalled/TestPlugin/account.less

@@ -1 +1 @@
-@import "my-plugin/account.less";
+@import "my-plugin/account.less";

tests/Alchemy/Tests/Phrasea/Plugin/Fixtures/PluginDirInstalled/TestPlugin/login.less

@@ -1 +1 @@
-@import "my-plugin/login.less";
+@import "my-plugin/login.less";

tests/Alchemy/Tests/Phrasea/Utilities/Less/CompilerTest.php

@@ -12,8 +12,9 @@
 namespace Alchemy\Tests\Phrasea\Utilities\Compiler;
 
 use Alchemy\Phrasea\Utilities\Less\Compiler;
+use Alchemy\BinaryDriver\Exception\ExecutionFailureException;
 
-class CompilerTest extends \PHPUnit_Framework_TestCase
+class CompilerTest extends \PhraseanetPHPUnitAbstract
 {
     public function testCompileSuccess()
     {
@@ -28,4 +29,42 @@ class CompilerTest extends \PHPUnit_Framework_TestCase
 
         $compiler->compile(__DIR__ . '/output.css', __FILE__);
     }
+
+    public function testCreate()
+    {
+        $compiler = Compiler::create(self::$DI['app']);
+        $this->assertInstanceOf('Alchemy\Phrasea\Utilities\Less\Compiler', $compiler);
+    }
+
+   /**
+     * @expectedException Alchemy\Phrasea\Exception\RuntimeException
+     */
+    public function testCompileFileNotExists()
+    {
+        $recessDriver = $this->getMock('Alchemy\BinaryDriver\BinaryInterface');
+
+        $filesystem = $this->getMock('Symfony\Component\Filesystem\Filesystem');
+        $filesystem->expects($this->once())->method('mkdir');
+
+        $compiler = new Compiler($filesystem, $recessDriver);
+
+        $compiler->compile(__DIR__ . '/output.css', 'not_existsing_file');
+    }
+
+   /**
+     * @expectedException Alchemy\Phrasea\Exception\RuntimeException
+     */
+    public function testCompileExecutionFailure()
+    {
+        $recessDriver = $this->getMock('Alchemy\BinaryDriver\BinaryInterface');
+        $recessDriver->expects($this->once())->method('command')->will(
+            $this->throwException(new ExecutionFailureException())
+        );
+
+        $filesystem = $this->getMock('Symfony\Component\Filesystem\Filesystem');
+
+        $compiler = new Compiler($filesystem, $recessDriver);
+
+        $compiler->compile(__DIR__ . '/output.css', __FILE__);
+    }
 }