Merge pull request #7 from theodi/feature-detect-blank-rows

Looks good, merged

features/validation_errors.feature

@@ -32,4 +32,17 @@ Feature: Get validation errors
     When I ask if there are errors
     Then there should be 1 error
     And that error should have the type "whitespace"
-    And that error should have the position "1"
+    And that error should have the position "1"
+
+  Scenario: Successfully report a CSV with blank rows
+    Given I have a CSV with the following content:
+    """
+"Foo","Bar","Baz"
+"","",
+"Baz","Bar","Foo"
+    """
+    And it is stored at the url "http://example.com/example1.csv"
+    When I ask if there are errors
+    Then there should be 1 error
+    And that error should have the type "blank_rows"
+    And that error should have the position "2"

lib/csvlint.rb

@@ -39,6 +39,7 @@ def validate
             row = CSV.parse( line )[0]
             expected_columns = row.count unless expected_columns != 0
             build_errors(:ragged_rows, current_line) if row.count != expected_columns
+            build_errors(:blank_rows, current_line) if row.reject{ |c| c.nil? || c.empty? }.count == 0
           rescue CSV::MalformedCSVError => e
             type = fetch_error(e)
             build_errors(type, current_line)
@@ -68,4 +69,5 @@ def fetch_error(error)
     end
 
   end
-end
+
+end