Как протестировать защищенный метод абстрактного класса?

Question

[ganjo888]

class Document extends AbstractClass
{
    public function handle($params)
    {
        $requestBody = 'body';
        return $this->request($requestBody);
    }
}

abstract class AbstractClass 
{
    protected function request($body)
    {
  
    }
}

Пробовал так

$class = new ReflectionClass(AbstractClass::class);
$method = $class->getMethod('request');
$method->setAccessible(true);
$document = new Document();
$result =  $method->invoke($document,$params);

Получаю ошибку
TypeError: Argument 2 passed to Illuminate\Log\LogManager::info() must be of the type array, object given, c

Comments

[Дмитрий]

ну его строки затронутся тестом, при тесте handle

[ganjo888]

Дмитрий, вот именно, мне нужно как то вот эту часть вызвать в тесте
$this->request($requestBody);

[Дмитрий]

ganjo888, ну тесты метода handle вам протестируют и защищенный метод, обычно тестят публичное, защищенное и приватное прицепом протестится, те строки которые не зеленеют, значит не используются, лишнее удаляется, в шторме есть удобный плагин, который подсвечивает строки которые под тестом и без

[ganjo888]

Дмитрий, Спасибо я понял)
Подскажите пожалуйста а вот в этом случаи как быть:
handle принимает один параметр handle($params),
но в самом методе вызывается client guzzle, как я могу замокать guzzle и передать его в handle ?

[Дмитрий]

ganjo888, docs.guzzlephp.org/en/stable/testing.html#mock-handler

[dass45]

Дмитрий, плагин не подскажете для шторма?

[Дмитрий]

dass45, в поиске напишите phpunit, один из двух нужный, на панеле синий значок появится, отключать/включать подсветку

Answer 1

[user49981]

abstract class AbstractClass
{
    protected function request($body)
    {
        return Document::TEST_VALUE;
    }
}

class Document extends AbstractClass
{
    const TEST_VALUE = 403;

    public function handle($params)
    {
        $requestBody = 'body';
        return $this->request($requestBody);
    }
}

class DocumentTest
{
    public function testRequest()
    {
        $doc = new Document();
        $result = $this->invokeMethod($doc, 'request', [1]);
        self::assertEquals(Document::TEST_VALUE, $result);
    }

    private function invokeMethod(&$object, $methodName, array $parameters = array())
    {
        $reflection = new \ReflectionClass(get_class($object));
        $method = $reflection->getMethod($methodName);
        $method->setAccessible(true);

        return $method->invokeArgs($object, $parameters);
    }
}

Answer 2

[Кирилл Несмеянов]

Защищённые методы не являются частью публичного интерфейса, а значит их тестирование не подразумевается. Так же как и абстрактные классы не являются самостоятельными сущностями. Себе дороже.

Comments

[Vamp]

Как раз наоборот - protected члены являются частью публичного интерфейса. Проставляя этот модификатор автор класса даёт чётко понять, что метод предназначен для переопределения/вызова из дочернего класса. Вы не можете просто так взять и удалить protected метод из класса или кардинально поменять его сигнатуру, так как вы не знаете сколько пользовательского кода может быть завязано на него. Подобные действия требуют увеличения мажорной версии проекта (если применяется semver), прямо как с публичными методами. Вот private члены да, не являются участниками публичного интерфейса. Их область действия очень чётко ограничена и поэтому можно делать с ними что угодно, не боясь кого-нибудь сломать.

Я не вижу ничего зазорного в том, чтобы протестировать protected метод. Правда, я бы сделал полноценного наследника вместо рефлексии.