csvlint 1.0.0 → 1.2.0

data/spec/validator_spec.rb

@@ -1,49 +1,45 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Csvlint::Validator do
-
   before do
-    stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :body => "")
-    stub_request(:get, "http://example.com/.well-known/csvm").to_return(:status => 404)
-    stub_request(:get, "http://example.com/example.csv-metadata.json").to_return(:status => 404)
-    stub_request(:get, "http://example.com/csv-metadata.json").to_return(:status => 404)
+    stub_request(:get, "http://example.com/example.csv").to_return(status: 200, body: "")
+    stub_request(:get, "http://example.com/.well-known/csvm").to_return(status: 404)
+    stub_request(:get, "http://example.com/example.csv-metadata.json").to_return(status: 404)
+    stub_request(:get, "http://example.com/csv-metadata.json").to_return(status: 404)
   end
 
   it "should validate from a URL" do
-    stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+    stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
     validator = Csvlint::Validator.new("http://example.com/example.csv")
 
     expect(validator.valid?).to eql(true)
-    expect(validator.instance_variable_get("@expected_columns")).to eql(3)
-    expect(validator.instance_variable_get("@col_counts").count).to eql(3)
+    expect(validator.instance_variable_get(:@expected_columns)).to eql(3)
+    expect(validator.instance_variable_get(:@col_counts).count).to eql(3)
     expect(validator.data.size).to eql(3)
   end
 
   it "should validate from a file path" do
-    validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+    validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
 
     expect(validator.valid?).to eql(true)
-    expect(validator.instance_variable_get("@expected_columns")).to eql(3)
-    expect(validator.instance_variable_get("@col_counts").count).to eql(3)
+    expect(validator.instance_variable_get(:@expected_columns)).to eql(3)
+    expect(validator.instance_variable_get(:@col_counts).count).to eql(3)
     expect(validator.data.size).to eql(3)
   end
 
   it "should validate from a file path including whitespace" do
-    validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__),'..','features','fixtures','white space in filename.csv')))
+    validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "white space in filename.csv")))
 
     expect(validator.valid?).to eql(true)
   end
 
   context "multi line CSV validation with included schema" do
-
   end
 
   context "single line row validation with included schema" do
-
   end
 
   context "validation with multiple lines: " do
-
     # TODO multiple lines permits testing of warnings
     # TODO need more assertions in each test IE @formats
     # TODO the phrasing of col_counts if only consulting specs might be confusing
@@ -52,22 +48,21 @@ describe Csvlint::Validator do
 
     it ".each() -> parse_contents method validates a well formed CSV" do
       # when invoking parse contents
-      data = StringIO.new(%Q{"Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1"})
+      data = StringIO.new(%("Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1"))
 
       validator = Csvlint::Validator.new(data)
 
       expect(validator.valid?).to eql(true)
       # TODO would be beneficial to know how formats functions WRT to headers - check_format.feature:17 returns 3 rows total
       # TODO in its formats object but is provided with 5 rows (with one nil row) [uses validation_warnings_steps.rb]
-      expect(validator.instance_variable_get("@expected_columns")).to eql(3)
-      expect(validator.instance_variable_get("@col_counts").count).to eql(4)
+      expect(validator.instance_variable_get(:@expected_columns)).to eql(3)
+      expect(validator.instance_variable_get(:@col_counts).count).to eql(4)
       expect(validator.data.size).to eql(4)
-
     end
 
     it ".each() -> `parse_contents` parses malformed CSV and catches unclosed quote" do
       # doesn't build warnings because check_consistency isn't invoked
