csvlint 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 651e0cab26de36c97d333823f6ac72ad1377397e5e5dbf02397725fd8b42d663
-  data.tar.gz: cf81fa72bc040e242e947c8abb7009f7dd8ec6d5c9f08c21c9177eb62ab9c648
+  metadata.gz: 7f9c598fca283466c50e65c4b436166fccb0e9e0340d3d370fb3f4e814cf1f3e
+  data.tar.gz: 321481d47fca1f6ef1df7963e5d705ce05685a9861e48d954397eb288f46541a
 SHA512:
-  metadata.gz: dbe214a6737375e2e5f4b6c08c63345ce850eaaea30373fab732a8352b84e58f862af5490d39dc18415f0256b92d0625c51c8e61ade9d2ef562050239340d985
-  data.tar.gz: 79296f466687b53a26a3e1c402a2ac7a6f2c2f999092a11567c897d71e3b78eca592cba97da211af5226e09ffd7086ed2cf0433cdeb14ad8c0635e5f6b55a7b8
+  metadata.gz: 1e9f755b174f3c1b3052f43621e8c15cf3a3b947a5a611f370b42f309492797664a622886520a7d209d109f271a2d101aaa24017529900c82d76a71b8cb46173
+  data.tar.gz: 73da87ddbbc71a1e62f90ffaebca3947833921f546dcb01524bd464c7fbf1a3dbf086757443838e6ec401bac86e5cacf26a9e3b397904ba9ee32a0688c4271a0

data/.github/dependabot.yml

@@ -5,3 +5,7 @@ updates:
   schedule:
     interval: daily
   open-pull-requests-limit: 10
+- package-ecosystem: github-actions
+  directory: "/"
+  schedule:
+    interval: weekly

data/.github/workflows/push.yml

@@ -9,10 +9,10 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        ruby-version: ['2.5', '2.6', '2.7', '3.0', '3.1']
+        ruby-version: ['2.5', '2.6', '2.7', '3.0', '3.1', '3.2']
       fail-fast: false
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
       - uses: ruby/setup-ruby@v1
         with:
           bundler-cache: true
@@ -21,3 +21,15 @@ jobs:
         run: bundle install
       - name: Run the tests
         run: bundle exec rake
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true
+          ruby-version: "3.2"
+      - name: Install dependencies
+        run: bundle install
+      - name: Run the tests
+        run: bundle exec standardrb

data/.ruby-version

@@ -1 +1 @@
-3.1.2
+3.2.0

data/.standard_todo.yml

@@ -0,0 +1,43 @@
+# Auto generated files with errors to ignore.
+# Remove from this list as you refactor files.
+---
+ignore:
+- features/step_definitions/validation_errors_steps.rb:
+  - Lint/Void
+- features/support/load_tests.rb:
+  - Style/For
+  - Security/Open
+  - Lint/UselessAssignment
+- lib/csvlint/cli.rb:
+  - Style/NonNilCheck
+- lib/csvlint/csvw/column.rb:
+  - Style/TernaryParentheses
+- lib/csvlint/csvw/date_format.rb:
+  - Lint/MixedRegexpCaptureTypes
+- lib/csvlint/csvw/number_format.rb:
+  - Style/SlicingWithRange
+  - Style/IdenticalConditionalBranches
+- lib/csvlint/csvw/property_checker.rb:
+  - Performance/InefficientHashSearch
+  - Lint/UselessAssignment
+  - Naming/VariableName
+  - Style/SlicingWithRange
+  - Security/Open
+  - Lint/BooleanSymbol
+- lib/csvlint/csvw/table_group.rb:
+  - Style/OptionalArguments
+- lib/csvlint/field.rb:
+  - Naming/VariableName
+- lib/csvlint/schema.rb:
+  - Security/Open
+  - Lint/UselessAssignment
+  - Style/SlicingWithRange
+- lib/csvlint/validate.rb:
+  - Lint/UselessAssignment
+  - Performance/Count
+  - Lint/BooleanSymbol
+  - Naming/VariableName
+  - Security/Open
+  - Lint/NonLocalExitFromIterator
+- spec/validator_spec.rb:
+  - Lint/UselessAssignment

