DNADesign\Elemental\Tests\TopPage\TopPageTest

Complexity

Total Complexity

12

Size/Duplication

Total Lines	273
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	testNewBlock()	0	20	2
A	testTestGetTopPage()	0	16	1
A	testTestUpdateTopPageEmptyCache()	0	26	1
A	testPageDuplication()	0	49	2
A	testNewPage()	0	9	1
A	objectsProvider()	0	50	1
A	populateTopPageProvider()	0	5	1
A	setUp()	0	8	1
A	populateTopPageForAllObjects()	0	13	2

<?php

namespace DNADesign\Elemental\Tests\TopPage;

use DNADesign\Elemental\Extensions\ElementalPageExtension;
use DNADesign\Elemental\Models\BaseElement;
use DNADesign\Elemental\Models\ElementalArea;
use DNADesign\Elemental\TopPage;
use Page;

The type Page was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Dev\SapphireTest;
use SilverStripe\ORM\DataObject;

class TopPageTest extends SapphireTest
{
    /**
     * @var string
     */
    protected static $fixture_file = 'TopPageTest.yml';

    /**
     * @var array
     */
    protected static $required_extensions = [
        TestBlockPage::class => [
            ElementalPageExtension::class,
        ],
        TestChildPage::class => [
            ElementalPageExtension::class,
        ],
        Page::class => [
            TopPage\SiteTreeExtension::class,
        ],
        ElementalArea::class => [
            TopPage\DataExtension::class,
        ],
        BaseElement::class => [
            TopPage\DataExtension::class,
        ],
    ];

    /**
     * @var array
     */
    protected static $extra_dataobjects = [
        TestContent::class,
        TestList::class,
        TestBlockPage::class,
        TestChildPage::class,
    ];

    protected function setUp(): void
    {
        /** @var TopPage\DataExtension $extension */
        $extension = singleton(TopPage\DataExtension::class);

        // this makes sure that the data objects start with no top page data
        $extension->withoutTopPageUpdate(function () {
            parent::setUp();
        });
    }

    /**
     * @param string $pageIdentifier
     * @param string $pageClass
     * @param string $objectIdentifier
     * @param string $objectClass
     * @dataProvider objectsProvider
     */
    public function testTestGetTopPage(
        string $pageIdentifier,
        string $pageClass,
        string $objectIdentifier,
        string $objectClass
    ): void {
        /** @var Page|TopPage\SiteTreeExtension $content */
        $page = $this->objFromFixture($pageClass, $pageIdentifier);

        /** @var DataObject|TopPage\DataExtension $object */
        $object = $this->objFromFixture($objectClass, $objectIdentifier);

        $topPage = $object->getTopPage();

        $this->assertNotNull($topPage);
        $this->assertEquals((int) $page->ID, (int) $topPage->ID);
    }

    /**
     * @param string $pageIdentifier
     * @param string $pageClass
     * @param string $objectIdentifier
     * @param string $objectClass
     * @dataProvider objectsProvider
     */
    public function testTestUpdateTopPageEmptyCache(
        string $pageIdentifier,
        string $pageClass,
        string $objectIdentifier,
        string $objectClass
    ): void {
        /** @var Page|TopPage\SiteTreeExtension $content */
        $page = $this->objFromFixture($pageClass, $pageIdentifier);

        /** @var DataObject|TopPage\DataExtension $object */
        $object = $this->objFromFixture($objectClass, $objectIdentifier);

        $this->assertEquals(0, (int) $object->TopPageID);

        $object->forceChange();
        $id = $object->write();
        $object = DataObject::get($object->ClassName)->byID($id);

        $this->assertEquals((int) $page->ID, (int) $object->TopPageID);

        // do a second write to make sure that we won't override existing top page
        $object->forceChange();
        $id = $object->write();
        $object = DataObject::get($object->ClassName)->byID($id);

        $this->assertEquals((int) $page->ID, (int) $object->TopPageID);
    }

    public function testNewPage(): void
    {
        $page = TestBlockPage::create();
        $page->Title = 'New page test';
        $page->write();

        /** @var ElementalArea|TopPage\DataExtension $area */
        $area = $page->ElementalArea();
        $this->assertEquals((int) $page->ID, (int) $area->TopPageID);

The property TopPageID does not exist on DNADesign\Elemental\Models\ElementalArea. Since you implemented __get, consider adding a @property annotation.

    }