-      data = StringIO.new(%Q{"Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1})
+      data = StringIO.new(%("Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1))
 
       validator = Csvlint::Validator.new(data)
 
@@ -79,7 +74,7 @@ describe Csvlint::Validator do
     it ".each() -> `parse_contents` parses malformed CSV and catches whitespace and edge case" do
       # when this data gets passed the header it rescues a whitespace error, resulting in the header row being discarded
       # TODO - check if this is an edge case, currently passing because it requires advice on how to specify
-      data = StringIO.new(%Q{ "Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1" })
+      data = StringIO.new(%( "Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1" ))
 
       validator = Csvlint::Validator.new(data)
 
@@ -89,13 +84,13 @@ describe Csvlint::Validator do
     end
 
     it "handles line breaks within a cell" do
-      data = StringIO.new(%Q{"a","b","c"\r\n"d","e","this is\r\nvalid"\r\n"a","b","c"})
+      data = StringIO.new(%("a","b","c"\r\n"d","e","this is\r\nvalid"\r\n"a","b","c"))
       validator = Csvlint::Validator.new(data)
       expect(validator.valid?).to eql(true)
     end
 
     it "handles multiple line breaks within a cell" do
-      data = StringIO.new(%Q{"a","b","c"\r\n"d","this is\r\n valid","as is this\r\n too"})
+      data = StringIO.new(%("a","b","c"\r\n"d","this is\r\n valid","as is this\r\n too"))
       validator = Csvlint::Validator.new(data)
       expect(validator.valid?).to eql(true)
     end
@@ -104,50 +99,50 @@ describe Csvlint::Validator do
   context "csv dialect" do
     it "should provide sensible defaults for CSV parsing" do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      opts = validator.instance_variable_get("@csv_options")
+      opts = validator.instance_variable_get(:@csv_options)
       expect(opts).to include({
-        :col_sep => ",",
-        :row_sep => :auto,
-        :quote_char => '"',
-        :skip_blanks => false
+        col_sep: ",",
+        row_sep: :auto,
+        quote_char: '"',
+        skip_blanks: false
       })
     end
 
     it "should map CSV DDF to correct values" do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      opts = validator.dialect_to_csv_options( {
+      opts = validator.dialect_to_csv_options({
         "lineTerminator" => "\n",
         "delimiter" => "\t",
         "quoteChar" => "'"
       })
       expect(opts).to include({
-        :col_sep => "\t",
-        :row_sep => "\n",
-        :quote_char => "'",
-        :skip_blanks => false
+        col_sep: "\t",
+        row_sep: "\n",
+        quote_char: "'",
+        skip_blanks: false
       })
     end
 
     it ".each() -> `validate` to pass input in streaming fashion" do
       # warnings are built when validate is used to call all three methods
-      data = StringIO.new(%Q{"Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1"})
+      data = StringIO.new(%("Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"3","2","1"))
       validator = Csvlint::Validator.new(data)
 
       expect(validator.valid?).to eql(true)
-      expect(validator.instance_variable_get("@expected_columns")).to eql(3)
-      expect(validator.instance_variable_get("@col_counts").count).to eql(4)
+      expect(validator.instance_variable_get(:@expected_columns)).to eql(3)
+      expect(validator.instance_variable_get(:@col_counts).count).to eql(4)
       expect(validator.data.size).to eql(4)
       expect(validator.info_messages.count).to eql(1)
     end
 
     it ".each() -> `validate` parses malformed CSV, populates errors, warnings & info_msgs,invokes finish()" do
-      data = StringIO.new(%Q{"Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"1","two","3"\r\n"3","2",   "1"})
+      data = StringIO.new(%("Foo","Bar","Baz"\r\n"1","2","3"\r\n"1","2","3"\r\n"1","two","3"\r\n"3","2",   "1"))
 
       validator = Csvlint::Validator.new(data)
 
       expect(validator.valid?).to eql(false)
-      expect(validator.instance_variable_get("@expected_columns")).to eql(3)
-      expect(validator.instance_variable_get("@col_counts").count).to eql(4)
+      expect(validator.instance_variable_get(:@expected_columns)).to eql(3)
+      expect(validator.instance_variable_get(:@col_counts).count).to eql(4)
       expect(validator.data.size).to eql(5)
       expect(validator.info_messages.count).to eql(1)
       expect(validator.errors.count).to eql(1)
@@ -157,7 +152,7 @@ describe Csvlint::Validator do
     end
 
     it "File.open.each_line -> `validate` passes a valid csv" do
-      filename = 'valid_many_rows.csv'
+      filename = "valid_many_rows.csv"
       file = File.join(File.expand_path(Dir.pwd), "features", "fixtures", filename)
       validator = Csvlint::Validator.new(File.new(file))
 
@@ -166,11 +161,9 @@ describe Csvlint::Validator do
       expect(validator.info_messages.first.type).to eql(:assumed_header)
       expect(validator.info_messages.first.category).to eql(:structure)
     end
-
   end
 
   context "with a single row" do
-
     it "validates correctly" do
       stream = "\"a\",\"b\",\"c\"\r\n"
       validator = Csvlint::Validator.new(StringIO.new(stream), "header" => false)
@@ -204,10 +197,10 @@ describe Csvlint::Validator do
       stream = "1,2,3\r\n"
       validator = Csvlint::Validator.new(StringIO.new(stream))
 
-      expect( validator.valid? ).to eql(true)
-      expect( validator.info_messages.size ).to eql(1)
-      expect( validator.info_messages.first.type).to eql(:assumed_header)
-      expect( validator.info_messages.first.category).to eql(:structure)
+      expect(validator.valid?).to eql(true)
+      expect(validator.info_messages.size).to eql(1)
+      expect(validator.info_messages.first.type).to eql(:assumed_header)
+      expect(validator.info_messages.first.category).to eql(:structure)
     end
 
     it "should evaluate the row as 'row 2' when stipulated" do
@@ -217,11 +210,9 @@ describe Csvlint::Validator do
       expect(validator.valid?).to eql(true)
       expect(validator.info_messages.size).to eql(0)
     end
-
   end
 
   context "it returns the correct error from ERROR_MATCHES" do
-
     it "checks for unclosed quotes" do
       stream = "\"a,\"b\",\"c\"\n"
       validator = Csvlint::Validator.new(StringIO.new(stream))
@@ -230,7 +221,6 @@ describe Csvlint::Validator do
       expect(validator.errors.first.type).to eql(:unclosed_quote)
     end
 
-
     # TODO stray quotes is not covered in any spec in this library
     # it "checks for stray quotes" do
     #   stream = "\"a\",“b“,\"c\"" "\r\n"
@@ -258,64 +248,61 @@ describe Csvlint::Validator do
       expect(validator.errors.count).to eq(1)
       expect(validator.errors.first.type).to eql(:line_breaks)
     end
-
   end
 
   context "when validating headers" do
-
     it "should warn if column names aren't unique" do
-      data = StringIO.new( "minimum, minimum" )
+      data = StringIO.new("minimum, minimum")
       validator = Csvlint::Validator.new(data)
       validator.reset
-      expect( validator.validate_header(["minimum", "minimum"]) ).to eql(true)
-      expect( validator.warnings.size ).to eql(1)
-      expect( validator.warnings.first.type).to eql(:duplicate_column_name)
-      expect( validator.warnings.first.category).to eql(:schema)
+      expect(validator.validate_header(["minimum", "minimum"])).to eql(true)
+      expect(validator.warnings.size).to eql(1)
+      expect(validator.warnings.first.type).to eql(:duplicate_column_name)
+      expect(validator.warnings.first.category).to eql(:schema)
     end
 
     it "should warn if column names are blank" do
-      data = StringIO.new( "minimum," )
+      data = StringIO.new("minimum,")
       validator = Csvlint::Validator.new(data)
 
-      expect( validator.validate_header(["minimum", ""]) ).to eql(true)
-      expect( validator.warnings.size ).to eql(1)
-      expect( validator.warnings.first.type).to eql(:empty_column_name)
-      expect( validator.warnings.first.category).to eql(:schema)
+      expect(validator.validate_header(["minimum", ""])).to eql(true)
+      expect(validator.warnings.size).to eql(1)
+      expect(validator.warnings.first.type).to eql(:empty_column_name)
+      expect(validator.warnings.first.category).to eql(:schema)
     end
 
     it "should include info message about missing header when we have assumed a header" do
-      data = StringIO.new( "1,2,3\r\n" )
+      data = StringIO.new("1,2,3\r\n")
       validator = Csvlint::Validator.new(data)
-      expect( validator.valid? ).to eql(true)
-      expect( validator.info_messages.size ).to eql(1)
-      expect( validator.info_messages.first.type).to eql(:assumed_header)
-      expect( validator.info_messages.first.category).to eql(:structure)
+      expect(validator.valid?).to eql(true)
+      expect(validator.info_messages.size).to eql(1)
+      expect(validator.info_messages.first.type).to eql(:assumed_header)
+      expect(validator.info_messages.first.category).to eql(:structure)
     end
 
     it "should not include info message about missing header when we are told about the header" do
-      data = StringIO.new( "1,2,3\r\n" )
+      data = StringIO.new("1,2,3\r\n")
       validator = Csvlint::Validator.new(data, "header" => false)
-      expect( validator.valid? ).to eql(true)
-      expect( validator.info_messages.size ).to eql(0)
+      expect(validator.valid?).to eql(true)
+      expect(validator.info_messages.size).to eql(0)
     end
   end
 
   context "build_formats" do
-
     {
-        :string => "foo",
-        :numeric => "1",
-        :uri => "http://www.example.com",
-        :dateTime_iso8601 => "2013-01-01T13:00:00Z",
-        :date_db => "2013-01-01",
-        :dateTime_hms => "13:00:00"
+      string: "foo",
+      numeric: "1",
+      uri: "http://www.example.com",
+      dateTime_iso8601: "2013-01-01T13:00:00Z",
+      date_db: "2013-01-01",
+      dateTime_hms: "13:00:00"
     }.each do |type, content|
       it "should return the format of #{type} correctly" do
         row = [content]
 
         validator = Csvlint::Validator.new("http://example.com/example.csv")
         validator.build_formats(row)
-        formats = validator.instance_variable_get("@formats")
+        formats = validator.instance_variable_get(:@formats)
 
         expect(formats[0].keys.first).to eql type
       end
@@ -326,7 +313,7 @@ describe Csvlint::Validator do
 
       validator = Csvlint::Validator.new("http://example.com/example.csv")
       validator.build_formats(row)
-      formats = validator.instance_variable_get("@formats")
+      formats = validator.instance_variable_get(:@formats)
 
       expect(formats[0].keys.first).to eql :numeric
       expect(formats[1].keys.first).to eql :numeric
@@ -338,15 +325,15 @@ describe Csvlint::Validator do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
       validator.build_formats(row)
 
-      formats = validator.instance_variable_get("@formats")
+      formats = validator.instance_variable_get(:@formats)
       expect(formats).to eql []
     end
 
     it "should work correctly for single columns" do
       rows = [
-          ["foo"],
-          ["bar"],
-          ["baz"]
+        ["foo"],
+        ["bar"],
+        ["baz"]
       ]
 
       validator = Csvlint::Validator.new("http://example.com/example.csv")
@@ -355,14 +342,14 @@ describe Csvlint::Validator do
         validator.build_formats(row)
       end
 
-      formats = validator.instance_variable_get("@formats")
-      expect(formats).to eql [{:string => 3}]
+      formats = validator.instance_variable_get(:@formats)
+      expect(formats).to eql [{string: 3}]
     end
 
     it "should return formats correctly if a row is blank" do
       rows = [
-          [],
-          ["foo", "1", "$2345"]
+        [],
+        ["foo", "1", "$2345"]
       ]
 
       validator = Csvlint::Validator.new("http://example.com/example.csv")
@@ -371,72 +358,68 @@ describe Csvlint::Validator do
         validator.build_formats(row)
       end
 
-      formats = validator.instance_variable_get("@formats")
+      formats = validator.instance_variable_get(:@formats)
 
       expect(formats).to eql [
-        {:string => 1},
-        {:numeric => 1},
-        {:string => 1},
+        {string: 1},
+        {numeric: 1},
+        {string: 1}
       ]
     end
-
   end
 
   context "csv dialect" do
     it "should provide sensible defaults for CSV parsing" do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      opts = validator.instance_variable_get("@csv_options")
+      opts = validator.instance_variable_get(:@csv_options)
       expect(opts).to include({
-                              :col_sep => ",",
-                              :row_sep => :auto,
-                              :quote_char => '"',
-                              :skip_blanks => false
-                          })
+        col_sep: ",",
+        row_sep: :auto,
+        quote_char: '"',
+        skip_blanks: false
+      })
     end
 
     it "should map CSV DDF to correct values" do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
       opts = validator.dialect_to_csv_options({
-                                                  "lineTerminator" => "\n",
-                                                  "delimiter" => "\t",
-                                                  "quoteChar" => "'"
-                                              })
+        "lineTerminator" => "\n",
+        "delimiter" => "\t",
+        "quoteChar" => "'"
+      })
       expect(opts).to include({
-                              :col_sep => "\t",
-                              :row_sep => "\n",
-                              :quote_char => "'",
-                              :skip_blanks => false
-                          })
+        col_sep: "\t",
+        row_sep: "\n",
+        quote_char: "'",
+        skip_blanks: false
+      })
     end
-
   end
 
   context "check_consistency" do
-
     it "should return a warning if columns have inconsistent values" do
       formats = [
-          {:string => 3},
-          {:string => 2, :numeric => 1},
-          {:numeric => 3},
+        {string: 3},
+        {string: 2, numeric: 1},
+        {numeric: 3}
       ]
 
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      validator.instance_variable_set("@formats", formats)
+      validator.instance_variable_set(:@formats, formats)
       validator.check_consistency
 
-      warnings = validator.instance_variable_get("@warnings")
+      warnings = validator.instance_variable_get(:@warnings)
       warnings.delete_if { |h| h.type != :inconsistent_values }
 
       expect(warnings.count).to eql 1
     end
-
   end
 
-  #TODO the below tests are all the remaining tests from validator_spec.rb, annotations indicate their status HOWEVER these tests may be best refactored into client specs
+  # TODO the below tests are all the remaining tests from validator_spec.rb, annotations indicate their status HOWEVER these tests may be best refactored into client specs
   context "when detecting headers" do
     it "should default to expecting a header" do
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.header? ).to eql(true)
+      expect(validator.header?).to eql(true)
     end
 
     it "should look in CSV options to detect header" do
@@ -444,171 +427,165 @@ describe Csvlint::Validator do
         "header" => true
       }
       validator = Csvlint::Validator.new("http://example.com/example.csv", opts)
-      expect( validator.header? ).to eql(true)
+      expect(validator.header?).to eql(true)
       opts = {
         "header" => false
       }
       validator = Csvlint::Validator.new("http://example.com/example.csv", opts)
-      expect( validator.header? ).to eql(false)
+      expect(validator.header?).to eql(false)
     end
 
     it "should look in content-type for header=absent" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv; header=absent"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv; header=absent"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.header? ).to eql(false)
-      expect( validator.errors.size ).to eql(0)
+      expect(validator.header?).to eql(false)
+      expect(validator.errors.size).to eql(0)
     end
 
     it "should look in content-type for header=present" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv; header=present"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv; header=present"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.header? ).to eql(true)
