Skip to content

Commit 154ada0

Browse files
committed
chore: use vx array assert eq macro
Signed-off-by: Joe Isaacs <[email protected]>
1 parent 668e2d6 commit 154ada0

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

encodings/fastlanes/src/rle/compress.rs

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -214,23 +214,23 @@ mod test {
214214
fn test_encode_decode() {
215215
// u8
216216
let values_u8: Buffer<u8> = [1, 1, 2, 2, 3, 3].iter().copied().collect();
217-
let array_u8 = values_u8.clone().into_array();
217+
let array_u8 = values_u8.into_array();
218218
let encoded_u8 = RLEArray::encode(&array_u8.to_primitive()).unwrap();
219219
let decoded_u8 = encoded_u8.to_primitive();
220220
let expected_u8 = PrimitiveArray::from_iter(vec![1u8, 1, 2, 2, 3, 3]);
221221
assert_arrays_eq!(decoded_u8, expected_u8);
222222

223223
// u16
224224
let values_u16: Buffer<u16> = [100, 100, 200, 200].iter().copied().collect();
225-
let array_u16 = values_u16.clone().into_array();
225+
let array_u16 = values_u16.into_array();
226226
let encoded_u16 = RLEArray::encode(&array_u16.to_primitive()).unwrap();
227227
let decoded_u16 = encoded_u16.to_primitive();
228228
let expected_u16 = PrimitiveArray::from_iter(vec![100u16, 100, 200, 200]);
229229
assert_arrays_eq!(decoded_u16, expected_u16);
230230

231231
// u64
232232
let values_u64: Buffer<u64> = [1000, 1000, 2000].iter().copied().collect();
233-
let array_u64 = values_u64.clone().into_array();
233+
let array_u64 = values_u64.into_array();
234234
let encoded_u64 = RLEArray::encode(&array_u64.to_primitive()).unwrap();
235235
let decoded_u64 = encoded_u64.to_primitive();
236236
let expected_u64 = PrimitiveArray::from_iter(vec![1000u64, 1000, 2000]);
@@ -258,7 +258,7 @@ mod test {
258258
#[test]
259259
fn test_single_value() {
260260
let values: Buffer<u16> = vec![42; 2000].into_iter().collect();
261-
let array = values.clone().into_array();
261+
let array = values.into_array();
262262

263263
let encoded = RLEArray::encode(&array.to_primitive()).unwrap();
264264
assert_eq!(encoded.values.len(), 2); // 2 chunks, each storing value 42
@@ -271,7 +271,7 @@ mod test {
271271
#[test]
272272
fn test_all_different() {
273273
let values: Buffer<u8> = (0u8..=255).collect();
274-
let array = values.clone().into_array();
274+
let array = values.into_array();
275275

276276
let encoded = RLEArray::encode(&array.to_primitive()).unwrap();
277277
assert_eq!(encoded.values.len(), 256);
@@ -285,14 +285,13 @@ mod test {
285285
fn test_partial_last_chunk() {
286286
// Test array with partial last chunk (not divisible by 1024)
287287
let values: Buffer<u32> = (0..1500).map(|i| (i / 100) as u32).collect();
288-
let array = values.clone().into_array();
288+
let array = values.into_array();
289289

290290
let encoded = RLEArray::encode(&array.to_primitive()).unwrap();
291291
let decoded = encoded.to_primitive();
292292

293293
assert_eq!(encoded.len(), 1500);
294-
let expected_array = PrimitiveArray::from_iter(expected);
295-
assert_arrays_eq!(decoded, expected_array);
294+
assert_arrays_eq!(decoded, array);
296295
// 2 chunks: 1024 + 476 elements
297296
assert_eq!(encoded.values_idx_offsets().len(), 2);
298297
}
@@ -301,14 +300,13 @@ mod test {
301300
fn test_two_full_chunks() {
302301
// Array that spans exactly 2 chunks (2048 elements)
303302
let values: Buffer<u32> = (0..2048).map(|i| (i / 100) as u32).collect();
304-
let array = values.clone().into_array();
303+
let array = values.into_array();
305304

306305
let encoded = RLEArray::encode(&array.to_primitive()).unwrap();
307306
let decoded = encoded.to_primitive();
308307

309308
assert_eq!(encoded.len(), 2048);
310-
let expected_array = PrimitiveArray::from_iter(expected);
311-
assert_arrays_eq!(decoded, expected_array);
309+
assert_arrays_eq!(decoded, array);
312310
assert_eq!(encoded.values_idx_offsets().len(), 2);
313311
}
314312

0 commit comments

Comments
 (0)