From 65aecc8e9ebfd620d25ef44c4c2aa969ad77a290 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michal=20=C4=8Ciha=C5=99?= <michal@cihar.com>
Date: Tue, 5 Jan 2016 15:47:51 +0100
Subject: [PATCH] Test theme rendering
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Tested for both built in themes.

Signed-off-by: Michal Čihař <michal@cihar.com>
---
 test/classes/ThemeTest.php | 34 ++++++++++++++++++++++++++++++++++
 themes/sprites.css.php     |  2 +-
 2 files changed, 35 insertions(+), 1 deletion(-)

diff --git a/test/classes/ThemeTest.php b/test/classes/ThemeTest.php
index 4e1ee3e883..63f620511a 100644
--- a/test/classes/ThemeTest.php
+++ b/test/classes/ThemeTest.php
@@ -39,6 +39,7 @@ class ThemeTest extends PMATestCase
         $GLOBALS['text_dir'] = 'ltr';
         include 'themes/pmahomme/layout.inc.php';
         $GLOBALS['server'] = '99';
+        $GLOBALS['collation_connection'] = 'utf-8';
     }
 
     /**
@@ -123,6 +124,39 @@ class ThemeTest extends PMATestCase
         $this->assertNotNull($newTheme);
     }
 
+    /**
+     * Test for Theme::loadCss
+     *
+     * @param $theme string Path to theme files
+     *
+     * @return void
+     *
+     * @dataProvider listThemes
+     */
+    public function testLoadCss($theme)
+    {
+        $newTheme = Theme::load($theme);
+        ob_start();
+        $ret = $newTheme->loadCss();
+        $out = ob_get_contents();
+        ob_end_clean();
+        $this->assertTrue($ret);
+        $this->assertContains('FILE: navigation.css.php', $out);
+    }
+
+    /**
+     * Data provider for Theme::loadCss test
+     *
+     * @return array with theme paths
+     */
+    public function listThemes()
+    {
+        return array(
+            array('./themes/original'),
+            array('./themes/pmahomme/'),
+        );
+    }
+
     /**
      * Test for Theme::load
      *
diff --git a/themes/sprites.css.php b/themes/sprites.css.php
index f458077eea..768804ed73 100644
--- a/themes/sprites.css.php
+++ b/themes/sprites.css.php
@@ -7,7 +7,7 @@
  */
 
 // unplanned execution path
-if (! defined('PMA_MINIMUM_COMMON')) {
+if (! defined('PMA_MINIMUM_COMMON') && ! defined('TESTSUITE')) {
     exit();
 }