Skip to content

Commit 19403db

Browse files
authored
Update FSL take test to fail with assert_arrays_eq! without explicit ToCanonical (#7246)
Signed-off-by: Robert Kruszewski <github@robertk.io>
1 parent e3b7310 commit 19403db

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

  • vortex-array/src/arrays/fixed_size_list/tests

vortex-array/src/arrays/fixed_size_list/tests/take.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ use super::common::create_single_element_fsl;
1212
use crate::ArrayRef;
1313
use crate::DynArray;
1414
use crate::IntoArray;
15-
use crate::ToCanonical;
1615
use crate::arrays::FixedSizeListArray;
1716
use crate::arrays::PrimitiveArray;
1817
use crate::assert_arrays_eq;
@@ -161,8 +160,8 @@ fn test_element_index_overflow(
161160
#[case] indices: ArrayRef,
162161
#[case] expected: FixedSizeListArray,
163162
) {
164-
let result = fsl.take(indices).unwrap().to_fixed_size_list();
165-
assert_arrays_eq!(expected, result);
163+
let result = fsl.take(indices).unwrap();
164+
assert_arrays_eq!(result, expected);
166165
}
167166

168167
// Parameterized test for nullable array scenarios that are specific to FSL's implementation.

0 commit comments

Comments
 (0)