Skip to content

Commit

Permalink
Fix some tests
Browse files Browse the repository at this point in the history
  • Loading branch information
fmoletta committed Apr 25, 2024
1 parent d1c7404 commit bd5b551
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
4 changes: 2 additions & 2 deletions vm/src/hint_processor/builtin_hint_processor/hint_utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ mod tests {

assert_matches!(
get_ptr_from_var_name("value", &vm, &ids_data, &ApTracking::new()),
Err(HintError::IdentifierNotRelocatable(bx)) if *bx == "value"
Err(HintError::IdentifierNotRelocatable(bx)) if *bx == "value".to_string()
);
}

Expand Down Expand Up @@ -270,7 +270,7 @@ mod tests {

assert_matches!(
get_integer_from_var_name("value", &vm, &ids_data, &ApTracking::new()),
Err(HintError::IdentifierNotInteger(bx)) if *bx == "value"
Err(HintError::IdentifierNotInteger(bx)) if *bx == "value".to_string()
);
}
}
11 changes: 8 additions & 3 deletions vm/src/hint_processor/hint_processor_utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -220,21 +220,26 @@ mod tests {
hint_ref.offset1 = OffsetValue::Reference(Register::FP, 2_i32, false);

assert_matches!(
get_offset_value_reference(&vm, &hint_ref, &ApTracking::new(), &hint_ref.offset1),
get_offset_value(&vm, &hint_ref.offset1, &hint_ref.ap_tracking_data, &ApTracking::new()),
Some(x) if x == mayberelocatable!(1, 2)
);
}

#[test]
#[cfg_attr(target_arch = "wasm32", wasm_bindgen_test)]
fn get_offset_value_reference_invalid() {
fn get_offset_value_invalid() {
let mut vm = vm!();
vm.segments = segments![((1, 0), 0)];
let mut hint_ref = HintReference::new(0, 0, false, true);
hint_ref.offset1 = OffsetValue::Reference(Register::FP, -2_i32, false);

assert_matches!(
get_offset_value_reference(&vm, &hint_ref, &ApTracking::new(), &hint_ref.offset1),
get_offset_value(
&vm,
&hint_ref.offset1,
&hint_ref.ap_tracking_data,
&ApTracking::new()
),
None
);
}
Expand Down

0 comments on commit bd5b551

Please sign in to comment.