Browse Source

Test case compatibility assertIsA() to assertInstanceOf()

Kyle Robinson Young 14 years ago
parent
commit
c97dca1570

+ 5 - 5
lib/Cake/Test/Case/Model/ModelIntegrationTest.php

@@ -997,13 +997,13 @@ class ModelIntegrationTest extends BaseModelTest {
 	public function testLoadModelSecondIteration() {
 		$this->loadFixtures('Apple', 'Message', 'Thread', 'Bid');
 		$model = new ModelA();
-		$this->assertIsA($model,'ModelA');
+		$this->assertInstanceOf('ModelA', $model);
 
-		$this->assertIsA($model->ModelB, 'ModelB');
-		$this->assertIsA($model->ModelB->ModelD, 'ModelD');
+		$this->assertInstanceOf('ModelB', $model->ModelB);
+		$this->assertInstanceOf('ModelD', $model->ModelB->ModelD);
 
-		$this->assertIsA($model->ModelC, 'ModelC');
-		$this->assertIsA($model->ModelC->ModelD, 'ModelD');
+		$this->assertInstanceOf('ModelC', $model->ModelC);
+		$this->assertInstanceOf('ModelD', $model->ModelC->ModelD);
 	}
 
 /**

+ 7 - 7
lib/Cake/Test/Case/Network/Email/CakeEmailTest.php

@@ -495,7 +495,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->assertSame($this->CakeEmail->getHeaders(array('from' => true, 'to' => true)), $expected);
 
 		$result = $this->CakeEmail->setHeaders(array());
-		$this->assertIsA($result, 'CakeEmail');
+		$this->assertInstanceOf('CakeEmail', $result);
 	}
 
 /**
@@ -616,7 +616,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->assertSame($this->CakeEmail->transport(), 'Debug');
 
 		$result = $this->CakeEmail->transportClass();
-		$this->assertIsA($result, 'DebugTransport');
+		$this->assertInstanceOf('DebugTransport', $result);
 
 		$this->setExpectedException('SocketException');
 		$this->CakeEmail->transport('Invalid');
@@ -674,7 +674,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->assertEquals($configs->test['transport'], $result);
 
 		$result = $this->CakeEmail->transportClass();
-		$this->assertIsA($result, 'DebugTransport');
+		$this->assertInstanceOf('DebugTransport', $result);
 	}
 /**
  * testSendWithContent method
@@ -879,7 +879,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->CakeEmail->viewVars(array('time' => $timestamp));
 
 		$result = $this->CakeEmail->helpers(array('Time'));
-		$this->assertIsA($result, 'CakeEmail');
+		$this->assertInstanceOf('CakeEmail', $result);
 
 		$result = $this->CakeEmail->send();
 		$this->assertTrue((bool)strpos($result['message'], 'Right now: ' . date('Y-m-d\TH:i:s\Z', $timestamp)));
@@ -1001,7 +1001,7 @@ class CakeEmailTest extends CakeTestCase {
  */
 	public function testDeliver() {
 		$instance = CakeEmail::deliver('all@cakephp.org', 'About', 'Everything ok', array('from' => 'root@cakephp.org'), false);
-		$this->assertIsA($instance, 'CakeEmail');
+		$this->assertInstanceOf('CakeEmail', $instance);
 		$this->assertSame($instance->to(), array('all@cakephp.org' => 'all@cakephp.org'));
 		$this->assertSame($instance->subject(), 'About');
 		$this->assertSame($instance->from(), array('root@cakephp.org' => 'root@cakephp.org'));
@@ -1223,7 +1223,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->assertEquals('View', $result);
 
 		$result = $this->CakeEmail->viewRender('Theme');
-		$this->assertIsA($result, 'CakeEmail');
+		$this->assertInstanceOf('CakeEmail', $result);
 
 		$result = $this->CakeEmail->viewRender();
 		$this->assertEquals('Theme', $result);
@@ -1239,7 +1239,7 @@ class CakeEmailTest extends CakeTestCase {
 		$this->assertEquals('text', $result);
 
 		$result = $this->CakeEmail->emailFormat('html');
-		$this->assertIsA($result, 'CakeEmail');
+		$this->assertInstanceOf('CakeEmail', $result);
 
 		$result = $this->CakeEmail->emailFormat();
 		$this->assertEquals('html', $result);

+ 1 - 1
lib/Cake/Test/Case/Network/Http/HttpSocketTest.php

@@ -709,7 +709,7 @@ class HttpSocketTest extends CakeTestCase {
 
 		$this->Socket->responseClass = 'CustomResponse';
 		$response = $this->Socket->request('http://www.cakephp.org/');
-		$this->assertIsA($response, 'CustomResponse');
+		$this->assertInstanceOf('CustomResponse', $response);
 		$this->assertEquals($response->first10, 'HTTP/1.x 2');
 	}
 

+ 4 - 4
lib/Cake/Test/Case/Utility/DebuggerTest.php

@@ -383,16 +383,16 @@ class DebuggerTest extends CakeTestCase {
  */
 	public function testGetInstance() {
 		$result = Debugger::getInstance();
-		$this->assertIsA($result, 'Debugger');
+		$this->assertInstanceOf('Debugger', $result);
 
 		$result = Debugger::getInstance('DebuggerTestCaseDebugger');
-		$this->assertIsA($result, 'DebuggerTestCaseDebugger');
+		$this->assertInstanceOf('DebuggerTestCaseDebugger', $result);
 
 		$result = Debugger::getInstance();
-		$this->assertIsA($result, 'DebuggerTestCaseDebugger');
+		$this->assertInstanceOf('DebuggerTestCaseDebugger', $result);
 
 		$result = Debugger::getInstance('Debugger');
-		$this->assertIsA($result, 'Debugger');
+		$this->assertInstanceOf('Debugger', $result);
 	}
 
 /**

+ 1 - 1
lib/Cake/Test/Case/Utility/FileTest.php

@@ -107,7 +107,7 @@ class FileTest extends CakeTestCase {
 		$this->assertEquals($expecting, $result);
 
 		$result = $this->File->Folder();
-		$this->assertIsA($result, 'Folder');
+		$this->assertInstanceOf('Folder', $result);
 
 		$this->skipIf(DIRECTORY_SEPARATOR === '\\', 'File permissions tests not supported on Windows.');
 

+ 2 - 2
lib/Cake/Test/Case/Utility/ObjectCollectionTest.php

@@ -144,10 +144,10 @@ class ObjectCollectionTest extends CakeTestCase {
 		$this->assertEquals(array('First'), $result, 'loaded objects are wrong');
 
 		$result = $this->Objects->set('First', new SecondGenericObject());
-		$this->assertIsA($result['First'], 'SecondGenericObject', 'set failed');
+		$this->assertInstanceOf('SecondGenericObject', $result['First'], 'set failed');
 
 		$result = $this->Objects->set('Second', new SecondGenericObject());
-		$this->assertIsA($result['Second'], 'SecondGenericObject', 'set failed');
+		$this->assertInstanceOf('SecondGenericObject', $result['Second'], 'set failed');
 
 		$this->assertEquals(count($result), 2);
 	}