    /**
     * @param bool $populateTopPage
     * @dataProvider populateTopPageProvider
     */
    public function testNewBlock(bool $populateTopPage): void
    {
        if ($populateTopPage) {
            $this->populateTopPageForAllObjects();
        }

        /** @var TestBlockPage $page */
        $page = $this->objFromFixture(TestBlockPage::class, 'block-page1');

        /** @var ElementalArea $area */
        $area = $this->objFromFixture(ElementalArea::class, 'area3');

        /** @var TestContent|TopPage\DataExtension $content */
        $content = TestContent::create();
        $content->Title = 'Fresh block';

The property Title does not seem to exist on DNADesign\Elemental\TopPage\DataExtension.

        $area->Elements()->add($content);

It seems like $content can also be of type DNADesign\Elemental\TopPage\DataExtension; however, parameter $item of SilverStripe\ORM\HasManyList::add() does only seem to accept SilverStripe\ORM\DataObject|integer, maybe add an additional type check?

        $content = DataObject::get($content->ClassName)->byID($content->ID);

The property ClassName does not seem to exist on DNADesign\Elemental\TopPage\DataExtension.

The property ID does not seem to exist on DNADesign\Elemental\TopPage\DataExtension.

        $this->assertEquals((int) $page->ID, (int) $content->TopPageID);
    }

    public function testPageDuplication(): void
    {
        $this->populateTopPageForAllObjects();

        /** @var TestBlockPage $page */
        $page = $this->objFromFixture(TestBlockPage::class, 'block-page1');

        /** @var TestChildPage $childPage */
        $childPage = $this->objFromFixture(TestChildPage::class, 'child-page1');

        $page->duplicate();
        $pages = TestBlockPage::get()->filter(['Title' => 'BlockPage1'])->sort('ID', 'DESC');

        $this->assertCount(2, $pages);

        $pageClone = $pages->first();

The method first() does not exist on Traversable. It seems like you code against a sub-type of Traversable such as IntlCodePointBreakIterator or IntlRuleBasedBreakIterator or IntlBreakIterator or SilverStripe\ORM\SS_List or SilverStripe\View\ViewableData or Symfony\Component\DomCrawler\Crawler or SilverStripe\ORM\Connect\Query.

The method first() does not exist on Countable. It seems like you code against a sub-type of Countable such as SilverStripe\ORM\SS_List or Symfony\Component\DomCrawler\Crawler.

        $childPages = TestChildPage::get()->filter(['Title' => 'ChildPage1'])->sort('ID', 'DESC');

        $this->assertCount(2, $childPages);

        $childClone = $childPages->first();

        $this->assertNotEquals((int) $childPage->ID, (int) $childClone->ID);

        $objects = [
            [TestList::class, 'List1', $pageClone],
            [TestContent::class, 'Content1', $pageClone],
            [TestList::class, 'List2', $childClone],
            [TestContent::class, 'Content2', $childClone],
        ];

        foreach ($objects as $objectData) {
            $class = array_shift($objectData);
            $title = array_shift($objectData);
            $page = array_shift($objectData);

            $items = DataObject::get($class)->filter(['Title' => $title])->sort('ID', 'DESC');

            $this->assertCount(2, $items);

            /** @var DataObject|TopPage\DataExtension $objectClone */
            $objectClone = $items->first();

            $this->assertEquals((int) $page->ID, (int) $objectClone->TopPageID);

            /** @var ElementalArea|TopPage\DataExtension $area */
            $area = $objectClone->Parent();

            $this->assertEquals((int) $page->ID, (int) $area->TopPageID);

The property TopPageID does not exist on DNADesign\Elemental\Models\ElementalArea. Since you implemented __get, consider adding a @property annotation.

        }
    }

    public function objectsProvider(): array
    {
        return [
            [
                'block-page1',
                TestBlockPage::class,
                'content1',
                TestContent::class,
            ],
            [
                'block-page1',
                TestBlockPage::class,
                'list1',
                TestList::class,
            ],
            [
                'block-page1',
                TestBlockPage::class,
                'area1',
                ElementalArea::class,
            ],
            [
                'block-page1',
                TestBlockPage::class,
                'area3',
                ElementalArea::class,
            ],
            [
                'child-page1',
                TestChildPage::class,
                'content2',
                TestContent::class,
            ],
            [
                'child-page1',
                TestChildPage::class,
                'list2',
                TestList::class,
            ],
            [
                'child-page1',
                TestChildPage::class,
                'area2',
                ElementalArea::class,
            ],
            [
                'child-page1',
                TestChildPage::class,
                'area4',
                ElementalArea::class,
            ],
        ];
    }

    public function populateTopPageProvider(): array
    {
        return [
            [true],
            [false],
        ];
    }

    private function populateTopPageForAllObjects(): void
    {
        $list = $this->objectsProvider();

        foreach ($list as $objects) {
            array_shift($objects);
            array_shift($objects);
            $identifier = array_shift($objects);
            $class = array_shift($objects);

            $object = $this->objFromFixture($class, $identifier);
            $object->forceChange();
            $object->write();
        }
    }
}