From a0b04ed924314b672f1f83183a236f8b3f85f162 Mon Sep 17 00:00:00 2001
From: jperras <joel.perras@gmail.com>
Date: Sun, 2 Aug 2009 22:54:05 -0400
Subject: [PATCH] Adding proper skipIf to Javascript asset timestamp test.

---
 cake/tests/cases/libs/view/helpers/javascript.test.php | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/cake/tests/cases/libs/view/helpers/javascript.test.php b/cake/tests/cases/libs/view/helpers/javascript.test.php
index a9d546e06..eb64fec96 100644
--- a/cake/tests/cases/libs/view/helpers/javascript.test.php
+++ b/cake/tests/cases/libs/view/helpers/javascript.test.php
@@ -231,8 +231,7 @@ class JavascriptTest extends CakeTestCase {
  * @return void
  */
 	function testFilteringAndTimestamping() {
-		if (!is_writable(JS)) {
-			echo "<br />JavaScript directory not writable, skipping JS asset timestamp tests<br />";
+		if($this->skipIf(!is_writable(JS), "%s JavaScript directory not writable, skipping JS asset timestamp tests.")) {
 			return;
 		}