-      expect( validator.errors.size ).to eql(0)
+      expect(validator.header?).to eql(true)
+      expect(validator.errors.size).to eql(0)
     end
 
     it "assume header present if not specified in content type" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.header? ).to eql(true)
-      expect( validator.errors.size ).to eql(0)
-      expect( validator.info_messages.size ).to eql(1)
-      expect( validator.info_messages.first.type).to eql(:assumed_header)
+      expect(validator.header?).to eql(true)
+      expect(validator.errors.size).to eql(0)
+      expect(validator.info_messages.size).to eql(1)
+      expect(validator.info_messages.first.type).to eql(:assumed_header)
     end
 
     it "give wrong content type error if content type is wrong" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/html"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/html"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.header? ).to eql(true)
-      expect( validator.errors.size ).to eql(1)
-      expect( validator.errors[0].type).to eql(:wrong_content_type)
+      expect(validator.header?).to eql(true)
+      expect(validator.errors.size).to eql(1)
+      expect(validator.errors[0].type).to eql(:wrong_content_type)
     end
-
   end
 
   context "when validating headers" do
     it "should warn if column names aren't unique" do
-      data = StringIO.new( "minimum, minimum" )
+      data = StringIO.new("minimum, minimum")
       validator = Csvlint::Validator.new(data)
