reginaldojunior / winners

lib/Cake/Test/Case/AllNetworkTest.php

Spacing +3 added lines, -3 removed lines

@@ -33,9 +33,9 @@
 	public static function suite() {
 		$suite = new CakeTestSuite('All Network related class tests');
 
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'Network');
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'Network' . DS . 'Email');
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'Network' . DS . 'Http');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'Network');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'Network'.DS.'Email');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'Network'.DS.'Http');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/AllRoutingTest.php

Spacing +4 added lines, -4 removed lines

@@ -33,11 +33,11 @@
 	public static function suite() {
 		$suite = new CakeTestSuite('All Routing class tests');
 
-		$libs = CORE_TEST_CASES . DS;
+		$libs = CORE_TEST_CASES.DS;
 
-		$suite->addTestDirectory($libs . 'Routing');
-		$suite->addTestDirectory($libs . 'Routing' . DS . 'Route');
-		$suite->addTestDirectory($libs . 'Routing' . DS . 'Filter');
+		$suite->addTestDirectory($libs.'Routing');
+		$suite->addTestDirectory($libs.'Routing'.DS.'Route');
+		$suite->addTestDirectory($libs.'Routing'.DS.'Filter');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/AllTestSuiteTest.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	public static function suite() {
 		$suite = new CakeTestSuite('All Test Suite classes tests');
 
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'TestSuite');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'TestSuite');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/AllTestsTest.php

Spacing +21 added lines, -21 removed lines

@@ -33,28 +33,28 @@
 	public static function suite() {
 		$suite = new PHPUnit_Framework_TestSuite('All Tests');
 
-		$path = CORE_TEST_CASES . DS;
+		$path = CORE_TEST_CASES.DS;
 
-		$suite->addTestFile($path . 'BasicsTest.php');
-		$suite->addTestFile($path . 'AllConsoleTest.php');
-		$suite->addTestFile($path . 'AllBehaviorsTest.php');
-		$suite->addTestFile($path . 'AllCacheTest.php');
-		$suite->addTestFile($path . 'AllComponentsTest.php');
-		$suite->addTestFile($path . 'AllConfigureTest.php');
-		$suite->addTestFile($path . 'AllCoreTest.php');
-		$suite->addTestFile($path . 'AllControllerTest.php');
-		$suite->addTestFile($path . 'AllDatabaseTest.php');
-		$suite->addTestFile($path . 'AllErrorTest.php');
-		$suite->addTestFile($path . 'AllEventTest.php');
-		$suite->addTestFile($path . 'AllHelpersTest.php');
-		$suite->addTestFile($path . 'AllLogTest.php');
-		$suite->addTestFile($path . 'Model' . DS . 'ModelTest.php');
-		$suite->addTestFile($path . 'AllRoutingTest.php');
-		$suite->addTestFile($path . 'AllNetworkTest.php');
-		$suite->addTestFile($path . 'AllTestSuiteTest.php');
-		$suite->addTestFile($path . 'AllUtilityTest.php');
-		$suite->addTestFile($path . 'AllViewTest.php');
-		$suite->addTestFile($path . 'AllI18nTest.php');
+		$suite->addTestFile($path.'BasicsTest.php');
+		$suite->addTestFile($path.'AllConsoleTest.php');
+		$suite->addTestFile($path.'AllBehaviorsTest.php');
+		$suite->addTestFile($path.'AllCacheTest.php');
+		$suite->addTestFile($path.'AllComponentsTest.php');
+		$suite->addTestFile($path.'AllConfigureTest.php');
+		$suite->addTestFile($path.'AllCoreTest.php');
+		$suite->addTestFile($path.'AllControllerTest.php');
+		$suite->addTestFile($path.'AllDatabaseTest.php');
+		$suite->addTestFile($path.'AllErrorTest.php');
+		$suite->addTestFile($path.'AllEventTest.php');
+		$suite->addTestFile($path.'AllHelpersTest.php');
+		$suite->addTestFile($path.'AllLogTest.php');
+		$suite->addTestFile($path.'Model'.DS.'ModelTest.php');
+		$suite->addTestFile($path.'AllRoutingTest.php');
+		$suite->addTestFile($path.'AllNetworkTest.php');
+		$suite->addTestFile($path.'AllTestSuiteTest.php');
+		$suite->addTestFile($path.'AllUtilityTest.php');
+		$suite->addTestFile($path.'AllViewTest.php');
+		$suite->addTestFile($path.'AllI18nTest.php');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/AllUtilityTest.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
  */
 	public static function suite() {
 		$suite = new CakeTestSuite('All Utility class tests');
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'Utility');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'Utility');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/AllViewTest.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 	public static function suite() {
 		$suite = new CakeTestSuite('All View class tests');
 
-		$suite->addTestDirectory(CORE_TEST_CASES . DS . 'View');
+		$suite->addTestDirectory(CORE_TEST_CASES.DS.'View');
 		return $suite;
 	}
 }

