если в дело вступают еще и фикстуры с данными. которые пользуют зависимые сервисы - все становится очень и очень трудно

а что там трудного ?

в конечном счете ты 90% времени будешь писать тесты :D а не код

ну где-то это нормально

если денег много да

ботов действительно можно писать слабосвязанно и относительно легко тестить

сложные апи нет 😞

почему ? не вижу причин для этого

все можно слабосвязанно написать было бы желание и руки

ну вот из практики

давай пример где нельзя

есть генератор отчетов, который делает группировку произвольного уровня сложности из elasticsearch

мокать ответ сервера ты будешь примерно вечность

я мокать буду сервисы

и их в начале тестировать

потому что есть очень много композиций и вариантов группировки

ну вот у тебя есть сервис - генератор отчетов

из-за этого тестировать буду сервисы. что бы я уверен что 1+1 = 2

он зависит от сервиса - соединения с эластиксерчем

зачем его мокать? если есть и тестовая база, и фикстуры?

ну в начале покрою юнит тестами.

а потом уже возьмусь другие

а зачем? 😊

а ты уверен что из 100500 вариантов генерации отчета он будет правильно 100500 раз ?

как раз юнит тесты частично решают проблему

ну вот в этом и мякотка

что можно в функциональном тесте эти варианты сразу описать

интеграционный тест тут такое себе . на одних данных он проверит а на других ? или вы на всех вариантов проверяете ?

у тебя будут тонкие контроллеры и сервис будет протестирован

ты же не тестируешь сервис. а тестируешь эндпоинт

так эндпойнт по сути это фасад для сервиса

для работоспособности приложения необходимо и достаточно чтобы эндпойнт был рабочим

на данный внутри плевать ? =)

мы ведь для того и отделяем бл в сервис? чтоб скрыть реализацию и оставить снаружи простой интерфейс

проще на голанге переписать за некоторое небольшое время

мне гошники говорили, что это может быть и не лучше

:D

на данный внутри плевать ? =)

может быть ты и приватные методы тестируешь? :D

нет.

но у тебя там не один класс

а 100500 еще есть, я надеюсь

100500

ну вот там наверняка еще 100500 паблик методов

я про эти данные

но внешний интерфейс-то у сервиса обычно один или несколько

генератору отчета по сути надо только генерацию и запустить

и дать результат

а потом мы изменим 1+2 = 4 и в 1% ответа будут приходить не правильные данные. на хабре был похожий случай, когда что-то там не работало во вторник

эти баги не отловить если тестировать сам ендпоинт

а потом мы изменим 1+2 = 4 и в 1% ответа будут приходить не правильные данные. на хабре был похожий случай, когда что-то там не работало во вторник

добавляешь функциональный тест, фиксишь 😊

хм . а как ты поймешь где бага ? вот у юзера это сработало во вторник. а ты проверяешь в среду

можно мокать же текущую дату

в том же карбоне есть setTestNow()

ну в среду тесты погоняешь и там все хорошо. как додуматься до вторника ? =)

это все нюансы (:

Давайте я вам гавнеца подкину? ;D

https://pastebin.com/CfSbaKjb

у меня пять таких однотипных экшенов :D

куда спрятать эту ебаную логику? )

Нам своего хватает

:D

я реально устал уже думать )

шо так гавно шо эдак, сотановился на этом

return [ 'error' => [ 'code' => 1, 'message' => implode('<br>', $video->getErrorSummary()), ], ];

как печаль

как минимум надо обертку над респонсом сделать чтоб ошибки ловить

как минимум надо обертку над респонсом сделать чтоб ошибки ловить

какую? там жсон ответ

посмотреть как сериалайзер обрабатывает модели с ошибкой 😊

посмотреть как сериалайзер обрабатывает модели с ошибкой 😊

нихуя не понял, пропустим этот момент ;D

нихуя не понял, пропустим этот момент ;D

если валидация не прошла можно сделать return $model;

посмотри yii\rest\UpdateAction

это как-то неправильно

должен же респонс возвращаться вообще

куда спрятать эту ебаную логику? )

if (Yii::$app->user->can('videos.changeCategories')) { $this->caterory->change($new, $old); }

if (Yii::$app->user->can('videos.changeCategories')) { $this->caterory->change($new, $old); }

там иды передаются? а внутри типа логика? и этот метод в AR модели Category?

или как

все с 39 стороки пряч нафиг.

изначально я вообще думало использовать метод populateRelation http://www.yiiframework.com/doc-2.0/yii-db-baseactiverecord.html#populateRelation()-detail Т.е. закинуть туда новые категории, потом каким-нибудь евентом(которого нет) отследить изменение и применить изменения при сохранении

вот щас чо делать... пилить эту логику в AR моделях? :(

вынеси это куда-то с контроллера для начала

куда? вот вопрос

в отдельный класс