Suilven\FreeTextSearch\Tests\Factory\IndexableMutatorPayloadTest

Complexity

Total Complexity

1

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

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

<?php declare(strict_types = 1);

namespace Suilven\FreeTextSearch\Tests\Factory;

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

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 = ['Title' => $page->Title];
        $mutator = new IdentityIndexablePayloadMutator();
        $mutator->mutatePayload($page, $payload);

It seems like $page can also be of type null; however, parameter $dataObject of Suilven\FreeTextSearch\Implementation\IdentityIndexablePayloadMutator::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);
    }
}