From 63bc602f2da2e49ac2267346042774a370db2336 Mon Sep 17 00:00:00 2001
From: Jonas Nicklas <jonas.nicklas@gmail.com>
Date: Wed, 18 Nov 2009 22:06:06 +0100
Subject: [PATCH] Don't send query params from 'action' in GET forms

---
 lib/capybara/driver/rack_test_driver.rb | 2 +-
 spec/session_spec.rb                    | 2 +-
 spec/views/form.erb                     | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/capybara/driver/rack_test_driver.rb b/lib/capybara/driver/rack_test_driver.rb
index 094baf3f..387e3b6b 100644
--- a/lib/capybara/driver/rack_test_driver.rb
+++ b/lib/capybara/driver/rack_test_driver.rb
@@ -102,7 +102,7 @@ class Capybara::Driver::RackTest
       if post?
         driver.submit(node['action'].to_s, params(button)) 
       else
-        driver.visit(node['action'].to_s + '?' + params(button)) 
+        driver.visit(node['action'].to_s.split('?').first + '?' + params(button)) 
       end
     end
 
diff --git a/spec/session_spec.rb b/spec/session_spec.rb
index 10efa508..c3c0918e 100644
--- a/spec/session_spec.rb
+++ b/spec/session_spec.rb
@@ -156,9 +156,9 @@ shared_examples_for "session" do
     it "should serialize and send GET forms" do
       @session.visit('/form')
       @session.click_button('med')
-      #puts @session.body
       @results = extract_results(@session)
       @results['middle_name'].should == 'Darren'
+      @results['foo'].should be_nil
     end
 
     it "should follow redirects" do
diff --git a/spec/views/form.erb b/spec/views/form.erb
index e04d18b8..d985e6a0 100644
--- a/spec/views/form.erb
+++ b/spec/views/form.erb
@@ -88,7 +88,7 @@
   </p>
 </form>
 
-<form action="/form/get" method="get">
+<form action="/form/get?foo=bar" method="get">
   <p>
     <label for="form_middle_name">Middle Name</label>
     <input type="text" name="form[middle_name]" value="Darren" id="form_middle_name"/>