RO
В таком случае, я и то и то учу пока
Size: a a a
2020 October 21
RO
Изучаю*
DM
В игродел надо идти 😉
U
Привет, вопрос по rspec, было приложение на rails 4, покрыто частично тестами, переписали на rails 5 api + react/redux на фронте. Теперь rails выполняет только api функцию, логики в моделях нет, все теперь делает react/redux. Стоит ли покрывать тестами контроллеры? Они однотипные, методы вида:
По ТЗ это нужно сделать, вопрос в том, насколько это нужно?
def index
issue = Issue.all
render json: issue
end
def create
issue = Issue.new(issue_params)
if issue.save
render json: issue
else
head 422
end
end
По ТЗ это нужно сделать, вопрос в том, насколько это нужно?
DM
ну реквест тесты
DM
нужны
DM
можно хотя бы затестить 422/200 response, возможно еще какие то поля чтобы были, сериалайзера нету никакого? зачем везде все поля отдавать на фронт?
D
Привет, вопрос по rspec, было приложение на rails 4, покрыто частично тестами, переписали на rails 5 api + react/redux на фронте. Теперь rails выполняет только api функцию, логики в моделях нет, все теперь делает react/redux. Стоит ли покрывать тестами контроллеры? Они однотипные, методы вида:
По ТЗ это нужно сделать, вопрос в том, насколько это нужно?
def index
issue = Issue.all
render json: issue
end
def create
issue = Issue.new(issue_params)
if issue.save
render json: issue
else
head 422
end
end
По ТЗ это нужно сделать, вопрос в том, насколько это нужно?
ну у Вас валидация параметров проходит в моделях, следовательно нужно проверять что возвращается при разных условиях. Ну и если вы УЖЕ пишите тесты, сомневаетесь нужны ли они - тогда нужны.
E
Овладеть навыком принятие самостоятельных решений. Очень полезный навык, кстати.
это прям в FAQ можно заносить
K
это прям в FAQ можно заносить
Но для начала сам FAQ поправить, там полная жесть происходит с этими пустыми пространствами.
K
Да и таких вот фраз я могу нагенерировать 100500 штук.
U
можно хотя бы затестить 422/200 response, возможно еще какие то поля чтобы были, сериалайзера нету никакого? зачем везде все поля отдавать на фронт?
Спасибо всем за ответы. Тестам быть, лучше поздно, чем слишком поздно.
VV
Ощущение, что всё равно его мало читают
Он не для этого нужен! :)