EOffice\Testing\Concerns\InteractsWithResourceProperties

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testMutableProperties()	0	10	1

<?php

/*
 * This file is part of the EOffice project.
 *
 * (c) Anthonius Munthi <https://itstoni.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace EOffice\Testing\Concerns;

trait InteractsWithResourceProperties
{
    /**
     * @param string                  $propertyName
     * @param mixed|string|array|null $value
     * @dataProvider getPropertiesToTests
     */
    public function testMutableProperties(string $propertyName, $value): void
    {
        $class = $this->getResourceClassName();
        $ob    = new $class('test', 'test');

        $setter = 'set'.$propertyName;
        $getter = 'get'.$propertyName;

        \call_user_func([$ob, $setter], $value);
        $this->assertSame($value, \call_user_func([$ob, $getter]));

It seems like assertSame() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    abstract public function getPropertiesToTests(): array;

    /**
     * @return string|class-string

The doc comment string|class-string at position 2 could not be parsed: Unknown type name 'class-string' at position 2 in string|class-string.

     */
    abstract protected function getResourceClassName(): string;
}