-      expect( validator.warnings.size ).to eql(1)
-      expect( validator.warnings.first.type).to eql(:duplicate_column_name)
-      expect( validator.warnings.first.category).to eql(:schema)
+      expect(validator.warnings.size).to eql(1)
+      expect(validator.warnings.first.type).to eql(:duplicate_column_name)
+      expect(validator.warnings.first.category).to eql(:schema)
     end
 
     it "should warn if column names are blank" do
-      data = StringIO.new( "minimum," )
+      data = StringIO.new("minimum,")
       validator = Csvlint::Validator.new(data)
 
-      expect( validator.validate_header(["minimum", ""]) ).to eql(true)
-      expect( validator.warnings.size ).to eql(1)
-      expect( validator.warnings.first.type).to eql(:empty_column_name)
-      expect( validator.warnings.first.category).to eql(:schema)
+      expect(validator.validate_header(["minimum", ""])).to eql(true)
+      expect(validator.warnings.size).to eql(1)
+      expect(validator.warnings.first.type).to eql(:empty_column_name)
+      expect(validator.warnings.first.category).to eql(:schema)
     end
 
     it "should include info message about missing header when we have assumed a header" do
-      data = StringIO.new( "1,2,3\r\n" )
+      data = StringIO.new("1,2,3\r\n")
       validator = Csvlint::Validator.new(data)
 
