diff --git a/.gitignore b/.gitignore index 8c7320b..dfafece 100644 --- a/.gitignore +++ b/.gitignore @@ -12,4 +12,4 @@ log results test_apps *.tmproj -webrat.log +*.log \ No newline at end of file diff --git a/History.txt b/History.txt index e1fe167..2a47287 100644 --- a/History.txt +++ b/History.txt @@ -1,4 +1,4 @@ -== brynary/master (in git) +== 0.4.5 / ? * Major enhancements @@ -9,16 +9,15 @@ * Added support for field_labeled_locators ending in non word characters lh 148 (Zach Dennis) * Filled in tests on click link lh 195 (diabolo) - * Canonicalize all URLs (Shalon Wood) == 0.4.4 / 2009-04-06 * Major enhancements * Make selenium process management code more robust and informative - + * Minor enhancements - + * Add support for Rails javascript post links (Mark Menard) * Upgrade selenium-client dependency to 1.2.14, and update for new waiting API (Balint Erdi) @@ -29,7 +28,7 @@ * Don't create a new instance of WWW::Mechanize for each request (Mark Menard) * Select fields with duplicate selected options sent an incorrect value (Noah Davis) - + == 0.4.3 / 2009-03-17 * Minor enhancements @@ -39,7 +38,7 @@ * Initial Merb and Sinatra compatibility for Selenium mode (Corey Donohoe) * When faced with a label with no for attribute, that contains a hidden field and another field, as can be the case in Rails 2.3's checkbox view, - webrat now locates the non-hidden field. (Luke Melia) + webrat now locates the non-hidden field. (Luke Melia) * Add application_framework config for Selenium mode to determine how to start and stop the app server (Corey Donohoe) @@ -55,7 +54,7 @@ attributes in a hash and :count and :content options. See have_selector_spec.rb for more. * Add the same functionality mentioned above to have_xpath - + * Minor enhancements * Squeeze extra whitespace out of failures messages from contain @@ -82,7 +81,7 @@ redirected to (Adam Greene) * Recognize input tags with type button (Lena Herrmann) * Add uncheck method to Selenium mode (Lee Bankewitz) - + * Bug fixes * Make requests to a Rails app using a full URL instead of a relative path. This change @@ -93,7 +92,7 @@ to behave correctly (Noah Davis) * Switch to using selenium.click instead of .check when checking a checkbox (Noah Davis) - * Create tmp/pids directory if directory does not exist. (Amos King and Mike Gaffney) + * Create tmp/pids directory if directory does not exist. (Amos King and Mike Gaffney) * Setup deprecated writers for the selenium_environment= and selenium_port= config * Ensure the previous pages params aren't passed through redirect (Daniel Lucraft and Bryan Helmkamp) @@ -169,7 +168,7 @@ * Added save_and_open_screengrab for Selenium mode (Luke Melia) * Bug fixes - + * field_labeled should disregard labels without matching fields (Kyle Hargraves) * Fixed bug where Scope was creating a new DOM rather than re-using the existing DOM. [#105] (Zach Dennis) diff --git a/lib/webrat/core/elements/area.rb b/lib/webrat/core/elements/area.rb index 03bea18..e2f08bb 100644 --- a/lib/webrat/core/elements/area.rb +++ b/lib/webrat/core/elements/area.rb @@ -8,7 +8,7 @@ module Webrat end def click(method = nil, options = {}) - @session.request_page(href, :get, {}) + @session.request_page(absolute_href, :get, {}) end protected @@ -16,5 +16,16 @@ module Webrat def href Webrat::XML.attribute(@element, "href") end + + def absolute_href + if href =~ /^\?/ + "#{@session.current_url}#{href}" + elsif href !~ %r{^https?://[\w|.]+(/.*)} && (href !~ /^\//) + "#{@session.current_url}/#{href}" + else + href + end + end + end end diff --git a/lib/webrat/core/elements/link.rb b/lib/webrat/core/elements/link.rb index 607655b..5b83e36 100644 --- a/lib/webrat/core/elements/link.rb +++ b/lib/webrat/core/elements/link.rb @@ -16,9 +16,9 @@ module Webrat options[:javascript] = true if options[:javascript].nil? if options[:javascript] - @session.request_page(href, method, data) + @session.request_page(absolute_href, method, data) else - @session.request_page(href, :get, {}) + @session.request_page(absolute_href, :get, {}) end end @@ -40,6 +40,16 @@ module Webrat Webrat::XML.attribute(@element, "href") end + def absolute_href + if href =~ /^\?/ + "#{@session.current_url}#{href}" + elsif href !~ %r{^https?://} && (href !~ /^\//) + "#{@session.current_url}/#{href}" + else + href + end + end + def authenticity_token return unless onclick && onclick.include?("s.setAttribute('name', 'authenticity_token');") && onclick =~ /s\.setAttribute\('value', '([a-f0-9]{40})'\);/ diff --git a/lib/webrat/core/session.rb b/lib/webrat/core/session.rb index e4bbaeb..13f3f90 100644 --- a/lib/webrat/core/session.rb +++ b/lib/webrat/core/session.rb @@ -97,32 +97,10 @@ For example: @default_headers.dup.merge(@custom_headers.dup) end - def canonicalize_url(href_url) - @current_url ||= "http://www.example.com/" # @current_url can't be blank, or things break - # Case one: relative url - if href_url !~ %r{^https?://} && (href_url !~ /^\//) - # If the relative url starts with # or ?, we need append it as is _if_ there are three slashes in the current url; - # otherwise, ensure there's a slash between it and the current - # url - if (href_url =~ /^\?/ or href_url =~ /^#/) && current_url.scan('/').length > 2 - "#{current_url}#{href_url}" - else - "#{current_url.chomp('/')}/#{href_url}" - end - # Case two: absolute url without host - elsif href_url =~ /^\// - "http://#{current_host}#{href_url}" - # Case three: absolute url with scheme and host. - else - href_url - end - end - def request_page(url, http_method, data) #:nodoc: h = headers h['HTTP_REFERER'] = @current_url if @current_url - url = canonicalize_url(url) debug_log "REQUESTING PAGE: #{http_method.to_s.upcase} #{url} with #{data.inspect} and HTTP headers #{h.inspect}" process_request(http_method, url, data, h) @@ -289,7 +267,7 @@ For example: end def response_location - canonicalize_url(response.headers["Location"]) + response.headers["Location"] end def current_host diff --git a/spec/integration/rails/Rakefile b/spec/integration/rails/Rakefile index 9f1361e..e09ebc6 100644 --- a/spec/integration/rails/Rakefile +++ b/spec/integration/rails/Rakefile @@ -16,13 +16,13 @@ namespace :test_unit do desc "runs the test::unit based tests in webrat mode" task :rails do ENV['WEBRAT_INTEGRATION_MODE'] = 'rails' - Rake::Task['test:integration'].execute nil + Rake::Task['test:integration'].execute end desc "runs the test::unit based tests in selenium mode" task :selenium do ENV['WEBRAT_INTEGRATION_MODE'] = 'selenium' - Rake::Task['test:integration'].execute nil + Rake::Task['test:integration'].execute end desc "run both selenium and rails mode Test::Unit suites" diff --git a/spec/private/core/session_spec.rb b/spec/private/core/session_spec.rb index cdda4f9..7f20af8 100644 --- a/spec/private/core/session_spec.rb +++ b/spec/private/core/session_spec.rb @@ -118,7 +118,7 @@ describe Webrat::Session do webrat_session.request_page("/oldurl", :get, {}) - webrat_session.current_url.should == "http://www.example.com/oldurl" + webrat_session.current_url.should == "/oldurl" end end @@ -195,73 +195,4 @@ describe Webrat::Session do end - describe "#canonicalize_url"do - - it "should turn 'http://www.example.com/' into 'http://www.example.com/'" do - webrat_session.stub!(:current_url => "http://www.example.com") - webrat_session.canonicalize_url("http://www.example.com/").should == "http://www.example.com/" - end - - it "should turn '/login' into 'http://www.example.com/login' if current_url is 'http://www.example.com/'" do - webrat_session.stub!(:current_url => "http://www.example.com/") - webrat_session.canonicalize_url("/login").should == "http://www.example.com/login" - end - - it "should turn '/login' into 'http://www.example.com/login' if current_url is 'http://www.example.com'" do - webrat_session.stub!(:current_url => "http://www.example.com") - webrat_session.canonicalize_url("/login").should == "http://www.example.com/login" - end - - it "should turn 'login' into 'http://www.example.com/login' if current_url is 'http://www.example.com'" do - webrat_session.stub!(:current_url => "http://www.example.com") - webrat_session.canonicalize_url("login").should == "http://www.example.com/login" - end - - it "should turn 'login' into 'http://www.example.com/login' if current_url is 'http://www.example.com/'" do - webrat_session.stub!(:current_url => "http://www.example.com/") - webrat_session.canonicalize_url("login").should == "http://www.example.com/login" - end - - it "should turn '?login' into 'http://www.example.com/?login' if current_url is 'http://www.example.com'" do - webrat_session.stub!(:current_url => "http://www.example.com") - webrat_session.canonicalize_url("?login").should == "http://www.example.com/?login" - end - - it "should turn '?login' into 'http://www.example.com/?login' if current_url is 'http://www.example.com/'" do - webrat_session.stub!(:current_url => "http://www.example.com/") - webrat_session.canonicalize_url("?login").should == "http://www.example.com/?login" - end - - it "should turn '?login' into 'http://www.example.com/page?login' if current_url is 'http://www.example.com/page'" do - webrat_session.stub!(:current_url => "http://www.example.com/page") - webrat_session.canonicalize_url("?login").should == "http://www.example.com/page?login" - end - - it "should turn '?login' into 'http://www.example.com/page/?login' if current_url is 'http://www.example.com/page/'" do - webrat_session.stub!(:current_url => "http://www.example.com/page/") - webrat_session.canonicalize_url("?login").should == "http://www.example.com/page/?login" - end - - it "should turn '#anchor' into 'http://www.example.com/#anchor' if current_url is 'http://www.example.com'" do - webrat_session.stub!(:current_url => "http://www.example.com") - webrat_session.canonicalize_url("#anchor").should == "http://www.example.com/#anchor" - end - - it "should turn '#anchor' into 'http://www.example.com/#anchor' if current_url is 'http://www.example.com/'" do - webrat_session.stub!(:current_url => "http://www.example.com/") - webrat_session.canonicalize_url("#anchor").should == "http://www.example.com/#anchor" - end - - it "should turn '#anchor' into 'http://www.example.com/page#anchor' if current_url is 'http://www.example.com/page'" do - webrat_session.stub!(:current_url => "http://www.example.com/page") - webrat_session.canonicalize_url("#anchor").should == "http://www.example.com/page#anchor" - end - - it "should turn '#anchor' into 'http://www.example.com/page/#anchor' if current_url is 'http://www.example.com/page/'" do - webrat_session.stub!(:current_url => "http://www.example.com/page/") - webrat_session.canonicalize_url("#anchor").should == "http://www.example.com/page/#anchor" - end - - end - end diff --git a/spec/private/rails/attaches_file_spec.rb b/spec/private/rails/attaches_file_spec.rb index cd3a863..e044db0 100644 --- a/spec/private/rails/attaches_file_spec.rb +++ b/spec/private/rails/attaches_file_spec.rb @@ -27,7 +27,7 @@ describe "attach_file" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/widgets", { "widget" => { "file" => "" } }) + webrat_session.should_receive(:post).with("/widgets", { "widget" => { "file" => "" } }) click_button end @@ -41,7 +41,7 @@ describe "attach_file" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/widgets", { "widget" => { "file" => @uploaded_file } }) + webrat_session.should_receive(:post).with("/widgets", { "widget" => { "file" => @uploaded_file } }) attach_file "Document", @filename click_button end @@ -58,7 +58,7 @@ describe "attach_file" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/widgets", { "widget" => { "files" => [@uploaded_file, @uploaded_file] } }) + webrat_session.should_receive(:post).with("/widgets", { "widget" => { "files" => [@uploaded_file, @uploaded_file] } }) attach_file "Document", @filename attach_file "Spreadsheet", @filename click_button diff --git a/spec/private/rails/rails_session_spec.rb b/spec/private/rails/rails_session_spec.rb index 81eeed6..978e892 100644 --- a/spec/private/rails/rails_session_spec.rb +++ b/spec/private/rails/rails_session_spec.rb @@ -96,8 +96,7 @@ describe Webrat::RailsSession do response = mock("response", :body => body, :headers => {}, :code => 200) @integration_session.stub!(:response => response) - @integration_session.stub!(:https!) - @integration_session.should_receive(:get).with("http://www.example.com/page2", {}, nil) + @integration_session.should_receive(:get).with("/page2", {}, nil) rails_session = Webrat::RailsSession.new(@integration_session) diff --git a/spec/public/basic_auth_spec.rb b/spec/public/basic_auth_spec.rb index caec284..1d6b590 100644 --- a/spec/public/basic_auth_spec.rb +++ b/spec/public/basic_auth_spec.rb @@ -6,7 +6,7 @@ describe "Basic Auth HTTP headers" do end it "should be present in visit" do - webrat_session.should_receive(:get).with("http://www.example.com/", {}, {'HTTP_AUTHORIZATION' => "Basic dXNlcjpzZWNyZXQ=\n"}) + webrat_session.should_receive(:get).with("/", {}, {'HTTP_AUTHORIZATION' => "Basic dXNlcjpzZWNyZXQ=\n"}) visit("/") end @@ -18,7 +18,7 @@ describe "Basic Auth HTTP headers" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/form1", {}, {'HTTP_AUTHORIZATION' => "Basic dXNlcjpzZWNyZXQ=\n"}) + webrat_session.should_receive(:post).with("/form1", {}, {'HTTP_AUTHORIZATION' => "Basic dXNlcjpzZWNyZXQ=\n"}) click_button end end diff --git a/spec/public/check_spec.rb b/spec/public/check_spec.rb index ef2e56f..9376f88 100644 --- a/spec/public/check_spec.rb +++ b/spec/public/check_spec.rb @@ -36,7 +36,7 @@ describe "check" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "1"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "1"}) check "TOS" click_button end @@ -51,7 +51,7 @@ describe "check" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "remember_me" => "on") + webrat_session.should_receive(:post).with("/login", "remember_me" => "on") check "remember_me" click_button end @@ -79,7 +79,7 @@ describe "check" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "remember_me" => "yes") + webrat_session.should_receive(:post).with("/login", "remember_me" => "yes") check "remember_me" click_button end @@ -132,7 +132,7 @@ describe "uncheck" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "0"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "0"}) check "TOS" uncheck "TOS" click_button @@ -147,7 +147,7 @@ describe "uncheck" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", {}) + webrat_session.should_receive(:post).with("/login", {}) uncheck "remember_me" click_button end @@ -164,7 +164,7 @@ describe "uncheck" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", {"options" => ["1", "2"]}) + webrat_session.should_receive(:post).with("/login", {"options" => ["1", "2"]}) check 'Option 1' check 'Option 2' click_button @@ -183,7 +183,7 @@ describe "uncheck" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "0"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "0"}) uncheck "TOS" click_button end diff --git a/spec/public/choose_spec.rb b/spec/public/choose_spec.rb index 97fd97e..5a07c17 100644 --- a/spec/public/choose_spec.rb +++ b/spec/public/choose_spec.rb @@ -36,7 +36,7 @@ describe "choose" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"gender" => "M"}) + webrat_session.should_receive(:get).with("/login", "user" => {"gender" => "M"}) choose "Male" click_button end @@ -53,7 +53,7 @@ describe "choose" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"gender" => "M"}) + webrat_session.should_receive(:get).with("/login", "user" => {"gender" => "M"}) choose "Female" choose "Male" click_button @@ -81,7 +81,7 @@ describe "choose" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "first_option" => "on") + webrat_session.should_receive(:post).with("/login", "first_option" => "on") choose "first_option" click_button end @@ -95,7 +95,7 @@ describe "choose" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "first_option" => "on") + webrat_session.should_receive(:post).with("/login", "first_option" => "on") click_button end @@ -111,7 +111,7 @@ describe "choose" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"gender" => "M"}) + webrat_session.should_receive(:post).with("/login", "user" => {"gender" => "M"}) choose "Male" click_button end diff --git a/spec/public/click_area_spec.rb b/spec/public/click_area_spec.rb index cb7c7df..e039323 100644 --- a/spec/public/click_area_spec.rb +++ b/spec/public/click_area_spec.rb @@ -9,7 +9,7 @@ describe "click_area" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_area "Berlin" end @@ -62,13 +62,13 @@ describe "click_area" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_area "berlin" end it "should follow relative links" do - webrat_session.stub!(:current_url => "http://www.example.com/page") + webrat_session.stub!(:current_url => "/page") with_html <<-HTML @@ -76,7 +76,7 @@ describe "click_area" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page/sub", {}) + webrat_session.should_receive(:get).with("/page/sub", {}) click_area "Berlin" end @@ -100,7 +100,7 @@ describe "click_area" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page?foo=bar", {}) + webrat_session.should_receive(:get).with("/page?foo=bar", {}) click_area "Berlin" end end diff --git a/spec/public/click_button_spec.rb b/spec/public/click_button_spec.rb index 45a0d30..4bea34e 100644 --- a/spec/public/click_button_spec.rb +++ b/spec/public/click_button_spec.rb @@ -86,7 +86,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form1", {}) + webrat_session.should_receive(:get).with("/form1", {}) click_button end @@ -113,7 +113,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", {}) + webrat_session.should_receive(:get).with("/form2", {}) click_button "Form2" end @@ -125,7 +125,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", {}) + webrat_session.should_receive(:get).with("/login", {}) click_button end @@ -150,7 +150,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "login" => "Login") + webrat_session.should_receive(:post).with("/login", "login" => "Login") click_button("Login") end @@ -163,7 +163,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", {}) + webrat_session.should_receive(:post).with("/login", {}) click_button("Login") end @@ -176,7 +176,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"password" => "mypass"}) + webrat_session.should_receive(:get).with("/login", "user" => {"password" => "mypass"}) click_button end @@ -189,7 +189,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"email" => "test@example.com"}) + webrat_session.should_receive(:get).with("/login", "user" => {"email" => "test@example.com"}) click_button end @@ -202,7 +202,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"email" => "test@example.com"}) + webrat_session.should_receive(:get).with("/login", "user" => {"email" => "test@example.com"}) click_button end @@ -219,7 +219,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", {}) + webrat_session.should_receive(:get).with("/login", {}) click_button end @@ -232,7 +232,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "1"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "1"}) click_button end @@ -248,7 +248,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"gender" => "F"}) + webrat_session.should_receive(:get).with("/login", "user" => {"gender" => "F"}) click_button end @@ -262,7 +262,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "0"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "0"}) click_button end @@ -276,7 +276,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"tos" => "1"}) + webrat_session.should_receive(:get).with("/login", "user" => {"tos" => "1"}) click_button end @@ -298,7 +298,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", + webrat_session.should_receive(:post).with("/login", "options" => ["burger", "fries", "soda", "soda", "dessert"], "response" => { "choices" => [{"selected" => "one"}, {"selected" => "two"}, {"selected" => "two"}]}) click_button @@ -313,7 +313,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", {}) + webrat_session.should_receive(:get).with("/login", {}) click_button end @@ -326,7 +326,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/posts", "post" => {"body" => "Post body here!"}) + webrat_session.should_receive(:post).with("/posts", "post" => {"body" => "Post body here!"}) click_button end @@ -340,7 +340,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/posts", "post" => {"body" => "Peanut butter & jelly"}) + webrat_session.should_receive(:post).with("/posts", "post" => {"body" => "Peanut butter & jelly"}) click_button end end @@ -357,7 +357,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "month" => "2") + webrat_session.should_receive(:get).with("/login", "month" => "2") click_button end @@ -373,7 +373,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "month" => "February") + webrat_session.should_receive(:get).with("/login", "month" => "February") click_button end @@ -389,7 +389,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "month" => "1") + webrat_session.should_receive(:get).with("/login", "month" => "1") click_button end @@ -403,7 +403,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "contestant" => {"scores" => {'1' => '2', '3' => '4'}}) + webrat_session.should_receive(:post).with("/login", "contestant" => {"scores" => {'1' => '2', '3' => '4'}}) click_button end @@ -416,7 +416,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"email" => ""}) + webrat_session.should_receive(:get).with("/login", "user" => {"email" => ""}) click_button end @@ -429,7 +429,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"email" => ""}) + webrat_session.should_receive(:get).with("/login", "user" => {"email" => ""}) click_button end @@ -490,7 +490,7 @@ describe "click_button" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/login", "user" => {"email" => ""}) + webrat_session.should_receive(:get).with("/login", "user" => {"email" => ""}) click_button "Login" end end diff --git a/spec/public/click_link_spec.rb b/spec/public/click_link_spec.rb index 82fcdfe..549c70b 100644 --- a/spec/public/click_link_spec.rb +++ b/spec/public/click_link_spec.rb @@ -7,7 +7,7 @@ describe "click_link" do Save & go back HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "Save & go back" end @@ -17,7 +17,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "Link text" end @@ -27,7 +27,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "Link text", :method => :get end @@ -37,7 +37,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "ink tex", :method => :get end @@ -47,7 +47,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:delete).with("http://www.example.com/page", {}) + webrat_session.should_receive(:delete).with("/page", {}) click_link "Link text", :method => :delete end @@ -58,7 +58,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:post).with("http://www.example.com/page", {}) + webrat_session.should_receive(:post).with("/page", {}) click_link "Link text", :method => :post end @@ -68,7 +68,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:put).with("http://www.example.com/page", {}) + webrat_session.should_receive(:put).with("/page", {}) click_link "Link text", :method => :put end @@ -78,7 +78,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link /link [a-z]/i end @@ -88,7 +88,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "link_text_link" end @@ -98,7 +98,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link /_text_/ end @@ -108,7 +108,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link 'piddle' end @@ -118,7 +118,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link /iddle/ end @@ -140,7 +140,7 @@ describe "click_link" do return false;">Posts HTML - webrat_session.should_receive(:post).with("http://www.example.com/posts", "authenticity_token" => "aa79cb354597a60a3786e7e291ed4f74d77d3a62") + webrat_session.should_receive(:post).with("/posts", "authenticity_token" => "aa79cb354597a60a3786e7e291ed4f74d77d3a62") click_link "Posts" end @@ -161,7 +161,7 @@ describe "click_link" do return false;">Delete HTML - webrat_session.should_receive(:delete).with("http://www.example.com/posts/1", {}) + webrat_session.should_receive(:delete).with("/posts/1", {}) click_link "Delete" end @@ -177,7 +177,7 @@ describe "click_link" do return false;">Posts HTML - webrat_session.should_receive(:post).with("http://www.example.com/posts", {}) + webrat_session.should_receive(:post).with("/posts", {}) click_link "Posts" end @@ -193,7 +193,7 @@ describe "click_link" do return false;">Posts HTML - webrat_session.should_receive(:get).with("http://www.example.com/posts", {}) + webrat_session.should_receive(:get).with("/posts", {}) click_link "Posts", :javascript => false end @@ -214,7 +214,7 @@ describe "click_link" do return false;">Post HTML - webrat_session.should_receive(:post).with("http://www.example.com/posts", {}) + webrat_session.should_receive(:post).with("/posts", {}) click_link "Post" end @@ -235,7 +235,7 @@ describe "click_link" do return false;">Put HTML - webrat_session.should_receive(:put).with("http://www.example.com/posts", {}) + webrat_session.should_receive(:put).with("/posts", {}) click_link "Put" end @@ -302,7 +302,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "LINK TEXT" end @@ -312,7 +312,7 @@ describe "click_link" do This is some cool link text, isn't it? HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "Link text" end @@ -322,7 +322,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page", {}) + webrat_session.should_receive(:get).with("/page", {}) click_link "Link text" end @@ -333,7 +333,7 @@ describe "click_link" do Link text HTML - webrat_session.should_receive(:get).with("http://www.example.com/page1", {}) + webrat_session.should_receive(:get).with("/page1", {}) click_link "Link text" end @@ -345,7 +345,7 @@ describe "click_link" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page2", {}) + webrat_session.should_receive(:get).with("/page2", {}) click_link "Link" end @@ -356,7 +356,7 @@ describe "click_link" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page1", {}) + webrat_session.should_receive(:get).with("/page1", {}) click_link "This is a link" end @@ -369,7 +369,7 @@ describe "click_link" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page2", {}) + webrat_session.should_receive(:get).with("/page2", {}) click_link "Location" end end @@ -384,7 +384,7 @@ describe "click_link" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page2", {}) + webrat_session.should_receive(:get).with("/page2", {}) click_link_within "#container", "Link" end @@ -400,18 +400,18 @@ describe "click_link" do end it "should follow relative links" do - webrat_session.stub!(:current_url => "http://www.example.com/page") + webrat_session.stub!(:current_url => "/page") with_html <<-HTML Jump to sub page HTML - webrat_session.should_receive(:get).with("http://www.example.com/page/sub", {}) + webrat_session.should_receive(:get).with("/page/sub", {}) click_link "Jump to sub page" end it "should follow fully qualified local links" do - webrat_session.stub!(:current_url => "http://www.example.com/page") + webrat_session.stub!(:current_url => "/page") with_html <<-HTML Jump to sub page @@ -432,13 +432,13 @@ describe "click_link" do end it "should follow query parameters" do - webrat_session.stub!(:current_url => "http://www.example.com/page") + webrat_session.stub!(:current_url => "/page") with_html <<-HTML Jump to foo bar HTML - webrat_session.should_receive(:get).with("http://www.example.com/page?foo=bar", {}) + webrat_session.should_receive(:get).with("/page?foo=bar", {}) click_link "Jump to foo bar" end diff --git a/spec/public/fill_in_spec.rb b/spec/public/fill_in_spec.rb index 83f8de1..f6b8f1e 100644 --- a/spec/public/fill_in_spec.rb +++ b/spec/public/fill_in_spec.rb @@ -11,7 +11,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"text" => "filling text area"}) + webrat_session.should_receive(:post).with("/login", "user" => {"text" => "filling text area"}) fill_in "User Text", :with => "filling text area" click_button end @@ -25,7 +25,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"text" => "pass"}) + webrat_session.should_receive(:post).with("/login", "user" => {"text" => "pass"}) fill_in "user_text", :with => "pass" click_button end @@ -65,7 +65,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"email" => "foo@example.com"}) + webrat_session.should_receive(:post).with("/login", "user" => {"email" => "foo@example.com"}) fill_in "user[email]", :with => "foo@example.com" click_button end @@ -83,7 +83,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"mail1" => "", "mail2" => "value"}) + webrat_session.should_receive(:post).with("/login", "user" => {"mail1" => "", "mail2" => "value"}) fill_in "Some", :with => "value" click_button end @@ -101,7 +101,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"mail1" => "value", "mail2" => ""}) + webrat_session.should_receive(:post).with("/login", "user" => {"mail1" => "value", "mail2" => ""}) fill_in "Some mail", :with => "value" click_button end @@ -144,7 +144,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"email" => "foo@example.com"}) + webrat_session.should_receive(:post).with("/login", "user" => {"email" => "foo@example.com"}) fill_in "Email", :with => "foo@example.com" click_button end @@ -158,7 +158,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"email" => "foo@example.com"}) + webrat_session.should_receive(:post).with("/login", "user" => {"email" => "foo@example.com"}) fill_in "user[email]", :with => "foo@example.com" click_button end @@ -172,7 +172,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"email" => "foo@example.com"}) + webrat_session.should_receive(:post).with("/login", "user" => {"email" => "foo@example.com"}) fill_in "user[email]", :with => "foo@example.com" click_button end @@ -187,7 +187,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "user" => {"email" => "foo@example.com"}) + webrat_session.should_receive(:post).with("/login", "user" => {"email" => "foo@example.com"}) fill_in :email, :with => "foo@example.com" click_button end @@ -202,7 +202,7 @@ describe "fill_in" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/users", "user" => {"phone" => "+1 22 33"}) + webrat_session.should_receive(:post).with("/users", "user" => {"phone" => "+1 22 33"}) fill_in 'Phone', :with => "+1 22 33" click_button end diff --git a/spec/public/reload_spec.rb b/spec/public/reload_spec.rb index 5c9e344..cce2bb5 100644 --- a/spec/public/reload_spec.rb +++ b/spec/public/reload_spec.rb @@ -2,9 +2,9 @@ require File.expand_path(File.dirname(__FILE__) + "/../spec_helper") describe "reloads" do it "should reload the page with http referer" do - webrat_session.should_receive(:get).with("http://www.example.com/", {}) + webrat_session.should_receive(:get).with("/", {}) + webrat_session.should_receive(:get).with("/", {}, {"HTTP_REFERER"=>"/"}) visit("/") - webrat_session.should_receive(:get).with("http://www.example.com/", {}, {"HTTP_REFERER"=>"http://www.example.com/"}) reload end end diff --git a/spec/public/select_date_spec.rb b/spec/public/select_date_spec.rb index a5923a5..7611ada 100644 --- a/spec/public/select_date_spec.rb +++ b/spec/public/select_date_spec.rb @@ -19,7 +19,7 @@ describe "select_date" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"date(1i)" => '2003', "date(2i)" => "12", "date(3i)" => "25"}) select_date "December 25, 2003", :from => "Date" click_button @@ -43,7 +43,7 @@ describe "select_date" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"date(1i)" => '2003', "date(2i)" => "12", "date(3i)" => "25"}) select_date Date.parse("December 25, 2003"), :from => "date" click_button @@ -66,7 +66,7 @@ describe "select_date" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"date(1i)" => '2003', "date(2i)" => "12", "date(3i)" => "25"}) select_date "December 25, 2003" click_button @@ -90,7 +90,7 @@ describe "select_date" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"date(1i)" => '2003', "date(2i)" => "12", "date(3i)" => "25"}) select_date Date.parse("December 25, 2003"), :from => "date ?" click_button diff --git a/spec/public/select_datetime_spec.rb b/spec/public/select_datetime_spec.rb index 575ad37..0297177 100644 --- a/spec/public/select_datetime_spec.rb +++ b/spec/public/select_datetime_spec.rb @@ -25,7 +25,7 @@ describe "select_datetime" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(1i)" => '2003', "time(2i)" => "12", "time(3i)" => "25", "time(4i)" => "09", "time(5i)" => "30"}) select_datetime "December 25, 2003 9:30", :from => "Time" click_button @@ -55,7 +55,7 @@ describe "select_datetime" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(1i)" => '2003', "time(2i)" => "12", "time(3i)" => "25", "time(4i)" => "09", "time(5i)" => "30"}) select_datetime Time.parse("December 25, 2003 9:30"), :from => "Time" click_button @@ -85,7 +85,7 @@ describe "select_datetime" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(1i)" => '2003', "time(2i)" => "12", "time(3i)" => "25", "time(4i)" => "09", "time(5i)" => "30"}) select_datetime "December 25, 2003 9:30", :from => "Time ?" click_button @@ -115,7 +115,7 @@ describe "select_datetime" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(1i)" => '2003', "time(2i)" => "12", "time(3i)" => "25", "time(4i)" => "09", "time(5i)" => "30"}) select_datetime "December 25, 2003 9:30" click_button diff --git a/spec/public/select_spec.rb b/spec/public/select_spec.rb index 34b6c31..98dfca0 100644 --- a/spec/public/select_spec.rb +++ b/spec/public/select_spec.rb @@ -62,7 +62,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "month" => "1") + webrat_session.should_receive(:post).with("/login", "month" => "1") select "January", :from => "month" click_button end @@ -76,7 +76,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "encoded" => "A & B") + webrat_session.should_receive(:post).with("/login", "encoded" => "A & B") select "Encoded", :from => "encoded" click_button end @@ -90,7 +90,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", 'month' => '') + webrat_session.should_receive(:post).with("/login", 'month' => '') click_button end @@ -103,7 +103,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "month" => "1") + webrat_session.should_receive(:post).with("/login", "month" => "1") select "January" click_button end @@ -118,7 +118,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "start_month" => "s1", "end_month" => "e1") + webrat_session.should_receive(:post).with("/login", "start_month" => "s1", "end_month" => "e1") select "January", :from => "end_month" click_button end @@ -135,7 +135,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "start_month" => "s1", "end_month" => "e1") + webrat_session.should_receive(:post).with("/login", "start_month" => "s1", "end_month" => "e1") select "January", :from => "End Month" click_button end @@ -149,7 +149,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "month" => "January") + webrat_session.should_receive(:post).with("/login", "month" => "January") select "January", :from => "month" click_button end @@ -163,7 +163,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "month" => "January") + webrat_session.should_receive(:post).with("/login", "month" => "January") select /jan/i click_button end @@ -195,7 +195,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "start_month" => "s1", "end_month" => "e1") + webrat_session.should_receive(:post).with("/login", "start_month" => "s1", "end_month" => "e1") select /jan/i, :from => "End Month" click_button end @@ -210,7 +210,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "month" => "Peanut butter & jelly") + webrat_session.should_receive(:post).with("/login", "month" => "Peanut butter & jelly") click_button end end @@ -242,7 +242,7 @@ describe "select" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/login", "clothes" => "pants") + webrat_session.should_receive(:post).with("/login", "clothes" => "pants") click_button end diff --git a/spec/public/select_time_spec.rb b/spec/public/select_time_spec.rb index fcc8678..3eafbaa 100644 --- a/spec/public/select_time_spec.rb +++ b/spec/public/select_time_spec.rb @@ -16,7 +16,7 @@ describe "select_time" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(4i)" => "09", "time(5i)" => "30"}) select_time "9:30AM", :from => "Time" click_button @@ -37,7 +37,7 @@ describe "select_time" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(4i)" => "09", "time(5i)" => "30"}) select_time Time.parse("9:30AM"), :from => "Time" click_button @@ -58,7 +58,7 @@ describe "select_time" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(4i)" => "09", "time(5i)" => "30"}) select_time "9:30AM", :from => "Time #" click_button @@ -78,7 +78,7 @@ describe "select_time" do HTML - webrat_session.should_receive(:post).with("http://www.example.com/appointments", + webrat_session.should_receive(:post).with("/appointments", "appointment" => {"time(4i)" => "09", "time(5i)" => "30"}) select_time "9:30" click_button diff --git a/spec/public/visit_spec.rb b/spec/public/visit_spec.rb index 37042f6..dc73fc6 100644 --- a/spec/public/visit_spec.rb +++ b/spec/public/visit_spec.rb @@ -10,7 +10,7 @@ describe "visit" do end it "should use get" do - webrat_session.should_receive(:get).with("http://www.example.com/", {}) + webrat_session.should_receive(:get).with("/", {}) visit("/") end @@ -36,7 +36,7 @@ describe "visit" do visit("/oldurl") - current_url.should == "http://www.example.com/oldurl" + current_url.should == "/oldurl" end end @@ -51,7 +51,7 @@ describe "visit with referer" do end it "should use get with referer header" do - webrat_session.should_receive(:get).with("http://www.example.com/", {}, {"HTTP_REFERER" => "/old_url"}) + webrat_session.should_receive(:get).with("/", {}, {"HTTP_REFERER" => "/old_url"}) visit("/") end diff --git a/spec/public/within_spec.rb b/spec/public/within_spec.rb index 6d3fd9c..ab083d2 100644 --- a/spec/public/within_spec.rb +++ b/spec/public/within_spec.rb @@ -13,7 +13,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page2", {}) + webrat_session.should_receive(:get).with("/page2", {}) within "#container" do within "div" do click_link "Link" @@ -31,7 +31,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/page2", {}) + webrat_session.should_receive(:get).with("/page2", {}) within "#container" do click_link "Link" end @@ -51,7 +51,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", "email" => "test@example.com") + webrat_session.should_receive(:get).with("/form2", "email" => "test@example.com") within "#form2" do fill_in "Email", :with => "test@example.com" click_button @@ -70,7 +70,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", "email" => "test@example.com") + webrat_session.should_receive(:get).with("/form2", "email" => "test@example.com") within ".important" do fill_in "Email", :with => "test@example.com" end @@ -92,7 +92,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", "email" => "test@example.com") + webrat_session.should_receive(:get).with("/form2", "email" => "test@example.com") within "form[@action='/form2']" do fill_in "Email", :with => "test@example.com" click_button "Add" @@ -117,7 +117,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", "email2" => "test@example.com") + webrat_session.should_receive(:get).with("/form2", "email2" => "test@example.com") within "form[@action='/form2']" do fill_in "Email", :with => "test@example.com" click_button "Add" @@ -138,7 +138,7 @@ describe "within" do HTML - webrat_session.should_receive(:get).with("http://www.example.com/form2", "email" => "test@example.com") + webrat_session.should_receive(:get).with("/form2", "email" => "test@example.com") within "#form2" do fill_in "Email", :with => "test@example.com" click_button "Add"