0
@@ -8,6 +8,7 @@
0
   * Support relative links, including href="?foo=bar" (Patch from Kyle Hargraves)
0
   * Separated Rails-specific code from the Webrat core to make it easier to use Webrat with other environments
0
   * Alias visits as visit, clicks_link as click_link, etc. for better readability
0
+  * Raise error when trying to interact with a disabled form element (Luke Melia)
0
   
0
 * Minor enhancements
0
 

0
@@ -45,7 +45,15 @@ module Webrat
0
     def matches_alt?(alt)
0
       @element["alt"] =~ /^\W*#{Regexp.escape(alt.to_s)}/i
0
     end
0
+
0
+    def disabled?
0
+      !@element["disabled"].nil? && @element["disabled"] != 'false'
0
+    end
0
     
0
+    def raise_error_if_disabled
0
+      raise "Cannot interact with disabled form element (#{self})" if disabled?
0
+    end
0
+        
0
     def to_param
0
       value = @value.to_s.gsub('&', '%26')
0
       param_parser.parse_query_parameters("#{name}=#{value}")
0
@@ -144,6 +152,7 @@ module Webrat
0
     end
0
 
0
     def click
0
+      raise_error_if_disabled
0
       set(@element["value"]) unless @element["name"].blank?
0
       @form.submit
0
     end
0
@@ -182,10 +191,12 @@ module Webrat
0
     end
0
 
0
     def check
0
+      raise_error_if_disabled
0
       set(@element["value"] || "on")
0
     end
0
 
0
     def uncheck
0
+      raise_error_if_disabled
0
       set(nil)
0
     end
0
 
0
@@ -212,6 +223,7 @@ module Webrat
0
     end
0
     
0
     def choose
0
+      raise_error_if_disabled
0
       other_options.each do |option|
0
         option.set(nil)
0
       end

0
@@ -22,7 +22,9 @@ module Webrat
0
     # <tt>field</tt> can be either the value of a name attribute (i.e. <tt>user[email]</tt>)
0
     # or the text inside a <tt><label></tt> element that points at the <tt><input></tt> field.
0
     def fills_in(id_or_name_or_label, options = {})
0
-      find_field(id_or_name_or_label, TextField, TextareaField, PasswordField).set(options[:with])
0
+      field = find_field(id_or_name_or_label, TextField, TextareaField, PasswordField)
0
+      field.raise_error_if_disabled
0
+      field.set(options[:with])
0
     end
0
 
0
     alias_method :fill_in, :fills_in

0
@@ -15,6 +15,7 @@ module Webrat
0
     end
0
     
0
     def choose
0
+      @select.raise_error_if_disabled
0
       @select.set(value)
0
     end
0
     

0
@@ -50,6 +50,16 @@ describe "checks" do
0
     @session.clicks_button
0
   end
0
   
0
+  it "should fail if the checkbox is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="post" action="/login">
0
+        <input type="checkbox" name="remember_me" disabled="disabled" />
0
+        <input type="submit" />
0
+      </form>
0
+    EOS
0
+    lambda { @session.checks "remember_me" }.should raise_error
0
+  end
0
+  
0
   it "should result in a custom value being posted" do
0
     @session.response_body = <<-EOS
0
       <form method="post" action="/login">
0
@@ -87,6 +97,16 @@ describe "unchecks" do
0
     lambda { @session.unchecks "remember_me" }.should raise_error
0
   end
0
   
0
+  it "should fail if the checkbox is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="post" action="/login">
0
+        <input type="checkbox" name="remember_me" checked="checked" disabled="disabled" />
0
+        <input type="submit" />
0
+      </form>
0
+    EOS
0
+    lambda { @session.unchecks "remember_me" }.should raise_error
0
+  end
0
+  
0
   it "should uncheck rails style checkboxes" do
0
     @session.response_body = <<-EOS
0
       <form method="get" action="/login">

0
@@ -53,7 +53,18 @@ describe "chooses" do
0
     @session.chooses "Female"
0
     @session.chooses "Male"
0
     @session.clicks_button
0
-  end  
0
+  end
0
+  
0
+  it "should fail if the radio button is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="post" action="/login">
0
+        <input type="radio" name="first_option" disabled="disabled" />
0
+        <input type="submit" />
0
+      </form>
0
+    EOS
0
+    
0
+    lambda { @session.chooses "first_option" }.should raise_error
0
+  end
0
   
0
   it "should result in the value on being posted if not specified" do
0
     @session.response_body = <<-EOS

0
@@ -22,6 +22,17 @@ describe "clicks_button" do
0
 
0
     lambda { @session.clicks_button }.should raise_error
0
   end
0
+
0
+  
0
+  it "should fail if button is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="get" action="/login">
0
+        <input type="submit" disabled="disabled" />
0
+      </form>
0
+    EOS
0
+
0
+    lambda { @session.clicks_button }.should raise_error
0
+  end
0
   
0
   it "should default to get method" do
0
     @session.response_body = <<-EOS

0
@@ -39,6 +39,18 @@ describe "fills_in" do
0
     lambda { @session.fills_in "Email", :with => "foo@example.com" }.should raise_error
0
   end
0
   
0
+  it "should fail if input is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="get" action="/login">
0
+        <label for="user_email">Email</label>
0
+        <input id="user_email" name="user[email]" type="text" disabled="disabled" />
0
+        <input type="submit" />
0
+      </form>
0
+    EOS
0
+    
0
+    lambda { @session.fills_in "Email", :with => "foo@example.com" }.should raise_error
0
+  end
0
+  
0
   it "should allow overriding default form values" do
0
     @session.response_body = <<-EOS
0
       <form method="post" action="/login">

0
@@ -35,6 +35,18 @@ describe "selects" do
0
 
0
     lambda { @session.selects "February", :from => "year" }.should raise_error
0
   end
0
+
0
+  
0
+  it "should fail if the select is disabled" do
0
+    @session.response_body = <<-EOS
0
+      <form method="post" action="/login">
0
+        <select name="month" disabled="disabled"><option value="1">January</option></select>
0
+        <input type="submit" />
0
+      </form>
0
+    EOS
0
+
0
+    lambda { @session.selects "January", :from => "month" }.should raise_error
0
+  end
0
   
0
   it "should send value from option" do
0
     @session.response_body = <<-EOS