-      expect( validator.valid? ).to eql(true)
-      expect( validator.info_messages.size ).to eql(1)
-      expect( validator.info_messages.first.type).to eql(:assumed_header)
-      expect( validator.info_messages.first.category).to eql(:structure)
+      expect(validator.valid?).to eql(true)
+      expect(validator.info_messages.size).to eql(1)
+      expect(validator.info_messages.first.type).to eql(:assumed_header)
+      expect(validator.info_messages.first.category).to eql(:structure)
     end
 
     it "should not include info message about missing header when we are told about the header" do
-      data = StringIO.new( "1,2,3\r\n" )
-      validator = Csvlint::Validator.new(data, "header"=>false)
-      expect( validator.valid? ).to eql(true)
-      expect( validator.info_messages.size ).to eql(0)
+      data = StringIO.new("1,2,3\r\n")
+      validator = Csvlint::Validator.new(data, "header" => false)
+      expect(validator.valid?).to eql(true)
+      expect(validator.info_messages.size).to eql(0)
     end
 
     it "should not be an error if we have assumed a header, there is no dialect and content-type doesn't declare header, as we assume header=present" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
       validator = Csvlint::Validator.new("http://example.com/example.csv")
-      expect( validator.valid? ).to eql(true)
+      expect(validator.valid?).to eql(true)
     end
 
     it "should be valid if we have a dialect and the data is from the web" do
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
-      #header defaults to true in csv dialect, so this is valid
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
+      # header defaults to true in csv dialect, so this is valid
       validator = Csvlint::Validator.new("http://example.com/example.csv", {})
