1

Здравствуйте, подскажите пожалуйста почему не срабатывает тест?

RSpec.describe QuestionsController, type: :controller do

  describe "Show action" do
    it "renders show template if an item is found" do
      question = Question.create
      get :show, {id: question.id}
      response.should render_template('show')
    end

    it "renders 404 page if an question is not found" do
      get :show, {id: 10}
      expect(response).to have_http_status(:error)
    end
  end
end

В контроллере QuestionsController, есть before_action: set_question там

def set_question
  @question = Question.find(params[:id])
  unless @question
    render status: 404
  end
end

При вводе в адресной строке id вопроса, который не существует происходит редирект на страницу 404. Но при прохождении 2 части теста выскакивает ошибка

  Failure/Error: @question = Question.find(params[:id])

     ActiveRecord::RecordNotFound:
       Couldn't find Question with 'id'=10

Подскажите пожалуйста что не так.

1

2 ответа 2

0

Вот эта часть метода

unless @question
  render status: 404
end

никогда не будет достигнута. Дело в том, что метод find(), если не найдена запись, генерирует исключение ActiveRecord::RecordNotFound, что приводит к возврату кода 500, вместо 404. Вам нужно его либо перехватить исключение и в обработчике отдать 404, либо сохранить текущую организацию экшена, но отказаться от find() в пользу

@question = Question.where(id: params[:id]).first
2
  • спасибо, толково 7 июл 2016 в 20:43
  • 1
    > что приводит к возврату кода 500, вместо 404 По-моему, это не так. При ошибке ActiveRecord::RecordNotFound рельсы отдадут именно 404-ую ошибку.
    – installero
    8 июл 2016 в 8:06
0

Вы ловите :error, а ваш контроллер отдает вам ошибку 404.

Как rspec воспринимает ключ :error можно посмотреть здесь.

Либо укажите статус явно (404), либо используйте ключ :missing

it "renders 404 page if an question is not found" do
  get :show, {id: 10}
  expect(response).to have_http_status(:missing)
end

А вот этот код, как написал cheops, действительно не нужен. Рельсы сами сделают это за Вас.

unless @question
  render status: 404
end

p.s. find на where имеет смысл заменять только в случае, если действия контроллера, в которых не нашелся вопрос, также имеют смысл. Обычно это не так.

1
  • пробовал и 404 и error и :missing, помогло только использование where вместо find. 8 июл 2016 в 11:10

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.