MAGETWO-64085: Static versioning and styles minification break email fonts styles #8241

- add integration test

Author: cpartica

dev/tests/integration/testsuite/Magento/Framework/Css/PreProcessor/Adapter/CssInlinerTest.php

@@ -0,0 +1,103 @@
+<?php
+/**
+ * Copyright © 2013-2017 Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\Framework\Css\PreProcessor\Adapter;
+
+class CssInlinerTest extends \PHPUnit\Framework\TestCase
+{
+    /**
+     * @var \Magento\Framework\Css\PreProcessor\Adapter\CssInliner
+     */
+    private $model;
+
+    /**
+     * @var \Magento\Framework\ObjectManagerInterface
+     */
+    private $objectManager;
+
+    protected function setUp()
+    {
+        $this->objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
+
+        $this->model = $this->objectManager->create(\Magento\Framework\Css\PreProcessor\Adapter\CssInliner::class);
+    }
+
+    /**
+     * @param string $htmlFilePath
+     * @param string $cssFilePath
+     * @param string $cssExpected
+     * @dataProvider getFilesDataProvider
+     */
+    public function testGetFiles($htmlFilePath, $cssFilePath, $cssExpected)
+    {
+        $html = file_get_contents($htmlFilePath);
+        $css = file_get_contents($cssFilePath);
+        $this->model->setCss($css);
+        $this->model->setHtml($html);
+        $result = $this->model->process();
+        $this->assertContains($cssExpected, $result);
+    }
+
+    /**
+     * @return array
+     */
+    public function getFilesDataProvider()
+    {
+        $fixtureDir = dirname(dirname(__DIR__));
+        return [
+            'noSpacesCss'=>[
+                'resultHtml' => $fixtureDir . "/_files/css/test-input.html",
+                'cssWithoutSpaces' => $fixtureDir . "/_files/css/test-css-no-spaces.css",
+                'vertical-align: top; padding: 10px 10px 10px 0; width: 50%;'
+            ],
+            'withSpacesCss'=>[
+                'resultHtml' => $fixtureDir . "/_files/css/test-input.html",
+                'cssWithSpaces' => $fixtureDir . "/_files/css/test-css-with-spaces.css",
+                'vertical-align: top; padding: 10px 10px 10px 0; width: 50%;'
+            ],
+        ];
+    }
+
+
+
+    /**
+     * @param string $htmlFilePath
+     * @param string $cssFilePath
+     * @param string $cssExpected
+     * @dataProvider getFilesDataProviderEmogrifier
+     */
+    public function testGetFilesEmogrifier($htmlFilePath, $cssFilePath, $cssExpected)
+    {
+        $emogrifier = new \Pelago\Emogrifier;
+
+        $html = file_get_contents($htmlFilePath);
+        $css = file_get_contents($cssFilePath);
+        $emogrifier->setCss($css);
+        $emogrifier->setHtml($html);
+        $result = $emogrifier->emogrify();
+        /**
+         * Tests a bug in the library where there's no spaces to CSS string before passing to Emogrifier
+         * to fix known parsing issue with library.
+         * This test should will fail when this bug is fixed in the library and we should fix the adapter.
+         * https://github.com/jjriv/emogrifier/issues/370
+         */
+        $this->assertNotContains($cssExpected, $result);
+    }
+
+    /**
+     * @return array
+     */
+    public function getFilesDataProviderEmogrifier()
+    {
+        $fixtureDir = dirname(dirname(__DIR__));
+        return [
+            'noSpacesCss'=>[
+                'resultHtml' => $fixtureDir . "/_files/css/test-input.html",
+                'cssWithoutSpaces' => $fixtureDir . "/_files/css/test-css-no-spaces.css",
+                'vertical-align: top; padding: 10px 10px 10px 0; width: 50%;'
+            ]
+        ];
+    }
+}