Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

MINOR Replaced assertType() calls with assertInstanceOf(), deprecated…

… in PHPUnit 3.6 (throws warnings as of 3.5.10)
  • Loading branch information...
commit 65f6104cd6aaad00b3fbb4a6a1c13e431afa3f9a 1 parent ca2e80e
@chillu chillu authored
View
8 tests/DataObjectDecoratorTest.php
@@ -41,10 +41,10 @@ function testOneToManyAssociationWithDecorator() {
$object = DataObject::get_one('DataObjectDecoratorTest_RelatedObject', "\"ContactID\" = {$contactID}");
$this->assertNotNull($object, 'Related object not null');
- $this->assertType('DataObjectDecoratorTest_Member', $object->Contact(), 'Related contact is a member dataobject');
- $this->assertType('DataObjectDecoratorTest_Member', $object->getComponent('Contact'), 'getComponent does the same thing as Contact()');
+ $this->assertInstanceOf('DataObjectDecoratorTest_Member', $object->Contact(), 'Related contact is a member dataobject');
+ $this->assertInstanceOf('DataObjectDecoratorTest_Member', $object->getComponent('Contact'), 'getComponent does the same thing as Contact()');
- $this->assertType('DataObjectDecoratorTest_RelatedObject', $contact->RelatedObjects()->First());
+ $this->assertInstanceOf('DataObjectDecoratorTest_RelatedObject', $contact->RelatedObjects()->First());
$this->assertEquals("Lorem ipsum dolor", $contact->RelatedObjects()->First()->FieldOne);
$this->assertEquals("Random notes", $contact->RelatedObjects()->First()->FieldTwo);
$contact->delete();
@@ -143,7 +143,7 @@ function testPopulateDefaults() {
function testDbObjectOnDecoratedFields() {
$member = $this->objFromFixture('DataObjectDecoratorTest_Member', 'member1');
$this->assertNotNull($member->dbObject('Website'));
- $this->assertType('Varchar', $member->dbObject('Website'));
+ $this->assertInstanceOf('Varchar', $member->dbObject('Website'));
}
function testDecoratorCanBeAppliedToDataObject() {
View
4 tests/DataObjectTest.php
@@ -685,7 +685,7 @@ function testNewClassInstance() {
$changedFields = $changedPage->getChangedFields();
// Don't write the record, it will reset changed fields
- $this->assertType('RedirectorPage', $changedPage);
+ $this->assertInstanceOf('RedirectorPage', $changedPage);
$this->assertEquals($changedPage->ClassName, 'RedirectorPage');
$this->assertEquals($changedPage->RedirectionType, 'Internal');
//$this->assertEquals($changedPage->RecordClassName, 'RedirectorPage');
@@ -694,7 +694,7 @@ function testNewClassInstance() {
$this->assertEquals($changedFields['ClassName']['after'], 'RedirectorPage');
$changedPage->write();
- $this->assertType('RedirectorPage', $changedPage);
+ $this->assertInstanceOf('RedirectorPage', $changedPage);
$this->assertEquals($changedPage->ClassName, 'RedirectorPage');
}
View
8 tests/ObjectTest.php
@@ -177,19 +177,19 @@ public function testGetExtensions() {
$inst = new ObjectTest_ExtensionTest();
$extensions = $inst->getExtensionInstances();
$this->assertEquals(count($extensions), 2);
- $this->assertType(
+ $this->assertInstanceOf(
'ObjectTest_ExtendTest1',
$extensions['ObjectTest_ExtendTest1']
);
- $this->assertType(
+ $this->assertInstanceOf(
'ObjectTest_ExtendTest2',
$extensions['ObjectTest_ExtendTest2']
);
- $this->assertType(
+ $this->assertInstanceOf(
'ObjectTest_ExtendTest1',
$inst->getExtensionInstance('ObjectTest_ExtendTest1')
);
- $this->assertType(
+ $this->assertInstanceOf(
'ObjectTest_ExtendTest2',
$inst->getExtensionInstance('ObjectTest_ExtendTest2')
);
View
2  tests/RequestHandlingTest.php
@@ -22,7 +22,7 @@ class RequestHandlingTest extends FunctionalTest {
function testConstructedWithNullRequest() {
$r = new RequestHandler();
- $this->assertType('NullHTTPRequest', $r->getRequest());
+ $this->assertInstanceOf('NullHTTPRequest', $r->getRequest());
}
function testRequestHandlerChainingAllParams() {
View
2  tests/SearchContextTest.php
@@ -108,7 +108,7 @@ function testRelationshipObjectsLinkedInSearch() {
$project = $results->First();
- $this->assertType('SearchContextTest_Project', $project);
+ $this->assertInstanceOf('SearchContextTest_Project', $project);
$this->assertEquals("Blog Website", $project->Name);
$this->assertEquals(2, $project->Actions()->Count());
View
4 tests/SiteTreeActionsTest.php
@@ -64,7 +64,7 @@ function testActionsNoDeletePublishedRecord() {
// Get the live version of the page
$page = Versioned::get_one_by_stage("SiteTree", "Live", "\"SiteTree\".\"ID\" = $pageID");
- $this->assertType("SiteTree", $page);
+ $this->assertInstanceOf("SiteTree", $page);
// Check that someone without the right permission can't delete the page
$editor = $this->objFromFixture('Member', 'cmsnodeleteeditor');
@@ -116,7 +116,7 @@ function testActionsDeletedFromStageRecord() {
// Get the live version of the page
$page = Versioned::get_one_by_stage("SiteTree", "Live", "\"SiteTree\".\"ID\" = $pageID");
- $this->assertType('SiteTree', $page);
+ $this->assertInstanceOf('SiteTree', $page);
$actionsArr = $page->getCMSActions()->column('Name');
View
4 tests/SiteTreeTest.php
@@ -143,7 +143,7 @@ function testParentNodeCachedInMemory() {
$child->Title = 'Page Title';
$child->setParent($parent);
- $this->assertType("SiteTree", $child->Parent);
+ $this->assertInstanceOf("SiteTree", $child->Parent);
$this->assertEquals("Section Title", $child->Parent->Title);
}
@@ -152,7 +152,7 @@ function testParentModelReturnType() {
$child = new SiteTreeTest_PageNode();
$child->setParent($parent);
- $this->assertType('SiteTreeTest_PageNode', $child->Parent);
+ $this->assertInstanceOf('SiteTreeTest_PageNode', $child->Parent);
}
/**
View
2  tests/control/DirectorTest.php
@@ -165,7 +165,7 @@ public function testTestRequestCarriesGlobals() {
$url = 'DirectorTestRequest_Controller/' . sprintf($testfunction, ucfirst($method)) . '?' . http_build_query($fixture);
$getresponse = Director::test($url, $fixture, null, strtoupper($method), null, null, $fixture);
- $this->assertType('SS_HTTPResponse', $getresponse, 'Director::test() returns SS_HTTPResponse');
+ $this->assertInstanceOf('SS_HTTPResponse', $getresponse, 'Director::test() returns SS_HTTPResponse');
$this->assertEquals($fixture['somekey'], $getresponse->getBody(), 'Director::test() ' . $testfunction);
}
}
View
6 tests/dev/LogTest.php
@@ -18,18 +18,18 @@ function setUp() {
function testExistingWriter() {
$writers = SS_Log::get_writers();
- $this->assertType('array', $writers);
+ $this->assertInternalType('array', $writers);
$this->assertEquals(2, count($writers));
}
function testRemoveWriter() {
SS_Log::remove_writer($this->testEmailWriter);
$writers = SS_Log::get_writers();
- $this->assertType('array', $writers);
+ $this->assertInternalType('array', $writers);
$this->assertEquals(1, count($writers));
SS_Log::remove_writer($this->testFileWriter);
$writers = SS_Log::get_writers();
- $this->assertType('array', $writers);
+ $this->assertInternalType('array', $writers);
$this->assertEquals(0, count($writers));
}
View
4 tests/filesystem/FileTest.php
@@ -27,10 +27,10 @@ function testCreateWithFilenameWithSubfolder() {
$this->assertEquals($testfilePath, $file->Filename, '"Filename" property remains unchanged');
// TODO This should be auto-detected, see File->updateFilesystem()
- // $this->assertType('Folder', $file->Parent(), 'Parent folder is created in database');
+ // $this->assertInstanceOf('Folder', $file->Parent(), 'Parent folder is created in database');
// $this->assertFileExists($file->Parent()->getFullPath(), 'Parent folder is created on filesystem');
// $this->assertEquals('FileTest', $file->Parent()->Name);
- // $this->assertType('Folder', $file->Parent()->Parent(), 'Grandparent folder is created in database');
+ // $this->assertInstanceOf('Folder', $file->Parent()->Parent(), 'Grandparent folder is created in database');
// $this->assertFileExists($file->Parent()->Parent()->getFullPath(), 'Grandparent folder is created on filesystem');
// $this->assertEquals('assets', $file->Parent()->Parent()->Name);
}
View
6 tests/forms/FormTest.php
@@ -243,12 +243,12 @@ function testGloballyDisabledSecurityTokenInheritsToNewForm() {
SecurityToken::enable();
$form1 = $this->getStubForm();
- $this->assertType('SecurityToken', $form1->getSecurityToken());
+ $this->assertInstanceOf('SecurityToken', $form1->getSecurityToken());
SecurityToken::disable();
$form2 = $this->getStubForm();
- $this->assertType('NullSecurityToken', $form2->getSecurityToken());
+ $this->assertInstanceOf('NullSecurityToken', $form2->getSecurityToken());
SecurityToken::enable();
}
@@ -257,7 +257,7 @@ function testDisableSecurityTokenDoesntAddTokenFormField() {
SecurityToken::enable();
$formWithToken = $this->getStubForm();
- $this->assertType(
+ $this->assertInstanceOf(
'HiddenField',
$formWithToken->Fields()->fieldByName(SecurityToken::get_default_name()),
'Token field added by default'
View
8 tests/model/MoneyTest.php
@@ -21,14 +21,14 @@ class MoneyTest extends SapphireTest {
function testMoneyFieldsReturnedAsObjects() {
$obj = $this->objFromFixture('MoneyTest_DataObject', 'test1');
- $this->assertType('Money', $obj->MyMoney);
+ $this->assertInstanceOf('Money', $obj->MyMoney);
}
function testLoadFromFixture() {
$obj = $this->objFromFixture('MoneyTest_DataObject', 'test1');
- $this->assertType('Money', $obj->MyMoney);
+ $this->assertInstanceOf('Money', $obj->MyMoney);
$this->assertEquals($obj->MyMoney->getCurrency(), 'EUR');
$this->assertEquals($obj->MyMoney->getAmount(), 1.23);
}
@@ -42,7 +42,7 @@ function testDataObjectChangedFields() {
$this->assertNotContains('MyMoney', array_keys($changed));
// With changes
- $this->assertType('Money', $obj->MyMoney);
+ $this->assertInstanceOf('Money', $obj->MyMoney);
$obj->MyMoney->setAmount(99);
$changed = $obj->getChangedFields();
$this->assertContains('MyMoney', array_keys($changed));
@@ -280,7 +280,7 @@ function testHasValue() {
function testLoadIntoDataObject() {
$obj = new MoneyTest_DataObject();
- $this->assertType('Money', $obj->obj('MyMoney'));
+ $this->assertInstanceOf('Money', $obj->obj('MyMoney'));
$m = new Money();
$m->setValue(array(
View
2  tests/model/SiteConfigTest.php
@@ -39,7 +39,7 @@ function testCurrentCreatesDefaultForLocale() {
$configEn = SiteConfig::current_site_config();
$configFr = SiteConfig::current_site_config('fr_FR');
- $this->assertType('SiteConfig', $configFr);
+ $this->assertInstanceOf('SiteConfig', $configFr);
$this->assertEquals($configFr->Locale, 'fr_FR');
$this->assertEquals($configFr->Title, $configEn->Title, 'Copies title from existing config');
}
View
8 tests/model/TranslatableTest.php
@@ -220,7 +220,7 @@ function testUpdateCMSFieldsOnSiteTree() {
// first test with default language
$fields = $pageOrigLang->getCMSFields();
- $this->assertType(
+ $this->assertInstanceOf(
'TextField',
$fields->dataFieldByName('Title'),
'Translatable doesnt modify fields if called in default language (e.g. "non-translation mode")'
@@ -233,13 +233,13 @@ function testUpdateCMSFieldsOnSiteTree() {
// then in "translation mode"
$pageTranslated = $pageOrigLang->createTranslation('fr_FR');
$fields = $pageTranslated->getCMSFields();
- $this->assertType(
+ $this->assertInstanceOf(
'TextField',
$fields->dataFieldByName('Title'),
'Translatable leaves original formfield intact in "translation mode"'
);
$readonlyField = $fields->dataFieldByName('Title')->performReadonlyTransformation();
- $this->assertType(
+ $this->assertInstanceOf(
$readonlyField->class,
$fields->dataFieldByName('Title_original'),
'Translatable adds the original value as a ReadonlyField in "translation mode"'
@@ -793,7 +793,7 @@ function testCanTranslateAllowedLocales() {
"Users with canEdit() and TRANSLATE_ALL permission can't create a new translation if locale is not in Translatable::get_allowed_locales()"
);
- $this->assertType(
+ $this->assertInstanceOf(
'Page',
$testPage->createTranslation('ja_JP')
);
View
4 tests/model/VersionedTest.php
@@ -94,11 +94,11 @@ function testGetIncludingDeleted() {
function testVersionedFieldsAdded() {
$obj = new VersionedTest_DataObject();
// Check that the Version column is added as a full-fledged column
- $this->assertType('Int', $obj->dbObject('Version'));
+ $this->assertInstanceOf('Int', $obj->dbObject('Version'));
$obj2 = new VersionedTest_Subclass();
// Check that the Version column is added as a full-fledged column
- $this->assertType('Int', $obj2->dbObject('Version'));
+ $this->assertInstanceOf('Int', $obj2->dbObject('Version'));
}
function testPublishCreateNewVersion() {
View
2  tests/security/MemberTest.php
@@ -150,7 +150,7 @@ function testPasswordChangeLogging() {
$this->assertTrue($record->checkPassword('test1'), "Password test1 not found in MemberRecord");
$record = $passwords->next();
- $this->assertType('DataObject', $record);
+ $this->assertInstanceOf('DataObject', $record);
$this->assertTrue($record->checkPassword('1nitialPassword'), "Password 1nitialPassword not found in MemberRecord");
}
View
2  tests/security/PasswordEncryptorTest.php
@@ -3,7 +3,7 @@ class PasswordEncryptorTest extends SapphireTest {
function testCreateForCode() {
PasswordEncryptor::register('test', 'PasswordEncryptorTest_TestEncryptor');
$e = PasswordEncryptor::create_for_algorithm('test');
- $this->assertType(
+ $this->assertInstanceOf(
'PasswordEncryptorTest_TestEncryptor',
$e
);
View
2  tests/security/SecurityDefaultAdminTest.php
@@ -40,7 +40,7 @@ function testFindAnAdministratorCreatesNewUser() {
$admin = Security::findAnAdministrator();
- $this->assertType('Member', $admin);
+ $this->assertInstanceOf('Member', $admin);
$this->assertTrue(Permission::checkMember($admin, 'ADMIN'));
$this->assertNull($admin->Email);
$this->assertNull($admin->Password);
View
6 tests/security/SecurityTokenTest.php
@@ -41,7 +41,7 @@ function testIsEnabledStatic() {
function testInst() {
$inst1 = SecurityToken::inst();
- $this->assertType('SecurityToken', $inst1);
+ $this->assertInstanceOf('SecurityToken', $inst1);
}
function testInstReturnsSingleton() {
@@ -111,7 +111,7 @@ function testUpdateFieldSet() {
$t->updateFieldSet($fs);
$f = $fs->dataFieldByName($t->getName());
- $this->assertType('HiddenField', $f);
+ $this->assertInstanceOf('HiddenField', $f);
$this->assertEquals($f->Name(), $t->getName(), 'Name matches');
$this->assertEquals($f->Value(), $t->getValue(), 'Value matches');
}
@@ -123,7 +123,7 @@ function testUpdateFieldSetDoesntAddTwice() {
$t->updateFieldSet($fs); // second
$f = $fs->dataFieldByName($t->getName());
- $this->assertType('HiddenField', $f);
+ $this->assertInstanceOf('HiddenField', $f);
$this->assertEquals(1, $fs->Count());
}
Please sign in to comment.
Something went wrong with that request. Please try again.