This project is archived and is in readonly mode.

#2477 ✓stale
justin

assert_redirected_to should surface the message in all fail cases

Reported by justin | April 10th, 2009 @ 05:35 PM | in 3.x

Based on the current source (Rails 2.3.2), when assert_redirected_to fails due to the response redirect not matching the expected redirect the user-supplied message (second parameter) is not shown. This message is only shown if the response is not a redirect.

Example functional test code:


test "ensure all actions are protected" do
  %w{ index new create edit update destroy }.each do |action|
    get action
    assert_redirected_to login_path, "Expected redirect to login page for action: #{ action }"
  end
end

Corresponding controller code:


def index
  redirect_to some_other_page_path
end@@@

Comments and changes to this ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>

People watching this ticket

Attachments

Pages