Fixed a bug where nil fields would raise an exception.

owenbyrne · Mar 6, 2015 · 524f48d · 524f48d
1 parent 0d5c8b7
commit 524f48d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/pikelet/field_definition.rb b/lib/pikelet/field_definition.rb
@@ -34,7 +34,8 @@ def initialize(index, parse: nil, type: :alpha, format: nil, pad: nil, align: ni
 
     def parse(record)
       # TODO: Test that fields are always stripped.
-      if value = record[index].strip
+      if value = record[index]
+        value.strip!
         if parser
           parser.to_proc.call(value)
         else

diff --git a/spec/pikelet/field_definition_spec.rb b/spec/pikelet/field_definition_spec.rb
@@ -16,7 +16,7 @@
 
     subject { definition }
 
-    its(:parser)    { is_expected.to eq :strip }
+    its(:parser)    { is_expected.to be_nil }
     its(:formatter) { is_expected.to eq :to_s }
     its(:padding)   { is_expected.to eq " " }
     its(:alignment) { is_expected.to eq :left }