lib/Cake/Test/Case/BasicsTest.php

Spacing +65 added lines, -65 removed lines

@@ -16,7 +16,7 @@ 
  * @license       http://www.opensource.org/licenses/mit-license.php MIT License
  */
 
-require_once CAKE . 'basics.php';
+require_once CAKE.'basics.php';
 
 App::uses('Folder', 'Utility');
 App::uses('CakeResponse', 'Network');
@@ -36,7 +36,7 @@ 
 	public function setUp() {
 		parent::setUp();
 		App::build(array(
-			'Locale' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Locale' . DS)
+			'Locale' => array(CAKE.'Test'.DS.'test_app'.DS.'Locale'.DS)
 		));
 	}
 
@@ -76,7 +76,7 @@ 
  * @return void
  */
 	public function testEnv() {
-		$this->skipIf(!function_exists('ini_get') || ini_get('safe_mode') === '1', 'Safe mode is on.');
+		$this->skipIf( ! function_exists('ini_get') || ini_get('safe_mode') === '1', 'Safe mode is on.');
 
 		$server = $_SERVER;
 		$env = $_ENV;
@@ -277,13 +277,13 @@ 
 
 		Configure::write('Cache.disable', false);
 		$result = cache('basics_test', 'simple cache write');
-		$this->assertTrue((bool)$result);
-		$this->assertTrue(file_exists(CACHE . 'basics_test'));
+		$this->assertTrue((bool) $result);
+		$this->assertTrue(file_exists(CACHE.'basics_test'));
 
 		$result = cache('basics_test');
 		$this->assertEquals('simple cache write', $result);
-		if (file_exists(CACHE . 'basics_test')) {
-			unlink(CACHE . 'basics_test');
+		if (file_exists(CACHE.'basics_test')) {
+			unlink(CACHE.'basics_test');
 		}
 
 		cache('basics_test', 'expired', '+1 second');
@@ -303,56 +303,56 @@ 
 		$cacheOff = Configure::read('Cache.disable');
 		$this->skipIf($cacheOff, 'Cache is disabled, skipping clearCache() tests.');
 
-		cache('views' . DS . 'basics_test.cache', 'simple cache write');
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'basics_test.cache'));
+		cache('views'.DS.'basics_test.cache', 'simple cache write');
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'basics_test.cache'));
 
-		cache('views' . DS . 'basics_test_2.cache', 'simple cache write 2');
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'basics_test_2.cache'));
+		cache('views'.DS.'basics_test_2.cache', 'simple cache write 2');
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'basics_test_2.cache'));
 
-		cache('views' . DS . 'basics_test_3.cache', 'simple cache write 3');
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'basics_test_3.cache'));
+		cache('views'.DS.'basics_test_3.cache', 'simple cache write 3');
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'basics_test_3.cache'));
 
 		$result = clearCache(array('basics_test', 'basics_test_2'), 'views', '.cache');
 		$this->assertTrue($result);
-		$this->assertFalse(file_exists(CACHE . 'views' . DS . 'basics_test.cache'));
-		$this->assertFalse(file_exists(CACHE . 'views' . DS . 'basics_test.cache'));
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'basics_test_3.cache'));
+		$this->assertFalse(file_exists(CACHE.'views'.DS.'basics_test.cache'));
+		$this->assertFalse(file_exists(CACHE.'views'.DS.'basics_test.cache'));
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'basics_test_3.cache'));
 
 		$result = clearCache(null, 'views', '.cache');
 		$this->assertTrue($result);
