Как издеваться над классом, созданным внутри тестируемого класса?

Question

Как издеваться над классом, созданным внутри тестируемого класса?

0

В документации PHPUnit в разделе, посвященном mocks, объясняется, как прикрепить макет-объект внутри тестируемого класса:

...
$observer = $this->getMockBuilder('Observer')
                 ->setMethods(array('update'))
                 ->getMock();
...
$subject = new Subject('My subject');
$subject->attach($observer);
$subject->doSomething();

В моем случае я хочу протестировать метод Subject. Этот метод создает много таких объектов:

public function myMethod() {
    for($i = 0; $i < 10; $i++) {
        $this->collection []= new Observer(/* some data */);
    }
}

Как я могу издеваться над всеми наблюдателями?

Xavier Boubert 06 янв. 2015, в 18:12

Источник

Теги:

php

unit-testing

phpunit

1 ответ

Ещё вопросы

Schleis · Answer 1 · 2015-01-06T17-42-00.000Z

Вы не можете издеваться над классом. new оператор получит определение класса на основе вашего автозагрузчика или того, что уже является памятью. Вы не можете динамически заменить определение класса в своем тесте.

Вы могли бы создать альтернативное определение класса для Observer, которое определило бы поведение и, по сути, было бы "макетным" классом, который вы использовали бы. Проблема заключается в том, что если вы запустите этот тест с помощью тестов для фактического класса Observer, вы получите ошибку, которую вы не можете переопределить. И все становится очень сложным.

Ваш пример довольно упрощен, но я бы проверил его, убедившись, что все элементы в Subject::collection являются экземплярами Observer. Если бы были какие-то конкретные данные, которые я знал о каждом наблюдателе, я бы использовал фактические методы Observer для их проверки. (Однако это не очень хорошая практика).

Если ваш метод создает объекты, а затем вызывает методы на них, я бы серьезно пересмотрел ваш дизайн. Я бы myMethod как минимум на два метода. Один, который прикрепляет объект Observer который вы можете использовать в своем тесте, чтобы передать макет объекта в с. А другой, который называет методы Observer который будет остальной myMethod функции myMethod которой у вас нет.

Я пришел к такому выводу, но я не хочу создавать много методов attach () во многих классах. На самом деле, я не хочу адаптировать свой код к инструментам модульного тестирования.
Тебе не нужно это делать. Вы можете использовать либо абстрактный базовый класс, либо черту, чтобы указать метод attach (). Удаление нового оператора из класса не адаптирует ваш код к инструменту модульного тестирования. Это следует за хорошей дизайнерской практикой. Когда я создаю экземпляр Subject, я могу не знать, что Observer появится вместе. Обозреватель может ретранслировать на внешнюю систему, которую в моем конкретном случае использования я не хочу использовать. Или я могу захотеть изменить, какой Observer используется в моем случае использования. Из-за нового оператора я должен использовать Observer, и у меня нет выбора. Дизайн менее гибкий.
Хорошо я понял. Моя проблема в том, что я использую FuelPHP, и все классы уже загружены, когда я вызываю PhpUnit. Так что «Обозреватель» уже там.
Я не уверен, что вы полностью понимаете. Проблема не в загрузке классов, а в повторном использовании / изменении вашего кода. Существует также проблема явного определения того, что нужно вашему классу и что он использует. Если в будущем вы захотите изменить тему или добавить другого наблюдателя в определенных сценариях, это будет сложнее.
Я не думаю, что такие вещи, как new Observer(...) внутри циклов, являются плохим дизайном. Например, в контроллере мы можем использовать много объявлений классов и не будем создавать систему attach() для каждого из них.