Skip to content
This repository
Browse code

Fixing a couple of recently broken tests

  • Loading branch information...
commit 1814b06695f078f1153e125a7f4d4e27be818774 1 parent 5035bb9
José Lorenzo Rodríguez authored June 23, 2011
2  lib/Cake/Test/Case/Console/Command/ShellTest.php
@@ -344,7 +344,7 @@ public function testErr() {
344 344
  * @return void
345 345
  */
346 346
 	public function testNl() {
347  
-		$newLine = '\n';
  347
+		$newLine = "\n";
348 348
 		if (DS === '\\') {
349 349
 			$newLine = "\r\n";
350 350
 		}
36  lib/Cake/Test/Case/Console/ShellDispatcherTest.php
@@ -367,23 +367,25 @@ public function testParseParams() {
367 367
 		$Dispatcher->parseParams($params);
368 368
 		$this->assertEqual($expected, $Dispatcher->params);
369 369
 
370  
-		$params = array(
371  
-			'cake.php',
372  
-			'-working',
373  
-			'D:\www',
374  
-			'bake',
375  
-			'my_app',
376  
-		);
377  
-		$expected = array(
378  
-			'working' => 'D:\\\\www',
379  
-			'app' => 'www',
380  
-			'root' => 'D:\\',
381  
-			'webroot' => 'webroot'
382  
-		);
383  
-
384  
-		$Dispatcher->params = $Dispatcher->args = array();
385  
-		$Dispatcher->parseParams($params);
386  
-		$this->assertEqual($expected, $Dispatcher->params);
  370
+		if (DS === '\\') {
  371
+			$params = array(
  372
+				'cake.php',
  373
+				'-working',
  374
+				'D:\www',
  375
+				'bake',
  376
+				'my_app',
  377
+			);
  378
+			$expected = array(
  379
+				'working' => 'D:\\\\www',
  380
+				'app' => 'www',
  381
+				'root' => 'D:\\',
  382
+				'webroot' => 'webroot'
  383
+			);
  384
+
  385
+			$Dispatcher->params = $Dispatcher->args = array();
  386
+			$Dispatcher->parseParams($params);
  387
+			$this->assertEqual($expected, $Dispatcher->params);
  388
+		}
387 389
 	}
388 390
 
389 391
 /**

0 notes on commit 1814b06

Please sign in to comment.
Something went wrong with that request. Please try again.