@@ -74,7 +74,7 @@ module Clearance def deny_access(flash_message = nil, opts = {}) store_location flash[:failure] = flash_message if flash_message - render :template => "/sessions/new", :status => :unauthorized + redirect_to new_session_url end end lib/clearance/app/controllers/application_controller.rb @@ -27,7 +27,7 @@ module Clearance end should_deny_access(:flash => /User has not confirmed email. Confirmation email will be resent./i) - should_respond_with :unauthorized + should_redirect_to('new_session_url') { new_session_url } should "send the confirmation email" do assert_not_nil email = ActionMailer::Base.deliveries[0] lib/clearance/test/functional/sessions_controller_test.rb @@ -45,12 +45,7 @@ module Clearance should_not_set_the_flash end - should "respond with 401 Unauthorized and render sign_in template" do - assert_response :unauthorized, - "access was expected to be denied (401 unauthorized)" - assert_template "sessions/new", - "template was expected to be sign in (sessions/new)" - end + should_redirect_to('new_session_url') { new_session_url } end # HTTP FLUENCY shoulda_macros/clearance.rb fa424b8fe9fd8f151f32a726982a3fd42940b328 Matt Jankowski mjankowski@thoughtbot.com http://github.com/thoughtbot/clearance/commit/410b802a47e3bd77d06f9a1a383656ab956fea9e 410b802a47e3bd77d06f9a1a383656ab956fea9e 2009-03-20T10:40:15-07:00 2009-03-20T10:40:15-07:00 we should redirect on failed login, not render new e7c2fc22fb4874821b77d9256fe0e4899821600a Matt Jankowski mjankowski@thoughtbot.com