Permalink
Browse files

Removing cakeDebug from test_app layout.

Adding skips for tests that can cause failures when cache dirs are not writable.
  • Loading branch information...
1 parent 7467224 commit 3f7405c8d6eef19febc3e3ba3257b41446a08daf @markstory markstory committed Dec 20, 2009
Showing with 8 additions and 1 deletion.
  1. +8 −0 cake/tests/cases/libs/view/view.test.php
  2. +0 −1 cake/tests/test_app/views/layouts/default.ctp
@@ -510,6 +510,10 @@ function testElementCacheHelperNoCache() {
* @return void
*/
function testElementCache() {
+ $writable = is_writable(CACHE . 'views' . DS);
+ if ($this->skipIf(!$writable, 'CACHE/views dir is not writable, cannot test elementCache. %s')) {
+ return;
+ }
$View = new TestView($this->PostsController);
$element = 'test_element';
$expected = 'this is the test element';
@@ -775,6 +779,10 @@ function testViewFileName() {
* @return void
*/
function testRenderCache() {
+ $writable = is_writable(CACHE . 'views' . DS);
+ if ($this->skipIf(!$writable, 'CACHE/views dir is not writable, cannot test renderCache. %s')) {
+ return;
+ }
$view = 'test_view';
$View = new View($this->PostsController);
$path = CACHE . 'views' . DS . 'view_cache_'.$view;
@@ -52,6 +52,5 @@
?>
</div>
</div>
- <?php echo $cakeDebug?>
</body>
</html>

0 comments on commit 3f7405c

Please sign in to comment.