Allow nil datetimes as datetime fields (#52)

Beyond-Finance · May 18, 2023 · e3baa04 · e3baa04
1 parent 411f0d6
commit e3baa04
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/lib/active_force/field.rb b/lib/active_force/field.rb
@@ -12,7 +12,7 @@ def initialize name, options = {}
     def value_for_hash value
       case as
       when :datetime
-        value.to_fs(:iso8601)
+        value&.to_fs(:iso8601)
       else
         value
       end

diff --git a/spec/active_force/field_spec.rb b/spec/active_force/field_spec.rb
@@ -24,5 +24,11 @@
       names = field.new(:time, as: :datetime)
       expect(names.value_for_hash current_time).to eq current_time.to_fs(:iso8601)
     end
+
+    it 'a datetime field whose value is nil' do
+      current_time = nil
+      names = field.new(:time, as: :datetime)
+      expect(names.value_for_hash current_time).to be_nil
+    end
   end
 end