This commit is contained in:
Sam Minnee 2017-06-29 13:54:02 +12:00
parent 9816efc97f
commit 522af3f29d

View File

@ -449,7 +449,7 @@ class SiteTreeTest extends SapphireTest
$pageAbout->delete(); $pageAbout->delete();
$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageAbout->ID));
$this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page);
$this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page);
Config::inst()->update(SiteTree::class, 'enforce_strict_hierarchy', true); Config::inst()->update(SiteTree::class, 'enforce_strict_hierarchy', true);
@ -463,7 +463,7 @@ class SiteTreeTest extends SapphireTest
$pageAbout->delete(); $pageAbout->delete();
$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageAbout->ID));
$this->assertFalse(DataObject::get_by_id('Page', $pageStaff->ID)); $this->assertFalse(DataObject::get_by_id('Page', $pageStaff->ID));
$this->assertFalse(DataObject::get_by_id('Page', $pageStaffDuplicate->ID)); $this->assertFalse(DataObject::get_by_id('Page', $pageStaffDuplicate->ID));
} }
@ -495,7 +495,7 @@ class SiteTreeTest extends SapphireTest
Versioned::set_stage(Versioned::LIVE); Versioned::set_stage(Versioned::LIVE);
$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageAbout->ID));
$this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page);
$this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page);
Versioned::set_stage(Versioned::DRAFT); Versioned::set_stage(Versioned::DRAFT);
@ -518,7 +518,7 @@ class SiteTreeTest extends SapphireTest
$parentPage->doUnpublish(); $parentPage->doUnpublish();
Versioned::set_stage(Versioned::LIVE); Versioned::set_stage(Versioned::LIVE);
$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageAbout->ID));
$this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page);
$this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page); $this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page);
Versioned::set_stage(Versioned::DRAFT); Versioned::set_stage(Versioned::DRAFT);
@ -540,9 +540,9 @@ class SiteTreeTest extends SapphireTest
$parentPage->doUnpublish(); $parentPage->doUnpublish();
Versioned::set_stage(Versioned::LIVE); Versioned::set_stage(Versioned::LIVE);
$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageAbout->ID));
$this->assertFalse(DataObject::get_by_id('Page', $pageStaff->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageStaff->ID));
$this->assertFalse(DataObject::get_by_id('Page', $pageStaffDuplicate->ID)); $this->assertNull(DataObject::get_by_id('Page', $pageStaffDuplicate->ID));
Versioned::set_stage(Versioned::DRAFT); Versioned::set_stage(Versioned::DRAFT);
} }