-		$this->assertFalse(file_exists(CACHE . 'views' . DS . 'basics_test_3.cache'));
+		$this->assertFalse(file_exists(CACHE.'views'.DS.'basics_test_3.cache'));
 
 		// Different path from views and with prefix
-		cache('models' . DS . 'basics_test.cache', 'simple cache write');
-		$this->assertTrue(file_exists(CACHE . 'models' . DS . 'basics_test.cache'));
+		cache('models'.DS.'basics_test.cache', 'simple cache write');
+		$this->assertTrue(file_exists(CACHE.'models'.DS.'basics_test.cache'));
 
-		cache('models' . DS . 'basics_test_2.cache', 'simple cache write 2');
-		$this->assertTrue(file_exists(CACHE . 'models' . DS . 'basics_test_2.cache'));
+		cache('models'.DS.'basics_test_2.cache', 'simple cache write 2');
+		$this->assertTrue(file_exists(CACHE.'models'.DS.'basics_test_2.cache'));
 
-		cache('models' . DS . 'basics_test_3.cache', 'simple cache write 3');
-		$this->assertTrue(file_exists(CACHE . 'models' . DS . 'basics_test_3.cache'));
+		cache('models'.DS.'basics_test_3.cache', 'simple cache write 3');
+		$this->assertTrue(file_exists(CACHE.'models'.DS.'basics_test_3.cache'));
 
 		$result = clearCache('basics', 'models', '.cache');
 		$this->assertTrue($result);
-		$this->assertFalse(file_exists(CACHE . 'models' . DS . 'basics_test.cache'));
-		$this->assertFalse(file_exists(CACHE . 'models' . DS . 'basics_test_2.cache'));
-		$this->assertFalse(file_exists(CACHE . 'models' . DS . 'basics_test_3.cache'));
+		$this->assertFalse(file_exists(CACHE.'models'.DS.'basics_test.cache'));
+		$this->assertFalse(file_exists(CACHE.'models'.DS.'basics_test_2.cache'));
+		$this->assertFalse(file_exists(CACHE.'models'.DS.'basics_test_3.cache'));
 
 		// checking if empty files were not removed
-		$emptyExists = file_exists(CACHE . 'views' . DS . 'empty');
-		if (!$emptyExists) {
-			cache('views' . DS . 'empty', '');
+		$emptyExists = file_exists(CACHE.'views'.DS.'empty');
+		if ( ! $emptyExists) {
+			cache('views'.DS.'empty', '');
 		}
-		cache('views' . DS . 'basics_test.php', 'simple cache write');
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'basics_test.php'));
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'empty'));
+		cache('views'.DS.'basics_test.php', 'simple cache write');
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'basics_test.php'));
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'empty'));
 
 		$result = clearCache();
 		$this->assertTrue($result);
-		$this->assertTrue(file_exists(CACHE . 'views' . DS . 'empty'));
-		$this->assertFalse(file_exists(CACHE . 'views' . DS . 'basics_test.php'));
-		if (!$emptyExists) {
-			unlink(CACHE . 'views' . DS . 'empty');
+		$this->assertTrue(file_exists(CACHE.'views'.DS.'empty'));
+		$this->assertFalse(file_exists(CACHE.'views'.DS.'basics_test.php'));
+		if ( ! $emptyExists) {
+			unlink(CACHE.'views'.DS.'empty');
 		}
 	}
 
