Suilven\FreeTextSearch\Tests\Factory\IndexableMutatorPayloadTest

Complexity

Total Complexity

1

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testMutator()	0	13	1

<?php declare(strict_types = 1);

namespace Suilven\FreeTextSearch\Tests\Factory;

use SilverStripe\CMS\Model\SiteTree;
use SilverStripe\Dev\SapphireTest;
use Suilven\FreeTextSearch\Implementation\AddLinkIndexablePayloadMutator;

class IndexableMutatorPayloadTest extends SapphireTest
{
    protected static $fixture_file = ['tests/fixtures/sitetree.yml'];

    public function testMutator(): void
    {
        $page = SiteTree::get()->first();
        \error_log($page->Title);

        $payload = ['sitetree' => ['Title' => $page->Title]];
        $mutator = new AddLinkIndexablePayloadMutator();
        $mutator->mutatePayload($page, $payload);

It seems like $page can also be of type null; however, parameter $dataObject of Suilven\FreeTextSearch\Implementation\AddLinkIndexablePayloadMutator::mutatePayload() does only seem to accept SilverStripe\ORM\DataObject, maybe add an additional type check?

        $this->assertEquals([
            'Title' => 'The Break In San Marino Is Bright',
            'Link' => '/the-break-in-san-marino-is-bright/',
        ], $payload['sitetree']);
    }
}