Merge pull request #18 from learn-symfony/master

sync with main repository

Author: Eugene Matvejev

src/Container/FileContainer.php

@@ -12,7 +12,7 @@ class FileContainer
     const TYPE_UNKNOWN = 'unknown';
     const TYPE_SCSS    = 'scss';
     const TYPE_LESS    = 'less';
-    static $supportedTypes = [
+    public static $supportedTypes = [
         self::TYPE_SCSS,
         self::TYPE_LESS
     ];

src/Processor/Processor.php

@@ -11,6 +11,8 @@
 use scss_compass as CompassCompiler;
 
 /**
+ * @see   ProcessorTest
+ *
  * @since 0.1
  */
 class Processor
@@ -20,7 +22,7 @@ class Processor
     const FORMATTER_EXPANDED   = 'expanded';
     const FORMATTER_NESTED     = 'nested';
     const FORMATTER_COMPACT    = 'compact';
-    static $supportedFormatters = [
+    public static $supportedFormatters = [
         self::FORMATTER_COMPRESSED,
         self::FORMATTER_CRUNCHED,
         self::FORMATTER_EXPANDED,

src/ScriptHandler.php

@@ -5,6 +5,11 @@
 use Composer\Script\Event;
 use EM\CssCompiler\Processor\Processor;
 
+/**
+ * @see   ScriptHandlerTest
+ *
+ * @since 0.1
+ */
 class ScriptHandler
 {
     const CONFIG_MAIN_KEY          = 'css-compiler';

tests/phpunit/Processor/ProcessorTest.php

@@ -28,10 +28,11 @@ protected function setUp()
     public function attachFiles()
     {
         $paths = [
+            static::getSharedFixturesDirectory() . '/less'             => 1,
             static::getSharedFixturesDirectory() . '/compass'          => 1,
-            static::getSharedFixturesDirectory() . '/scss'             => 4,
             static::getSharedFixturesDirectory() . '/scss/layout.scss' => 1,
-            static::getSharedFixturesDirectory()                       => 6
+            static::getSharedFixturesDirectory() . '/scss'             => 4,
+            static::getSharedFixturesDirectory()                       => 7
         ];
         foreach ($paths as $path => $expectedFiles) {
             $processor = new Processor($this->io);
@@ -61,6 +62,15 @@ public function processFileOnSCSS()
         $this->invokeProcessFileMethod('scss/layout.scss', '');
     }
 
+    /**
+     * @see Processor::processFile
+     * @test
+     */
+    public function processFileOnLESS()
+    {
+        $this->invokeProcessFileMethod('less/print.less', '');
+    }
+
     /**
      * @see Processor::processFile
      * @test
@@ -114,7 +124,7 @@ public function processFileExpectedException()
      * @see Processor::getFormatterClass
      * @test
      */
-    public function getFormatterClass()
+    public function getFormatterClassOnCorrect()
     {
         foreach (Processor::$supportedFormatters as $formatter) {
             $expected = 'Leafo\\ScssPhp\\Formatter\\' . ucfirst($formatter);
@@ -125,4 +135,38 @@ public function getFormatterClass()
             );
         }
     }
+
+    /**
+     * @see Processor::getFormatterClass
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    public function getFormatterClassOnException()
+    {
+        $this->invokeMethod(new Processor($this->io), 'getFormatterClass', ['not-existing']);
+    }
+
+    /**
+     * @see Processor::fetchInputContextIntoFile
+     * @test
+     */
+    public function fetchInputContextIntoFileOnSuccess()
+    {
+        $file = new FileContainer(static::getSharedFixturesDirectory() . '/scss/layout.scss', '');
+        $this->invokeMethod(new Processor($this->io), 'fetchInputContextIntoFile', [$file]);
+
+        $this->assertNotNull($file->getInputContent());
+    }
+
+    /**
+     * @see Processor::fetchInputContextIntoFile
+     * @test
+     *
+     * @expectedException \EM\CssCompiler\Exception\FileException
+     */
+    public function fetchInputContextIntoFileOnException()
+    {
+        $this->invokeMethod(new Processor($this->io), 'fetchInputContextIntoFile', [new FileContainer('input', 'output')]);
+    }
 }

tests/phpunit/ScriptHandlerTest.php

@@ -28,10 +28,20 @@ function validateConfigurationExpectedExceptionOnNotExistingKey()
      *
      * @expectedException \InvalidArgumentException
      */
-    function validateConfigurationExpectedExceptionOnNotArray()
+    function validateConfigurationExpectedExceptionOnEmpty()
     {
         $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [[ScriptHandler::CONFIG_MAIN_KEY]]);
     }
+    /**
+     * @see ScriptHandler::validateConfiguration
+     * @test
+     *
+     * @expectedException \InvalidArgumentException
+     */
+    function validateConfigurationExpectedExceptionOnNotArray()
+    {
+        $this->invokeMethod(new ScriptHandler(), 'validateConfiguration', [[ScriptHandler::CONFIG_MAIN_KEY => 'string']]);
+    }
 
     /**
      * @see ScriptHandler::validateConfiguration

tests/shared-fixtures/less/print.less

@@ -0,0 +1,101 @@
+/*! Source: https://github.com/h5bp/html5-boilerplate/blob/master/src/css/main.css */
+
+// ==========================================================================
+// Print styles.
+// Inlined to avoid the additional HTTP request: h5bp.com/r
+// ==========================================================================
+
+@media print {
+    *,
+    *:before,
+    *:after {
+        background: transparent !important;
+        color: #000 !important; // Black prints faster: h5bp.com/s
+        box-shadow: none !important;
+        text-shadow: none !important;
+    }
+
+    a,
+    a:visited {
+        text-decoration: underline;
+    }
+
+    a[href]:after {
+        content: " (" attr(href) ")";
+    }
+
+    abbr[title]:after {
+        content: " (" attr(title) ")";
+    }
+
+    // Don't show links that are fragment identifiers,
+    // or use the `javascript:` pseudo protocol
+    a[href^="#"]:after,
+    a[href^="javascript:"]:after {
+        content: "";
+    }
+
+    pre,
+    blockquote {
+        border: 1px solid #999;
+        page-break-inside: avoid;
+    }
+
+    thead {
+        display: table-header-group; // h5bp.com/t
+    }
+
+    tr,
+    img {
+        page-break-inside: avoid;
+    }
+
+    img {
+        max-width: 100% !important;
+    }
+
+    p,
+    h2,
+    h3 {
+        orphans: 3;
+        widows: 3;
+    }
+
+    h2,
+    h3 {
+        page-break-after: avoid;
+    }
+
+    // Bootstrap specific changes start
+
+    // Bootstrap components
+    .navbar {
+        display: none;
+    }
+    .btn,
+    .dropup > .btn {
+        > .caret {
+            border-top-color: #000 !important;
+        }
+    }
+    .label {
+        border: 1px solid #000;
+    }
+
+    .table {
+        border-collapse: collapse !important;
+
+        td,
+        th {
+            background-color: #fff !important;
+        }
+    }
+    .table-bordered {
+        th,
+        td {
+            border: 1px solid #ddd !important;
+        }
+    }
+
+    // Bootstrap specific changes end
+}