Merge pull request #135 from FriendsOfCake/issue-134

Issue 134

composer.json

@@ -33,7 +33,8 @@
     },
     "autoload-dev": {
         "psr-4": {
-            "BootstrapUI\\Test\\": "tests"
+            "BootstrapUI\\Test\\": "tests",
+            "TestApp\\": "tests/test_app/TestApp"
         }
     }
 }

src/View/UIViewTrait.php

@@ -11,19 +11,21 @@ trait UIViewTrait
     /**
      * Initialization hook method.
      *
-     * ### Options: Associative array with valid keys:
+     * @param array $options Associative array with valid keys:
      *   - `layout`:
-     *      - If empty or true will use the plugin's layout
+     *      - If not set or true will use the plugin's layout
      *      - If a layout name passed it will be used
      *      - If false do nothing (will keep your layout)
      *
      * @return void
      */
     public function initializeUI(array $options = [])
     {
-        if (!isset($options['layout']) || $options['layout'] === true) {
+        if ((!isset($options['layout']) || $options['layout'] === true) &&
+            $this->layout === 'default'
+        ) {
             $this->layout = 'BootstrapUI.default';
-        } elseif (!empty($options['layout']) || $options['layout'] === '') {
+        } elseif (isset($options['layout']) && is_string($options['layout'])) {
             $this->layout = $options['layout'];
         }
 

tests/TestCase/View/UIViewTraitTest.php

@@ -2,6 +2,7 @@
 
 namespace BootstrapUI\View;
 
+use Cake\Core\Configure;
 use Cake\TestSuite\TestCase;
 
 class UIViewTraitTest extends TestCase
@@ -17,6 +18,7 @@ public function setUp()
     {
         parent::setUp();
 
+        Configure::write('App.namespace', 'TestApp');
         $this->View = new UIView();
         $this->View->layout = 'default';
     }
@@ -63,4 +65,12 @@ public function testInitializeUI()
         ]);
         $this->assertSame('', $this->View->layout);
     }
+
+    public function testCellRendering()
+    {
+        $cell = $this->View->cell('Articles');
+
+        $this->assertEquals('display', $cell->template);
+        $this->assertEquals("articles cell display\n", "{$cell}");
+    }
 }

tests/bootstrap.php

@@ -9,24 +9,37 @@
 
 require_once 'vendor/autoload.php';
 
-// Path constants to a few helpful things.
 if (!defined('DS')) {
     define('DS', DIRECTORY_SEPARATOR);
 }
-
-define('ROOT', dirname(__DIR__) . DS);
-define('CAKE_CORE_INCLUDE_PATH', ROOT . 'vendor' . DS . 'cakephp' . DS . 'cakephp');
-define('CORE_PATH', ROOT . 'vendor' . DS . 'cakephp' . DS . 'cakephp' . DS);
-define('CAKE', CORE_PATH . 'src' . DS);
-define('TESTS', ROOT . 'tests');
-define('APP', ROOT . 'tests' . DS . 'test_files' . DS . 'app' . DS);
-define('APP_DIR', 'app');
+define('ROOT', dirname(__DIR__));
+define('APP_DIR', 'TestApp');
 define('WEBROOT_DIR', 'webroot');
-define('WWW_ROOT', dirname(APP) . DS . 'webroot' . DS);
+
 define('TMP', sys_get_temp_dir() . DS);
-define('CONFIG', APP . 'config' . DS);
-define('CACHE', TMP);
-define('LOGS', TMP);
+define('LOGS', TMP . 'logs' . DS);
+define('CACHE', TMP . 'cache' . DS);
+define('SESSIONS', TMP . 'sessions' . DS);
+
+define('CAKE_CORE_INCLUDE_PATH', ROOT . DS . 'vendor' . DS . 'cakephp' . DS . 'cakephp');
+define('CORE_PATH', CAKE_CORE_INCLUDE_PATH . DS);
+define('CAKE', CORE_PATH . 'src' . DS);
+define('CORE_TESTS', CORE_PATH . 'tests' . DS);
+define('CORE_TEST_CASES', CORE_TESTS . 'TestCase');
+define('TEST_APP', ROOT . DS . 'tests' . DS . 'test_app' . DS);
+
+// Point app constants to the test app.
+define('APP', TEST_APP . 'TestApp' . DS);
+define('WWW_ROOT', TEST_APP . 'webroot' . DS);
+define('CONFIG', TEST_APP . 'config' . DS);
+
+//@codingStandardsIgnoreStart
+@mkdir(LOGS);
+@mkdir(SESSIONS);
+@mkdir(CACHE);
+@mkdir(CACHE . 'views');
+@mkdir(CACHE . 'models');
+//@codingStandardsIgnoreEnd
 
 require_once CORE_PATH . 'config/bootstrap.php';
 
@@ -65,4 +78,4 @@
     ]
 ]);
 
-Plugin::load('BootstrapUI', ['path' => ROOT]);
+Plugin::load('BootstrapUI', ['path' => ROOT . DS]);

tests/test_app/TestApp/Template/Cell/Articles/display.ctp

@@ -0,0 +1 @@
+articles cell display

tests/test_app/TestApp/View/Cell/ArticlesCell.php

@@ -0,0 +1,15 @@
+<?php
+namespace TestApp\View\Cell;
+
+class ArticlesCell extends \Cake\View\Cell
+{
+
+    /**
+     * Default cell action.
+     *
+     * @return void
+     */
+    public function display()
+    {
+    }
+}