Какова последовательность выполнения PHPUnit тестов?

Question

[zbender]

В классе PHPUnit есть зависимость тестов.

public function test1(){
...
}

/**
* @depends test1
*/
public function test2(){
...
}

/**
* @depends test1
*/
public function test3(){
...
}

При запуске теста, он выполняется в следующей последовательности: test3, test1, test2. Т.к. test3 зависит от test1 (а он еще не выполнен) - test3 пропускается.
Не пойму, почему именно такая последовательность выполнения и как её задать?

Answer 1

[Trrrrr]

Вообще тесты должны быть независимыми.
Нельзя ли как то переделать код, что бы не было такой проблемы?

Comments

[zbender]

Про то, что желательно делать тесты независимыми я знаю. Но из-за нехватки опыта и знаний не знаю как лучше это реализовать.
Код переделать можно, но будет проще задать последовательность тестов.

[Trrrrr]

А можно пример как так вышло?

[zbender]

test1 - create (создаёт объект (запись в БД))
test2 - change (изменяет объект)
test3 - remove (удаляет объект)

Можно до выполнения тестов создавать объект, но думаю, что лучше и проще использовать тот, что создавался в create.

[Trrrrr]

каждый тест должен быть самодостаточен,
возможно их вообще в 1 тест необходимо объединить - создал изменил уничтожил.

[zbender]

в 1 тест их точно нельзя объединить. в каждом тесте несколько входных параметров (которые задаются с помощью провайдера). Каждый тест проверяет отдельный метод класса, который проверяется. С помощью провайдера (т.е. различных входных параметров метода) обеспечивается максимальная покрытие кода этого метода.
На счет самодостаточности - это усложнит тесты (их написание и структуру) и при этом при рефакторинге методов и необходимости последующего изменения тестов, оно тоже усложнится. Если я, не ошибаюсь, и это действительно так, то зачем нужна самодостаточность тестов?

[Trrrrr]

я имею ввиду, что в начале теста если что то создаешь, то оно должно уничтожаться в конце теста, это однозначно.

[zbender]

так и есть, что я создаю в create очищается в конце тестов

[zbender]

ну и на счет однозначности уничтожения в конце тестов, я, наверное, не соглашусь. Это зависит от ситуации. У меня используется отдельная база для phpunit и, в большинстве случаях, не удаление созданных данных в тестах влияет только на "захламление" базы, которая периодически ресторится.

[Trrrrr]

В том виде, в котором вы это пишите я бы такие юниттесты не принял бы у программиста:
1) Тест который проверяет создание объекта - создаете объект с разными вариантами создания, в каждом проверяете что все создалось как хотели, в конце все уничтожаете.
2) Тест на уничтожение, создаете объекты, уничтожаете, убеждаетесь что его больше нет.
3) Тест на работу с объектом, опять создаете, меняете, проверяете, уничтожаете.

[zbender]

А если это усложнит написание, рефакторинг и читабельность кода - в чем выгода от этих правил?

[Trrrrr]

ничего не усложняется, все одинаковые действия выносятся в базовый класс, одинаковый код писать не надо. Но вот в вашем случае такая ситуация, ты ушел в отпуск, вместо тебя сел человек, начал писать еще пару тестов, но он не знал, что у тебя порядок важен и все поломал добавлением новых тестов. В итоге тебе звонят прямо в Турцию)

[zbender]

Убедительно. Согласен, что это будет правильно.