Coerce empty arrays to nil (#65)

data-axle · Dec 6, 2023 · 695f7b9 · 695f7b9
1 parent cfc163a
commit 695f7b9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/superstore/types/array_type.rb b/lib/superstore/types/array_type.rb
@@ -2,7 +2,7 @@ module Superstore
   module Types
     class ArrayType < Base
       def cast_value(value)
-        Array(value)
+        value.present? ? Array(value) : nil
       end
     end
   end

diff --git a/test/unit/types/array_type_test.rb b/test/unit/types/array_type_test.rb
@@ -4,5 +4,7 @@ class Superstore::Types::ArrayTypeTest < Superstore::Types::TestCase
   test 'cast_value' do
     assert_equal ['x', 'y'], type.cast_value(['x', 'y'].to_set)
     assert_equal ['x'], type.cast_value('x')
+    assert_nil type.cast_value(nil)
+    assert_nil type.cast_value([])
   end
 end