EB
Либо добавить в parametrize заодно ожидаемые результаты, значит для этого теста будет 500 передаваться параметром
Size: a a a
2019 January 16
AK
или вынести 500 в отдельный негативный тест
EB
Да, потому что при нескольких параметрайзах у тебя все сочетания будут работать
b
или вынести 500 в отдельный негативный тест
а комбинации я как проверю? вот cancelled нельзя создать только для FULL_SURVEY и OPPORTUNITIES, например, но для всех остальных он нужен
СС
Если иф на ассерт 1, максимум 2 то имхо простительно. Во всех остальных случаях лучше сделать отдельный тест
b
а что плохого в ифах на ассерты?
СС
а что плохого в ифах на ассерты?
Неявность
b
ну мне просто интересно
СС
Когда у тебя будет 100 параметров и ты будешь гадать почему вот тут упал а тут нет и что же тут вообще проверяется
EB
If зло потому что ты как бы не знаешь что в этом тесте будет или не будет
EB
Чем тупее тест без подразумевающейся логики, тем лучше
СС
Короче если 1 иф сокращает 100 строк абсолбтно дублирующегося кода то ок
b
так знаю же при каких условиях падать должно, ну вернее скипать этот ассерт
b
Короче если 1 иф сокращает 100 строк абсолбтно дублирующегося кода то ок
офк
СС
В остальных случаях имхо выносить отдельно
b
если я буду пилить тесты на все комбинации я офигею :)
СС
Опять же наследованием имхо лучше рулить чем ифами
b
мне просто интересно, как можно негативные вынести из этого, если там аж три параметрайза которые между собой только в некоторых случаях конфликтуют
СС
Обычно негативные всегда пишутся отдельно, проверяют сильно меньше чем то же самое но на позитив
b
да эт понятно