Merge pull request #68 from theodi/include-col-name-in-message

include column name in :header_name message
Data-Liberation-Front · Feb 18, 2014 · d961b3e · d961b3e
2 parents 0092995 + 53b174a
commit d961b3e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/csvlint/schema.rb b/lib/csvlint/schema.rb
@@ -19,7 +19,7 @@ def initialize(uri, fields=[], title=nil, description=nil)
     def validate_header(header)
       reset
       header.each_with_index do |name,i|
-        build_warnings(:header_name, :schema, nil, i+1) if fields[i].name != name
+        build_warnings(:header_name, :schema, nil, i+1, name) if fields[i].name != name
       end
       return valid?
     end

diff --git a/spec/schema_spec.rb b/spec/schema_spec.rb
@@ -86,6 +86,7 @@
       expect( schema.validate_header(["wrong", "required"]) ).to eql(true)
       expect( schema.warnings.size ).to eql(1)
       expect( schema.warnings.first.type).to eql(:header_name)
+      expect( schema.warnings.first.content).to eql("wrong")
       expect( schema.warnings.first.category).to eql(:schema)
 
       expect( schema.validate_header(["minimum", "Required"]) ).to eql(true)