Разработать DoubleRenderError - PullRequest
Новая
       3

Разработать DoubleRenderError

1 голос
/ 20 июня 2010

Стек приложений: Rails3, CanCan, Devise, Shoulda

У меня есть несколько вложенных ресурсов, и я хочу протестировать их с Ifa и получаю следующую DoubleRenderError: 

Error:
test: If anonymous user tries to GET index of fav_blogs should respond with 401.    (BlogItemsControllerTest):
AbstractController::DoubleRenderError: Render and/or redirect were called multiple times  in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".

Тест будет проверять, может ли пользователь, не прошедший проверку подлинности, получить доступ к вложенному ресурсу (он не должен)

TestMethod 

context "If anonymous user tries to GET index of fav_blogs" do
  setup do
    get :index, :end_user_id => end_users(:end_user_one).id, :format => "xml"
  end
  should_respond_with 401
  should_not_assign_to :blog_items
end

ControllerMethod: 

def index

  if params[:end_user_id] # if it is nested
    authenticate_end_user!
    authorize! :read, @end_user

    @blog_items = @end_user.blog_items
  else
    @PAGE_SIZE = 10
    page = Integer(params[:page]) || 0

    offset = @PAGE_SIZE * page
    @blog_items = BlogItem.limit( offset + @PAGE_SIZE ).offset( offset ).order('updated_at DESC').all
  end

  respond_with(@blog_items)
end

Все тесты с аутентифицированным пользователем работают нормально - может кто-нибудь подскажет. Большое спасибо! Бен

1 Ответ

0 голосов
/ 22 июня 2010

хорошо, я сделал - проблема возникает из-за 

authenticate_end_user!
authorize! :read, @end_user

блок. Первая строка отображает ошибку 401 в случае отсутствия авторизации, но в любом случае вторая строка будет выполнена. Поэтому поместите аутентификацию в before_filter и авторизацию в другое действие или действие вашего контроллера. Ben

...