@@ -605,8 +605,8 @@ 
  * @return void
  */
 	public function testLogError() {
-		if (file_exists(LOGS . 'error.log')) {
-			unlink(LOGS . 'error.log');
+		if (file_exists(LOGS.'error.log')) {
+			unlink(LOGS.'error.log');
 		}
 
 		// disable stderr output for this test
@@ -621,7 +621,7 @@ 
 			CakeLog::enable('stderr');
 		}
 
-		$result = file_get_contents(LOGS . 'error.log');
+		$result = file_get_contents(LOGS.'error.log');
 		$this->assertRegExp('/Error: Testing LogError\(\) basic function/', $result);
 		$this->assertNotRegExp("/Error: Testing with\nmulti-line\nstring/", $result);
 		$this->assertRegExp('/Error: Testing with multi-line string/', $result);
@@ -633,19 +633,19 @@ 
  * @return void
  */
 	public function testFileExistsInPath() {
-		if (!function_exists('ini_set')) {
+		if ( ! function_exists('ini_set')) {
 			$this->markTestSkipped('%s ini_set function not available');
 		}
 
 		$_includePath = ini_get('include_path');
 
-		$path = TMP . 'basics_test';
-		$folder1 = $path . DS . 'folder1';
-		$folder2 = $path . DS . 'folder2';
-		$file1 = $path . DS . 'file1.php';
-		$file2 = $folder1 . DS . 'file2.php';
-		$file3 = $folder1 . DS . 'file3.php';
-		$file4 = $folder2 . DS . 'file4.php';
+		$path = TMP.'basics_test';
+		$folder1 = $path.DS.'folder1';
+		$folder2 = $path.DS.'folder2';
+		$file1 = $path.DS.'file1.php';
+		$file2 = $folder1.DS.'file2.php';
+		$file3 = $folder1.DS.'file3.php';
+		$file4 = $folder2.DS.'file4.php';
 
 		new Folder($path, true);
 		new Folder($folder1, true);
@@ -655,11 +655,11 @@ 
 		touch($file3);
 		touch($file4);
 
-		ini_set('include_path', $path . PATH_SEPARATOR . $folder1);
+		ini_set('include_path', $path.PATH_SEPARATOR.$folder1);
 
 		$this->assertEquals(fileExistsInPath('file1.php'), $file1);
 		$this->assertEquals(fileExistsInPath('file2.php'), $file2);
-		$this->assertEquals(fileExistsInPath('folder1' . DS . 'file2.php'), $file2);
+		$this->assertEquals(fileExistsInPath('folder1'.DS.'file2.php'), $file2);
 		$this->assertEquals(fileExistsInPath($file2), $file2);
 		$this->assertEquals(fileExistsInPath('file3.php'), $file3);
 		$this->assertEquals(fileExistsInPath($file4), $file4);
@@ -704,7 +704,7 @@ 
 ###########################
 
 EXPECTED;
-		$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 9);
+		$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -9);
 
 		$this->assertEquals($expected, $result);
 
@@ -719,7 +719,7 @@ 
 </pre>
 </div>
 EXPECTED;
-		$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 10);
+		$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -10);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -733,7 +733,7 @@ 
 </pre>
 </div>
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 10);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -10);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -747,7 +747,7 @@ 
 </pre>
 </div>
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 10);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -10);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -769,9 +769,9 @@ 
 
 EXPECTED;
 		if (php_sapi_name() === 'cli') {
-			$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 18);
+			$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -18);
 		} else {
-			$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 19);
+			$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -19);
 		}
 		$this->assertEquals($expected, $result);
 
@@ -794,9 +794,9 @@ 
 
 EXPECTED;
 		if (php_sapi_name() === 'cli') {
-			$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 18);
+			$expected = sprintf($expectedText, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -18);
 		} else {
-			$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 19);
+			$expected = sprintf($expectedHtml, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -19);
 		}
 		$this->assertEquals($expected, $result);
 
@@ -810,7 +810,7 @@ 
 ###########################
 
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 9);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -9);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -823,7 +823,7 @@ 
 ###########################
 
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 9);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -9);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -836,7 +836,7 @@ 
 ###########################
 
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 9);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -9);
 		$this->assertEquals($expected, $result);
 
 		ob_start();
@@ -849,7 +849,7 @@ 
 ###########################
 
 EXPECTED;
-		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ - 9);
+		$expected = sprintf($expected, str_replace(CAKE_CORE_INCLUDE_PATH, '', __FILE__), __LINE__ -9);
 		$this->assertEquals($expected, $result);
 	}
 
