Skip to content

Commit

Permalink
Merge commit 'eddd97936'
Browse files Browse the repository at this point in the history
Conflicts:
	spec/integration/rails/test/integration/webrat_test.rb
  • Loading branch information
Mike Gaffney committed Jun 16, 2009
2 parents 9a42304 + eddd979 commit 0c9944c
Show file tree
Hide file tree
Showing 4 changed files with 25 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/webrat/core/session.rb
Expand Up @@ -273,7 +273,7 @@ def response_location
end

def current_host
URI.parse(current_url).host || "www.example.com"
URI.parse(current_url).host || @custom_headers["Host"] || "www.example.com"
end

def response_location_host
Expand Down
4 changes: 4 additions & 0 deletions spec/integration/rails/app/controllers/webrat_controller.rb
Expand Up @@ -25,6 +25,10 @@ def external_redirect
redirect_to "http://google.com"
end

def host_redirect
redirect_to submit_url
end

def before_redirect_form
end

Expand Down
1 change: 1 addition & 0 deletions spec/integration/rails/config/routes.rb
Expand Up @@ -7,6 +7,7 @@
webrat.internal_redirect "/internal_redirect", :action => "internal_redirect"
webrat.external_redirect "/external_redirect", :action => "external_redirect"
webrat.infinite_redirect "/infinite_redirect", :action => "infinite_redirect"
webrat.host_redirect "/host_redirect", :action => "host_redirect"

webrat.before_redirect_form "/before_redirect_form", :action => "before_redirect_form"
webrat.redirect_to_show_params "/redirect_to_show_params", :action => "redirect_to_show_params"
Expand Down
19 changes: 19 additions & 0 deletions spec/integration/rails/test/integration/webrat_test.rb
Expand Up @@ -52,6 +52,25 @@ class WebratTest < ActionController::IntegrationTest
assert response.redirect?
end

test "should recognize the host header to follow redirects properly" do
header "Host", "foo.bar"
visit host_redirect_path
assert !response.redirect?
assert response.body.include?("OK")
end

test "should click link by text" do
visit internal_redirect_path
click_link "Test Link Text"
assert_contain("Webrat Form")
end

test "should click link by id" do
visit internal_redirect_path
click_link "link_id"
assert_contain("Webrat Form")
end

test "should be able to assert xpath" do
visit root_path
assert_have_xpath "//h1"
Expand Down

0 comments on commit 0c9944c

Please sign in to comment.