Merge pull request #17 from doc75/fix_big_files

ensure post is used for files (otherwise big files check are crashing)

lib/w3c_validators/css_validator.rb

@@ -61,8 +61,8 @@ def validate_uri(uri)
     # Validate the CSS of a string.
     #
     # Returns W3CValidators::Results.
-    def validate_text(text)
-      return validate({:text => text})
+    def validate_text(text, request_mode = :get)
+      return validate({:text => text}, request_mode)
     end
 
     # Validate the CSS of a local file.
@@ -77,14 +77,16 @@ def validate_file(file_path)
       else
         src = read_local_file(file_path)
       end 
-      return validate_text(src)
+      # we force the :post mode otherwise it fails for
+      # big files
+      return validate_text(src, :post)
     end
 
 
 protected
-    def validate(options) # :nodoc:
+    def validate(options, request_mode = :get) # :nodoc:
       options = get_request_options(options)
-      response = send_request(options, :get)
+      response = send_request(options, request_mode)
       @results = parse_soap_response(response.body)
       @results
     end
@@ -150,4 +152,4 @@ def parse_soap_response(response) # :nodoc:
 
 
   end
-end
+end

test/test_css_validator.rb

@@ -55,4 +55,15 @@ def test_validating_text_via_file
       assert_errors r, 1
     end
   end
+
+  def test_validating_big_file
+    VCR.use_cassette('css_validating_big_file') do
+      file_path = File.expand_path(File.dirname(__FILE__) + '/fixtures/bootstrap.min.css')
+      fh = File.new(file_path, 'r+')
+      r = @v.validate_file(fh)
+      fh.close
+      assert_errors r
+    end
+  end
+
 end