@@ -902,14 +902,14 @@ 
 		$this->skipIf(ini_get('magic_quotes_sybase') === '1', 'magic_quotes_sybase is on.');
 
 		$this->assertEquals(stripslashes_deep("tes\'t"), "tes't");
-		$this->assertEquals(stripslashes_deep('tes\\' . chr(0) . 't'), 'tes' . chr(0) . 't');
+		$this->assertEquals(stripslashes_deep('tes\\'.chr(0).'t'), 'tes'.chr(0).'t');
 		$this->assertEquals(stripslashes_deep('tes\"t'), 'tes"t');
 		$this->assertEquals(stripslashes_deep("tes\'t"), "tes't");
 		$this->assertEquals(stripslashes_deep('te\\st'), 'test');
 
 		$nested = array(
 			'a' => "tes\'t",
-			'b' => 'tes\\' . chr(0) . 't',
+			'b' => 'tes\\'.chr(0).'t',
 			'c' => array(
 				'd' => 'tes\"t',
 				'e' => "te\'s\'t",
@@ -919,7 +919,7 @@ 
 		);
 		$expected = array(
 			'a' => "tes't",
-			'b' => 'tes' . chr(0) . 't',
+			'b' => 'tes'.chr(0).'t',
 			'c' => array(
 				'd' => 'tes"t',
 				'e' => "te's't",
@@ -936,7 +936,7 @@ 
  * @return void
  */
 	public function testStripslashesDeepSybase() {
-		if (!(ini_get('magic_quotes_sybase') === '1')) {
+		if ( ! (ini_get('magic_quotes_sybase') === '1')) {
 			$this->markTestSkipped('magic_quotes_sybase is off');
 		}
 

lib/Cake/Test/Case/Cache/CacheTest.php

Spacing +14 added lines, -14 removed lines

@@ -36,7 +36,7 @@ 
 		Configure::write('Cache.disable', false);
 
 		$this->_defaultCacheConfig = Cache::config('default');
-		Cache::config('default', array('engine' => 'File', 'path' => TMP . 'tests'));
+		Cache::config('default', array('engine' => 'File', 'path' => TMP.'tests'));
 	}
 
 /**
@@ -59,7 +59,7 @@ 
  * @return void
  */
 	public function testConfig() {
-		$settings = array('engine' => 'File', 'path' => TMP . 'tests', 'prefix' => 'cake_test_');
+		$settings = array('engine' => 'File', 'path' => TMP.'tests', 'prefix' => 'cake_test_');
 		$results = Cache::config('new', $settings);
 		$this->assertEquals(Cache::config('new'), $results);
 		$this->assertTrue(isset($results['engine']));
@@ -101,8 +101,8 @@ 
  */
 	public function testConfigWithLibAndPluginEngines() {
 		App::build(array(
-			'Lib' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Lib' . DS),
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Lib' => array(CAKE.'Test'.DS.'test_app'.DS.'Lib'.DS),
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		), App::RESET);
 		CakePlugin::load('TestPlugin');
 
@@ -181,10 +181,10 @@ 
 		$_cacheConfigSessions = Cache::config('sessions');
 		$_cacheConfigTests = Cache::config('tests');
 
-		$result = Cache::config('sessions', array('engine' => 'File', 'path' => TMP . 'sessions'));
+		$result = Cache::config('sessions', array('engine' => 'File', 'path' => TMP.'sessions'));
 		$this->assertEquals(Cache::settings('sessions'), $result['settings']);
 
-		$result = Cache::config('tests', array('engine' => 'File', 'path' => TMP . 'tests'));
+		$result = Cache::config('tests', array('engine' => 'File', 'path' => TMP.'tests'));
 		$this->assertEquals(Cache::settings('tests'), $result['settings']);
 
 		Cache::config('sessions', $_cacheConfigSessions['settings']);
@@ -228,7 +228,7 @@ 
 		Cache::write('test_something', 'this is the test data', 'tests');
 
 		$expected = array(
-			'path' => TMP . 'sessions' . DS,
+			'path' => TMP.'sessions'.DS,
 			'prefix' => 'cake_',
 			'lock' => true,
 			'serialize' => true,
@@ -328,7 +328,7 @@ 
  */
 	public function testInitSettings() {
 		$initial = Cache::settings();
-		$override = array('engine' => 'File', 'path' => TMP . 'tests');
+		$override = array('engine' => 'File', 'path' => TMP.'tests');
 		Cache::config('for_test', $override);
 
 		$settings = Cache::settings();
@@ -344,8 +344,8 @@ 
  */
 	public function testDrop() {
 		App::build(array(
-			'Lib' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Lib' . DS),
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Lib' => array(CAKE.'Test'.DS.'test_app'.DS.'Lib'.DS),
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		), App::RESET);
 
 		$result = Cache::drop('some_config_that_does_not_exist');
@@ -396,8 +396,8 @@ 
  */
 	public function testWriteTriggerError() {
 		App::build(array(
-			'Lib' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Lib' . DS),
-			'Plugin' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'Plugin' . DS)
+			'Lib' => array(CAKE.'Test'.DS.'test_app'.DS.'Lib'.DS),
+			'Plugin' => array(CAKE.'Test'.DS.'test_app'.DS.'Plugin'.DS)
 		), App::RESET);
 
 		Cache::config('test_trigger', array('engine' => 'TestAppCache', 'prefix' => ''));
@@ -421,7 +421,7 @@ 
  */
 	public function testCacheDisable() {
 		Configure::write('Cache.disable', false);
-		Cache::config('test_cache_disable_1', array('engine' => 'File', 'path' => TMP . 'tests'));
+		Cache::config('test_cache_disable_1', array('engine' => 'File', 'path' => TMP.'tests'));
 
 		$this->assertTrue(Cache::write('key_1', 'hello', 'test_cache_disable_1'));
 		$this->assertSame(Cache::read('key_1', 'test_cache_disable_1'), 'hello');
@@ -437,7 +437,7 @@ 
 		$this->assertSame(Cache::read('key_3', 'test_cache_disable_1'), 'hello');
 
 		Configure::write('Cache.disable', true);
-		Cache::config('test_cache_disable_2', array('engine' => 'File', 'path' => TMP . 'tests'));
+		Cache::config('test_cache_disable_2', array('engine' => 'File', 'path' => TMP.'tests'));
 
 		$this->assertFalse(Cache::write('key_4', 'hello', 'test_cache_disable_2'));
 		$this->assertFalse(Cache::read('key_4', 'test_cache_disable_2'));

lib/Cake/Test/Case/Cache/Engine/ApcEngineTest.php

Spacing +4 added lines, -4 removed lines

@@ -32,10 +32,10 @@ 
  */
 	public function setUp() {
 		parent::setUp();
-		$this->skipIf(!function_exists('apc_store'), 'Apc is not installed or configured properly.');
+		$this->skipIf( ! function_exists('apc_store'), 'Apc is not installed or configured properly.');
 
 		if (php_sapi_name() === 'cli') {
-			$this->skipIf(!ini_get('apc.enable_cli'), 'APC is not enabled for the CLI.');
+			$this->skipIf( ! ini_get('apc.enable_cli'), 'APC is not enabled for the CLI.');
 		}
 
 		$this->_cacheDisable = Configure::read('Cache.disable');
@@ -147,7 +147,7 @@ 
  * @return void
  */
 	public function testDecrement() {
-		$this->skipIf(!function_exists('apc_dec'), 'No apc_dec() function, cannot test decrement().');
+		$this->skipIf( ! function_exists('apc_dec'), 'No apc_dec() function, cannot test decrement().');
 
 		$result = Cache::write('test_decrement', 5, 'apc');
 		$this->assertTrue($result);
@@ -171,7 +171,7 @@ 
  * @return void
  */
 	public function testIncrement() {
-		$this->skipIf(!function_exists('apc_inc'), 'No apc_inc() function, cannot test increment().');
+		$this->skipIf( ! function_exists('apc_inc'), 'No apc_inc() function, cannot test increment().');
 
 		$result = Cache::write('test_increment', 5, 'apc');
 		$this->assertTrue($result);