Remove redundant assert

red-data-tools · Jan 19, 2025 · 56a0ab1 · 56a0ab1
1 parent e7e8fa5
commit 56a0ab1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/red_amber/data_frame_variable_operation.rb b/lib/red_amber/data_frame_variable_operation.rb
@@ -694,7 +694,11 @@ def append_to_fields_and_arrays(appender, fields, arrays, append_to_left)
           raise DataFrameArgumentError, "Data size mismatch (#{data.size} != #{size})"
         end
 
-        a = Arrow::Array.new(data.is_a?(Vector) ? data.to_a : data)
+        if data.is_a?(Vector)
+          a = data.data
+        else
+          a = Arrow::Array.new(data)
+        end
 
         if append_to_left
           fields.unshift(Arrow::Field.new(key.to_sym, a.value_data_type))

diff --git a/test/test_data_frame_variable_operation.rb b/test/test_data_frame_variable_operation.rb
@@ -441,7 +441,6 @@ class DataFrameVariableOperationTest < Test::Unit::TestCase
 
     sub_test_case 'Dataframe with zero n_records' do
       test 'assign by block' do
-        assert_equal :double, @df.b.type
         str = <<~STR
           RedAmber::DataFrame : 0 x 4 Vectors
           Vectors : 2 numeric, 1 string, 1 boolean