-      expect( validator.valid? ).to eql(true)
+      expect(validator.valid?).to eql(true)
 
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
-      validator = Csvlint::Validator.new("http://example.com/example.csv", {"header"=>true})
-      expect( validator.valid? ).to eql(true)
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
+      validator = Csvlint::Validator.new("http://example.com/example.csv", {"header" => true})
+      expect(validator.valid?).to eql(true)
 
-      stub_request(:get, "http://example.com/example.csv").to_return(:status => 200, :headers=>{"Content-Type" => "text/csv"}, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
-      validator = Csvlint::Validator.new("http://example.com/example.csv", {"header"=>false})
-      expect( validator.valid? ).to eql(true)
+      stub_request(:get, "http://example.com/example.csv").to_return(status: 200, headers: {"Content-Type" => "text/csv"}, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
+      validator = Csvlint::Validator.new("http://example.com/example.csv", {"header" => false})
+      expect(validator.valid?).to eql(true)
     end
-
   end
 
   context "accessing metadata" do
-
     before :all do
-      stub_request(:get, "http://example.com/crlf.csv").to_return(:status => 200, :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','windows-line-endings.csv')))
-      stub_request(:get, "http://example.com/crlf.csv-metadata.json").to_return(:status => 404)
+      stub_request(:get, "http://example.com/crlf.csv").to_return(status: 200, body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "windows-line-endings.csv")))
+      stub_request(:get, "http://example.com/crlf.csv-metadata.json").to_return(status: 404)
     end
 
     it "can get line break symbol" do
       validator = Csvlint::Validator.new("http://example.com/crlf.csv")
       expect(validator.line_breaks).to eql "\r\n"
     end
-
   end
 
   it "should give access to the complete CSV data file" do
-    stub_request(:get, "http://example.com/example.csv").to_return(:status => 200,
-        :headers=>{"Content-Type" => "text/csv; header=present"},
-        :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+    stub_request(:get, "http://example.com/example.csv").to_return(status: 200,
+      headers: {"Content-Type" => "text/csv; header=present"},
+      body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
     validator = Csvlint::Validator.new("http://example.com/example.csv")
-    expect( validator.valid? ).to eql(true)
+    expect(validator.valid?).to eql(true)
     data = validator.data
 
-    expect( data.count ).to eql 3
-    expect( data[0] ).to eql ['Foo','Bar','Baz']
-    expect( data[2] ).to eql ['3','2','1']
+    expect(data.count).to eql 3
+    expect(data[0]).to eql ["Foo", "Bar", "Baz"]
+    expect(data[2]).to eql ["3", "2", "1"]
   end
 
   it "should count the total number of rows read" do
-    stub_request(:get, "http://example.com/example.csv").to_return(:status => 200,
-        :headers=>{"Content-Type" => "text/csv; header=present"},
-        :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+    stub_request(:get, "http://example.com/example.csv").to_return(status: 200,
+      headers: {"Content-Type" => "text/csv; header=present"},
+      body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
     validator = Csvlint::Validator.new("http://example.com/example.csv")
     expect(validator.row_count).to eq(3)
   end
 
   it "should limit number of lines read" do
-    stub_request(:get, "http://example.com/example.csv").to_return(:status => 200,
-    :headers=>{"Content-Type" => "text/csv; header=present"},
-    :body => File.read(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')))
+    stub_request(:get, "http://example.com/example.csv").to_return(status: 200,
+      headers: {"Content-Type" => "text/csv; header=present"},
+      body: File.read(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")))
     validator = Csvlint::Validator.new("http://example.com/example.csv", {}, nil, limit_lines: 2)
-    expect( validator.valid? ).to eql(true)
+    expect(validator.valid?).to eql(true)
     data = validator.data
-    expect( data.count ).to eql 2
-    expect( data[0] ).to eql ['Foo','Bar','Baz']
+    expect(data.count).to eql 2
+    expect(data[0]).to eql ["Foo", "Bar", "Baz"]
   end
 
   context "with a lambda" do
-
     it "should call a lambda for each line" do
       @count = 0
-      mylambda = lambda { |row| @count = @count + 1 }
-      validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')), {}, nil, { lambda: mylambda })
+      mylambda = lambda { |row| @count += 1 }
+      validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")), {}, nil, {lambda: mylambda})
       expect(@count).to eq(3)
     end
 
     it "reports back the status of each line" do
       @results = []
       mylambda = lambda { |row| @results << row.current_line }
-      validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__),'..','features','fixtures','valid.csv')), {}, nil, { lambda: mylambda })
+      validator = Csvlint::Validator.new(File.new(File.join(File.dirname(__FILE__), "..", "features", "fixtures", "valid.csv")), {}, nil, {lambda: mylambda})
       expect(@results.count).to eq(3)
       expect(@results[0]).to eq(1)
       expect(@results[1]).to eq(2)
       expect(@results[2]).to eq(3)
     end
-
   end
 
   # Commented out because there is currently no way to mock redirects with Typhoeus and WebMock - see https://github.com/bblimke/webmock/issues/237