data/Dockerfile

@@ -0,0 +1,16 @@
+FROM ruby:2.5.8-buster
+
+# throw errors if Gemfile has been modified since Gemfile.lock
+RUN bundle config --global frozen 1
+
+WORKDIR /usr/src/app
+
+ENV LANG C.UTF-8
+
+COPY ./lib/csvlint/version.rb ./lib/csvlint/
+COPY csvlint.gemspec Gemfile Gemfile.lock ./
+RUN bundle install
+
+COPY ./ ./
+
+CMD ["./bin/csvlint"]

data/Gemfile

@@ -1,4 +1,4 @@
-source 'https://rubygems.org'
+source "https://rubygems.org"
 
 # Specify your gem's dependencies in csvlint.rb.gemspec
-gemspec
+gemspec

data/README.md

@@ -1,6 +1,6 @@
-[![Build Status](http://img.shields.io/travis/theodi/csvlint.rb.svg)](https://travis-ci.org/theodi/csvlint.rb)
-[![Dependency Status](http://img.shields.io/gemnasium/theodi/csvlint.rb.svg)](https://gemnasium.com/theodi/csvlint.rb)
-[![Coverage Status](http://img.shields.io/coveralls/theodi/csvlint.rb.svg)](https://coveralls.io/r/theodi/csvlint.rb)
+[![Build Status](https://img.shields.io/github/workflow/status/Data-Liberation-Front/csvlint.rb/CI/main)](https://travis-ci.org/theodi/csvlint.rb)
+[![Dependency Status](https://img.shields.io/librariesio/github/Data-Liberation-Front/csvlint.rb)](https://libraries.io/github/Data-Liberation-Front/csvlint.rb)
+[![Coverage Status](http://img.shields.io/coveralls/Data-Liberation-Front/csvlint.rb.svg)](https://coveralls.io/r/Data-Liberation-Front/csvlint.rb)
 [![License](http://img.shields.io/:license-mit-blue.svg)](http://theodi.mit-license.org)
 [![Badges](http://img.shields.io/:badges-5/5-ff6799.svg)](https://github.com/pikesley/badger)
 
@@ -10,14 +10,14 @@ A ruby gem to support validating CSV files to check their syntax and contents. Y
 
 ## Summary of features
 
-* Validation that checks the structural formatting of a CSV file  
+* Validation that checks the structural formatting of a CSV file
 * Validation of a delimiter-separated values (dsv) file accesible via URL, File, or an IO-style object (e.g. StringIO)
-* Validation against [CSV dialects](http://dataprotocols.org/csv-dialect/)  
-* Validation against multiple schema standards; [JSON Table Schema](https://github.com/theodi/csvlint.rb/blob/master/README.md#json-table-schema-support) and [CSV on the Web](https://github.com/theodi/csvlint.rb/blob/master/README.md#csv-on-the-web-validation-support) 
+* Validation against [CSV dialects](http://dataprotocols.org/csv-dialect/)
+* Validation against multiple schema standards; [JSON Table Schema](https://github.com/theodi/csvlint.rb/blob/master/README.md#json-table-schema-support) and [CSV on the Web](https://github.com/theodi/csvlint.rb/blob/master/README.md#csv-on-the-web-validation-support)
 
 ## Development
 
-`ruby version 2.1.4`
+`ruby version 3.2`
 
 ### Tests
 
@@ -71,7 +71,7 @@ After installing the gem, you can validate a CSV on the command line like so:
 
 	csvlint myfile.csv
 
-You may need to add the gem exectuable directory to your path, by adding '/usr/local/lib/ruby/gems/2.6.0/bin' 
+You may need to add the gem exectuable directory to your path, by adding '/usr/local/lib/ruby/gems/2.6.0/bin'
 or whatever your version is, to your .bash_profile PATH entry. [like so](https://stackoverflow.com/questions/2392293/ruby-gems-returns-command-not-found)
 
 You will then see the validation result, together with any warnings or errors e.g.
@@ -213,7 +213,7 @@ An example JSON Table Schema schema file is:
 				"name": "price",
 				"constraints": {
 					"required": true,
-					"minLength": 1 
+					"minLength": 1
 				}
 			},
 			{

data/Rakefile

@@ -1,12 +1,12 @@
 require "bundler/gem_tasks"
 
-$:.unshift File.join( File.dirname(__FILE__), "lib")
+$:.unshift File.join(File.dirname(__FILE__), "lib")
 
-require 'rubygems'
-require 'cucumber'
-require 'cucumber/rake/task'
-require 'coveralls/rake/task'
-require 'rspec/core/rake_task'
+require "rubygems"
+require "cucumber"
+require "cucumber/rake/task"
+require "coveralls/rake/task"
+require "rspec/core/rake_task"
 
 RSpec::Core::RakeTask.new(:spec)
 Coveralls::RakeTask.new
@@ -14,4 +14,4 @@ Cucumber::Rake::Task.new(:features) do |t|
   t.cucumber_opts = "features --format pretty"
 end
 
-task :default => [:spec, :features, 'coveralls:push']
+task default: [:spec, :features, "coveralls:push"]

data/csvlint.gemspec

@@ -1,24 +1,22 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'csvlint/version'
+require "csvlint/version"
 
 Gem::Specification.new do |spec|
-  spec.name          = "csvlint"
-  spec.version       = Csvlint::VERSION
-  spec.authors       = ["pezholio"]
-  spec.email         = ["pezholio@gmail.com"]
-  spec.description   = %q{CSV Validator}
-  spec.summary       = %q{CSV Validator}
-  spec.homepage      = "https://github.com/theodi/csvlint.rb"
-  spec.license       = "MIT"
+  spec.name = "csvlint"
+  spec.version = Csvlint::VERSION
+  spec.authors = ["pezholio"]
+  spec.email = ["pezholio@gmail.com"]
+  spec.description = "CSV Validator"
+  spec.summary = "CSV Validator"
+  spec.homepage = "https://github.com/theodi/csvlint.rb"
+  spec.license = "MIT"
 
-  spec.files         = `git ls-files`.split($/)
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.files = `git ls-files`.split($/)
+  spec.executables = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.required_ruby_version = ['>= 2.5', '< 3.2']
+  spec.required_ruby_version = [">= 2.5", "< 3.3"]
 
   spec.add_dependency "rainbow"
   spec.add_dependency "open_uri_redirections"
@@ -48,5 +46,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rdf", "< 4.0"
   spec.add_development_dependency "rdf-turtle"
   spec.add_development_dependency "henry"
-
+  spec.add_development_dependency "standardrb"
 end

data/docker_notes_for_windows.txt

@@ -0,0 +1,20 @@
+# Note that these commands are specific for a docker environment on MS Windows.
+
+# to generate Gemfile.lock file
+docker run --rm -v %CD%:/usr/src/app -w /usr/src/app ruby:2.5 bundle install
+
+# to build docker image from source (the ending dot is significant)
+docker build -t csvlint .
+
+# to run tests
+docker run -it --rm csvlint rake
+
+# to run csvlint command line with a CSV file.
+# cd to the directory with the CSV file then
+docker run -it --rm -v %CD%:/tmp csvlint ./bin/csvlint --dump-errors /tmp/file-to-lint.csv
+
+# to enter the linux container
+docker run -it --rm -v %CD%:/tmp csvlint bash
+
+# to enter the ruby REPL
+docker run -it --rm -v %CD%:/tmp csvlint irb

data/features/step_definitions/cli_steps.rb

@@ -1,37 +1,37 @@
-Given(/^I have stubbed stdin to contain "(.*?)"$/) do |file|
-  expect(STDIN).to receive(:read).and_return(File.read(file))
+Given(/^I have stubbed $stdin to contain "(.*?)"$/) do |file|
+  expect($stdin).to receive(:read).and_return(File.read(file))
 end
 
-Given(/^I have stubbed stdin to contain nothing$/) do
-  expect(STDIN).to receive(:read).and_return(nil)
+Given(/^I have stubbed $stdin to contain nothing$/) do
+  expect($stdin).to receive(:read).and_return(nil)
 end
 
 Then(/^nothing should be outputted to STDERR$/) do
-  expect(STDERR).to_not receive(:puts)
+  expect($stderr).to_not receive(:puts)
 end
 
 Then(/^the output should contain JSON$/) do
   @json = JSON.parse(all_stdout)
-  expect(@json['validation']).to be_present
+  expect(@json["validation"]).to be_present
 end
 
 Then(/^the JSON should have a state of "(.*?)"$/) do |state|
-  expect(@json['validation']['state']).to eq(state)
+  expect(@json["validation"]["state"]).to eq(state)
 end
 
 Then(/^the JSON should have (\d+) errors?$/) do |count|
   @index = count.to_i - 1
-  expect(@json['validation']['errors'].count).to eq(count.to_i)
+  expect(@json["validation"]["errors"].count).to eq(count.to_i)
 end
 
 Then(/^that error should have the "(.*?)" "(.*?)"$/) do |k, v|
-  expect(@json['validation']['errors'][@index][k].to_s).to eq(v)
+  expect(@json["validation"]["errors"][@index][k].to_s).to eq(v)
 end
 
 Then(/^error (\d+) should have the "(.*?)" "(.*?)"$/) do |index, k, v|
-  expect(@json['validation']['errors'][index.to_i - 1][k].to_s).to eq(v)
+  expect(@json["validation"]["errors"][index.to_i - 1][k].to_s).to eq(v)
 end
 
 Then(/^error (\d+) should have the constraint "(.*?)" "(.*?)"$/) do |index, k, v|
-  expect(@json['validation']['errors'][index.to_i - 1]['constraints'][k].to_s).to eq(v)
+  expect(@json["validation"]["errors"][index.to_i - 1]["constraints"][k].to_s).to eq(v)
 end

data/features/step_definitions/information_steps.rb

@@ -3,11 +3,11 @@ Given(/^the content type is "(.*?)"$/) do |arg1|
 end
 
 Then(/^the "(.*?)" should be "(.*?)"$/) do |type, encoding|
-  validator = Csvlint::Validator.new( @url, default_csv_options ) 
-  expect( validator.send(type.to_sym) ).to eq( encoding )
+  validator = Csvlint::Validator.new(@url, default_csv_options)
+  expect(validator.send(type.to_sym)).to eq(encoding)
 end
 
 Then(/^the metadata content type should be "(.*?)"$/) do |content_type|
-  validator = Csvlint::Validator.new( @url, default_csv_options ) 
-  expect( validator.headers['content-type'] ).to eq( content_type )
+  validator = Csvlint::Validator.new(@url, default_csv_options)
+  expect(validator.headers["content-type"]).to eq(content_type)
 end

data/features/step_definitions/parse_csv_steps.rb

@@ -12,31 +12,31 @@ Given(/^it is stored at the url "(.*?)"$/) do |url|
   charset = @encoding || "UTF-8"
   headers = {"Content-Type" => "#{content_type}; charset=#{charset}"}
   headers["Link"] = @link if @link
-  stub_request(:get, url).to_return(:status => 200, :body => @csv, :headers => headers)
-  stub_request(:get, URI.join(url, '/.well-known/csvm')).to_return(:status => 404)
-  stub_request(:get, url + '-metadata.json').to_return(:status => 404)
-  stub_request(:get, URI.join(url, 'csv-metadata.json')).to_return(:status => 404)
+  stub_request(:get, url).to_return(status: 200, body: @csv, headers: headers)
+  stub_request(:get, URI.join(url, "/.well-known/csvm")).to_return(status: 404)
+  stub_request(:get, url + "-metadata.json").to_return(status: 404)
+  stub_request(:get, URI.join(url, "csv-metadata.json")).to_return(status: 404)
 end
 
 Given(/^it is stored at the url "(.*?)" with no character set$/) do |url|
   @url = url
   content_type = @content_type || "text/csv"
-  stub_request(:get, url).to_return(:status => 200, :body => @csv, :headers => {"Content-Type" => "#{content_type}"})
-  stub_request(:get, URI.join(url, '/.well-known/csvm')).to_return(:status => 404)
-  stub_request(:get, url + '-metadata.json').to_return(:status => 404)
-  stub_request(:get, URI.join(url, 'csv-metadata.json')).to_return(:status => 404)
+  stub_request(:get, url).to_return(status: 200, body: @csv, headers: {"Content-Type" => content_type.to_s})
+  stub_request(:get, URI.join(url, "/.well-known/csvm")).to_return(status: 404)
+  stub_request(:get, url + "-metadata.json").to_return(status: 404)
+  stub_request(:get, URI.join(url, "csv-metadata.json")).to_return(status: 404)
 end
 
 When(/^I ask if the CSV is valid$/) do
   @csv_options ||= default_csv_options
-  @validator = Csvlint::Validator.new( @url, @csv_options )
+  @validator = Csvlint::Validator.new(@url, @csv_options)
   @valid = @validator.valid?
 end
 
 Then(/^I should get the value of true$/) do
-  expect( @valid ).to be(true)
+  expect(@valid).to be(true)
 end
 
 Then(/^I should get the value of false$/) do
-  expect( @valid ).to be(false)
+  expect(@valid).to be(false)
 end

data/features/step_definitions/schema_validation_steps.rb

@@ -10,24 +10,24 @@ end
 
 Given(/^I have a metadata file called "([^"]*)"$/) do |filename|
   @schema_type = :csvw_metadata
-  @schema_json = File.read( File.join( File.dirname(__FILE__), "..", "fixtures", filename ) )
+  @schema_json = File.read(File.join(File.dirname(__FILE__), "..", "fixtures", filename))
 end
 
-Given(/^the (schema|metadata) is stored at the url "(.*?)"$/) do |schema_type,schema_url|
+Given(/^the (schema|metadata) is stored at the url "(.*?)"$/) do |schema_type, schema_url|
   @schema_url = schema_url
-  stub_request(:get, @schema_url).to_return(:status => 200, :body => @schema_json.to_str)
+  stub_request(:get, @schema_url).to_return(status: 200, body: @schema_json.to_str)
 end
 
 Given(/^there is a file at "(.*?)" with the content:$/) do |url, content|
-  stub_request(:get, url).to_return(:status => 200, :body => content.to_str)
+  stub_request(:get, url).to_return(status: 200, body: content.to_str)
 end
 
-Given(/^I have a file called "(.*?)" at the url "(.*?)"$/) do |filename,url|
-  content = File.read( File.join( File.dirname(__FILE__), "..", "fixtures", filename ) )
-  content_type = filename =~ /.csv$/ ? "text/csv" : "application/csvm+json"
-  stub_request(:get, url).to_return(:status => 200, :body => content, :headers => {"Content-Type" => "#{content_type}; charset=UTF-8"})
+Given(/^I have a file called "(.*?)" at the url "(.*?)"$/) do |filename, url|
+  content = File.read(File.join(File.dirname(__FILE__), "..", "fixtures", filename))
+  content_type = /.csv$/.match?(filename) ? "text/csv" : "application/csvm+json"
+  stub_request(:get, url).to_return(status: 200, body: content, headers: {"Content-Type" => "#{content_type}; charset=UTF-8"})
 end
 
 Given(/^there is no file at the url "(.*?)"$/) do |url|
-  stub_request(:get, url).to_return(:status => 404)
-end
+  stub_request(:get, url).to_return(status: 404)
+end

data/features/step_definitions/sources_steps.rb

@@ -3,5 +3,5 @@ Given(/^it is parsed as a StringIO$/) do
 end
 
 Given(/^I parse a file called "(.*?)"$/) do |filename|
-  @url = File.new( File.join( File.dirname(__FILE__), "..", "fixtures", filename ) )
+  @url = File.new(File.join(File.dirname(__FILE__), "..", "fixtures", filename))
 end

data/features/step_definitions/validation_errors_steps.rb

@@ -2,14 +2,14 @@ When(/^I ask if there are errors$/) do
   @csv_options ||= default_csv_options
 
   if @schema_json
-    if @schema_type == :json_table
-      @schema = Csvlint::Schema.from_json_table( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+    @schema = if @schema_type == :json_table
+      Csvlint::Schema.from_json_table(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     else
-      @schema = Csvlint::Schema.from_csvw_metadata( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+      Csvlint::Schema.from_csvw_metadata(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     end
   end
 
-  @validator = Csvlint::Validator.new( @url, @csv_options, @schema )
+  @validator = Csvlint::Validator.new(@url, @csv_options, @schema)
   @errors = @validator.errors
 end
 
@@ -19,10 +19,10 @@ When(/^I carry out CSVW validation$/) do
   begin
     if @schema_json
       json = JSON.parse(@schema_json)
-      if @schema_type == :json_table
-        @schema = Csvlint::Schema.from_json_table( @schema_url || "http://example.org ", json )
+      @schema = if @schema_type == :json_table
+        Csvlint::Schema.from_json_table(@schema_url || "http://example.org ", json)
       else
-        @schema = Csvlint::Schema.from_csvw_metadata( @schema_url || "http://example.org ", json )
+        Csvlint::Schema.from_csvw_metadata(@schema_url || "http://example.org ", json)
       end
     end
 
@@ -30,12 +30,12 @@ When(/^I carry out CSVW validation$/) do
       @errors = []
       @warnings = []
       @schema.tables.keys.each do |table_url|
-        validator = Csvlint::Validator.new( table_url, @csv_options, @schema )
+        validator = Csvlint::Validator.new(table_url, @csv_options, @schema)
         @errors += validator.errors
         @warnings += validator.warnings
       end
     else
-      validator = Csvlint::Validator.new( @url, @csv_options, @schema )
+      validator = Csvlint::Validator.new(@url, @csv_options, @schema)
       @errors = validator.errors
       @warnings = validator.warnings
     end
@@ -49,42 +49,42 @@ end
 Then(/^there should be errors$/) do
   # this test is only used for CSVW testing; :invalid_encoding & :line_breaks mask lack of real errors
   @errors.delete_if { |e| e.instance_of?(Csvlint::ErrorMessage) && [:invalid_encoding, :line_breaks].include?(e.type) }
-  expect( @errors.count ).to be > 0
+  expect(@errors.count).to be > 0
 end
 
 Then(/^there should not be errors$/) do
-  expect( @errors.count ).to eq(0)
+  expect(@errors.count).to eq(0)
 end
 
 Then(/^there should be (\d+) error$/) do |count|
-  expect( @errors.count ).to eq( count.to_i )
+  expect(@errors.count).to eq(count.to_i)
 end
 
 Then(/^that error should have the type "(.*?)"$/) do |type|
-  expect( @errors.first.type ).to eq( type.to_sym )
+  expect(@errors.first.type).to eq(type.to_sym)
 end
 
 Then(/^that error should have the row "(.*?)"$/) do |row|
-  expect( @errors.first.row ).to eq( row.to_i )
+  expect(@errors.first.row).to eq(row.to_i)
 end
 
 Then(/^that error should have the column "(.*?)"$/) do |column|
-  expect( @errors.first.column ).to eq( column.to_i )
+  expect(@errors.first.column).to eq(column.to_i)
 end
 
 Then(/^that error should have the content "(.*)"$/) do |content|
-  expect( @errors.first.content.chomp ).to eq( content.chomp )
+  expect(@errors.first.content.chomp).to eq(content.chomp)
 end
 
 Then(/^that error should have no content$/) do
-  expect( @errors.first.content ).to eq( nil )
+  expect(@errors.first.content).to eq(nil)
 end
 
 Given(/^I have a CSV that doesn't exist$/) do
   @url = "http//www.example.com/fake-csv.csv"
-  stub_request(:get, @url).to_return(:status => 404)
+  stub_request(:get, @url).to_return(status: 404)
 end
 
 Then(/^there should be no "(.*?)" errors$/) do |type|
-  @errors.each do |error| error.type.should_not == type.to_sym end
+  @errors.each { |error| error.type.should_not == type.to_sym }
 end

data/features/step_definitions/validation_info_steps.rb

@@ -1,22 +1,22 @@
 Given(/^I ask if there are info messages$/) do
   @csv_options ||= default_csv_options
-  
+
   if @schema_json
-    if @schema_type == :json_table
-      @schema = Csvlint::Schema.from_json_table( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+    @schema = if @schema_type == :json_table
+      Csvlint::Schema.from_json_table(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     else
-      @schema = Csvlint::Schema.from_csvw_metadata( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+      Csvlint::Schema.from_csvw_metadata(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     end
   end
-  
-  @validator = Csvlint::Validator.new( @url, @csv_options, @schema ) 
+
+  @validator = Csvlint::Validator.new(@url, @csv_options, @schema)
   @info_messages = @validator.info_messages
 end
 
 Then(/^there should be (\d+) info messages?$/) do |num|
-  expect( @info_messages.count ).to eq( num.to_i )
+  expect(@info_messages.count).to eq(num.to_i)
 end
 
 Then(/^one of the messages should have the type "(.*?)"$/) do |msg_type|
-  expect( @info_messages.find{|x| x.type == msg_type.to_sym} ).to be_present
-end
+  expect(@info_messages.find { |x| x.type == msg_type.to_sym }).to be_present
+end

data/features/step_definitions/validation_warnings_steps.rb

@@ -12,35 +12,35 @@ Given(/^I do not set an encoding header$/) do
 end
 
 Given(/^I have a CSV file called "(.*?)"$/) do |filename|
-  @csv = File.read( File.join( File.dirname(__FILE__), "..", "fixtures", filename ) )
+  @csv = File.read(File.join(File.dirname(__FILE__), "..", "fixtures", filename))
 end
 
 When(/^I ask if there are warnings$/) do
   @csv_options ||= default_csv_options
   if @schema_json
-    if @schema_type == :json_table
-      @schema = Csvlint::Schema.from_json_table( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+    @schema = if @schema_type == :json_table
+      Csvlint::Schema.from_json_table(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     else
-      @schema = Csvlint::Schema.from_csvw_metadata( @schema_url || "http://example.org ", JSON.parse(@schema_json) )
+      Csvlint::Schema.from_csvw_metadata(@schema_url || "http://example.org ", JSON.parse(@schema_json))
     end
   end
 
-  @validator = Csvlint::Validator.new( @url, @csv_options, @schema )
+  @validator = Csvlint::Validator.new(@url, @csv_options, @schema)
   @warnings = @validator.warnings
 end
 
 Then(/^there should be warnings$/) do
-  expect( @warnings.count ).to be > 0
+  expect(@warnings.count).to be > 0
 end
 
 Then(/^there should not be warnings$/) do
   # this test is only used for CSVW testing, and :inconsistent_values warnings don't count in CSVW
   @warnings.delete_if { |w| [:inconsistent_values, :check_options].include?(w.type) }
-  expect( @warnings.count ).to eq(0)
+  expect(@warnings.count).to eq(0)
 end
 
 Then(/^there should be (\d+) warnings$/) do |count|
-  expect( @warnings.count ).to eq( count.to_i )
+  expect(@warnings.count).to eq(count.to_i)
 end
 
 Given(/^the content type is set to "(.*?)"$/) do |type|
@@ -48,13 +48,13 @@ Given(/^the content type is set to "(.*?)"$/) do |type|
 end
 
 Then(/^that warning should have the row "(.*?)"$/) do |row|
-  expect( @warnings.first.row ).to eq( row.to_i )
+  expect(@warnings.first.row).to eq(row.to_i)
 end
 
 Then(/^that warning should have the column "(.*?)"$/) do |column|
-  expect( @warnings.first.column ).to eq( column.to_i )
+  expect(@warnings.first.column).to eq(column.to_i)
 end
 
 Then(/^that warning should have the type "(.*?)"$/) do |type|
-  expect( @warnings.first.type ).to eq( type.to_sym )
+  expect(@warnings.first.type).to eq(type.to_sym)
 end