
Антон
06.03.2017
11:33:58
если 200 тогда уже не нил :(

Evgeny
06.03.2017
11:34:11
Ну смотри я продублирую

Alex
06.03.2017
11:34:24
подожди, ты говоришь что response.code == 200
и при этом response == nil

Google

Alex
06.03.2017
11:34:30
Верно?
nil сам response и все что далее соотвественно
response = post photos_path

Evgeny
06.03.2017
11:35:13

Alex
06.03.2017
11:35:13
но это не очень красиво.

Антон
06.03.2017
11:35:14
добавь expect(response).to be_success а то у меня какое-то недоверие

Alex
06.03.2017
11:35:22
ты byebug поставил ДО запроса
а ты поставь после и проверяй уже ....

Evgeny
06.03.2017
11:35:37
Я сдкелал вручную запрос

Alex
06.03.2017
11:35:51
response = post photos_path

Evgeny
06.03.2017
11:36:01
шас

Alex
06.03.2017
11:37:52
response доступен?

Google

Alex
06.03.2017
11:37:56
вот прям сейчас

Evgeny
06.03.2017
11:38:12
Вот такая ерунда. Я остальные контроллеры покрыл тестами, все ок было там с респонс, чем здесь отличается ума не приложу

Alex
06.03.2017
11:38:22
и никогда не пиши response = JSON.parse(response.body)

Evgeny
06.03.2017
11:38:29
nil
Да я это так для примера

Alex
06.03.2017
11:38:47
json = JSON.parse(response.body)

Evgeny
06.03.2017
11:38:47
Встал колом

Alex
06.03.2017
11:38:51
даже для примера )

Evgeny
06.03.2017
11:39:01
)

Alex
06.03.2017
11:39:09
весь код покажи

Антон
06.03.2017
11:39:27
nil
is_expected.to be_success
expect(response.body).not_to be_blank
только да, не стоит переоперделять респонс

Evgeny
06.03.2017
11:40:36
https://gist.github.com/EvgenyKungurov/cd303af5e7b8806e5b9e2abc98161396
в роутах только 2 create и post photos_path и photo_path

Alex
06.03.2017
11:41:16
RSpec.describe 'Photos', type: :request do
RSpec.describe PhotosController, type: :request do
и почему же в других работает? хммм, наверное дело в определении теста, т.к рспек к тесту относится не так как к другим, если я делаю тоже самое и оно не работает.
Я бы так подумал.

Антон
06.03.2017
11:43:11
-- доктор у меня болит нога
-- странно, у меня такая же, но все в порядке

Evgeny
06.03.2017
11:43:17
:D

Google

Victor
06.03.2017
11:43:24
Ещё раз – ключ должен быть 'message' :)

Evgeny
06.03.2017
11:43:29
Ну в тех тестах format указан для всего
тут же только json
В браузере все ок возвращается ajax

Антон
06.03.2017
11:43:48
is_expected.to be_success
expect(response.body).not_to be_blank
добавил?

Alex
06.03.2017
11:43:50
> expect(response.body).not_to be_blank
бесмысленная проверка если мы json парсим

Антон
06.03.2017
11:44:36
осмысленная, если мы дебажим

Alex
06.03.2017
11:44:46
если мы дебажим мы врубаем дебаггер и смотрим что не так
можно по шагам ходить если что.

Evgeny
06.03.2017
11:44:52
Виктор, если response у меня пустой, смысл не беспокоится за ключ, которых вообще 0) Но спс за указание)

Антон
06.03.2017
11:45:00

Alex
06.03.2017
11:45:06
var_dump не нужен
даешь binding.pry

Антон
06.03.2017
11:45:20
разорвало контекст к чертям собачим

Alex
06.03.2017
11:45:39
контекст? ты предлагаешь по сто раз гонять тест с разными проверками вместо того чтобы интерактивно самому проверить

Антон
06.03.2017
11:45:39

Alex
06.03.2017
11:45:45
прай быстрее.
Прай не требует думать над проверкой
ты сам проверяешь что хочешь.

Google

Alex
06.03.2017
11:46:03
не требует по 100 раз перезапускать тест чтобы проверить каждый кусок

Антон
06.03.2017
11:46:10

Victor
06.03.2017
11:46:12
А насчёт респонс nil – хз, я не поленился, сделал такой же абсолютно тест, у меня все норм отработало

Антон
06.03.2017
11:46:19

Alex
06.03.2017
11:46:41
и что? у меня например в проекте единичный запуск теста - 5 секунд

Victor
06.03.2017
11:46:47
Ребят, хорош флудить. Реально набрасываются, чуть что напишешь сюда

Alex
06.03.2017
11:46:48
мне лень столько ждать после каждой правки

Victor
06.03.2017
11:47:08
Нездоровая какая-то атмосфера

Alex
06.03.2017
11:47:09
весь код покажи

Admin
ERROR: S client not available

Alex
06.03.2017
11:47:23
https://gist.github.com/EvgenyKungurov/cd303af5e7b8806e5b9e2abc98161396
RSpec.describe 'Photos', type: :request do
RSpec.describe PhotosController, type: :request do

Vasiliy
06.03.2017
11:47:25
Виктор оформи гистом пусть Евгений твоё себе скопирует

Alex
06.03.2017
11:47:45

Антон
06.03.2017
11:48:05

Alex
06.03.2017
11:48:16
у тебя не будет response если ты тестируешь не controller *nigga meme*

Vasiliy
06.03.2017
11:48:24
ты решил своё проблему же, у чувака респонс нил же

Alex
06.03.2017
11:48:38
у чувака респонс нил потому что он rspec тест накидал на строку

Google

Evgeny
06.03.2017
11:48:43
context 'Anton sovet' do
it 'should be success' do
post photos_path
expect(response).to be_success
expect(response.body).not_to be_blank
end
end
так ?

Alex
06.03.2017
11:48:57
так ?
Ты заголовок обновил?

Evgeny
06.03.2017
11:49:44
Нет, я еще у себя не проверил

Антон
06.03.2017
11:49:48
+1 за заголовок

Victor
06.03.2017
11:49:53
Ну у меня кстати тоже отличие было в том, что я респонс не трогал, новую переменную завёл

Alex
06.03.2017
11:49:59
а стоп, я как раз таки неправ наверное
у него же интеграционный
там плевать что в заголовке.

Антон
06.03.2017
11:50:17

Alex
06.03.2017
11:50:34
он же по полному пути тестирует, а не экшен контроллера.
вполне очевидно что в консольке потыкать быстрее и проще.

Антон
06.03.2017
11:52:17
разногласия не мешают :)

Alex
06.03.2017
11:52:45
не согласен, но тему же мы не потеряли
зачем думать над условием и по 20 раз перезапускать тест, вместо того чтобы один раз в консольке проверить данные и прямо там же проверить пофикшенный код?

Evgeny
06.03.2017
11:53:08

Антон
06.03.2017
11:53:09

Evgeny
06.03.2017
11:53:17
context 'Anton sovet' do
it 'should be success' do
post photos_path
expect(response).to be_success
expect(response.body).not_to be_blank
end
end

Антон
06.03.2017
11:53:22
значит респонс не нил

Alex
06.03.2017
11:53:28

Антон
06.03.2017
11:53:47

Alex
06.03.2017
11:53:59
ты в чем тесты гоняешь?