Merge pull request #263 from creative-commoners/pulls/3.0/complete-ch…

…eckOwnerPage-test Add tests for getOwnerPage() method of ElementalArea
silverstripe · Jul 17, 2018 · d2bc854 · d2bc854
2 parents 7c63028 + ace6269
commit d2bc854
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/tests/ElementalAreaTest.php b/tests/ElementalAreaTest.php
@@ -35,7 +35,16 @@ public function testElementControllers()
 
     public function testGetOwnerPage()
     {
-        $this->markTestIncomplete();
+        $area1 = $this->objFromFixture(ElementalArea::class, 'area1');
+        $area2 = $this->objFromFixture(ElementalArea::class, 'area2');
+
+        // OwnerClassName not set
+        $ownerpage1 = $area1->getOwnerPage();
+        // OwnerClassName set
+        $ownerpage2 = $area2->getOwnerPage();
+
+        $this->assertEquals("DNADesign\Elemental\Tests\Src\TestPage", $ownerpage1);
+        $this->assertEquals("DNADesign\Elemental\Tests\Src\TestPage", $ownerpage2);
     }
 
     public function testForTemplate()

diff --git a/tests/ElementalAreaTest.yml b/tests/ElementalAreaTest.yml
@@ -1,12 +1,19 @@
 DNADesign\Elemental\Models\ElementalArea:
   area1:
     Title: Area 1
+  area2:
+    Title: Area 2
+    OwnerClassName: DNADesign\Elemental\Tests\Src\TestPage
 
 DNADesign\Elemental\Tests\Src\TestPage:
   page1:
     Title: Page 1
     URLSegment: test-page
     ElementalAreaID: =>DNADesign\Elemental\Models\ElementalArea.area1
+  page2:
+    Title: Page 2
+    URLSegment: test-page-2
+    ElementalAreaID: =>DNADesign\Elemental\Models\ElementalArea.area2
 
 DNADesign\Elemental\Tests\Src\TestElement:
   element1: