Skip to content

Commit 539288e

Browse files
authored
Fuzzer uses same mask for expected and actual value (#7133)
I think this is a typo that got in https://github.com/vortex-data/vortex/pull/6867/changes#r2978043644. This fixes latest fuzzer issue. Signed-off-by: Robert Kruszewski <github@robertk.io> Signed-off-by: Robert Kruszewski <github@robertk.io>
1 parent 7b09ee5 commit 539288e

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

fuzz/src/array/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ impl<'a> Arbitrary<'a> for FuzzArrayAction {
398398
current_array
399399
.to_canonical()
400400
.vortex_expect("to_canonical should succeed in fuzz test"),
401-
&!Mask::from_iter(mask.clone()),
401+
&Mask::from_iter(mask.clone()),
402402
)
403403
.vortex_expect("mask_canonical_array should succeed in fuzz test");
404404
// Update current_array to the result for chaining

fuzz/src/error.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,10 +114,10 @@ impl Display for VortexFuzzError {
114114
f,
115115
"Mismatch at step {step} at index {idx}\n\
116116
Expected scalar:\n{expected_scalar}\n\
117-
Actual scalar:\n{actual_scalar}\n\
117+
Actual scalar:\n{actual_scalar}\n\n\
118118
Expected tree:\n{expected_tree}\n\
119-
Current tree:\n{current_tree}\
120-
Expected values:\n{expected_values:#}\n\
119+
Current tree:\n{current_tree}\n\
120+
Expected values:\n{expected_values:#}\n\n\
121121
Current values:\n{current_values:#}\
122122
\n{backtrace}"
123123
)

0 commit comments

Comments
 (0)