Skip to content

Commit

Permalink
fix segment_arena tests to take into account that we no longer subtra…
Browse files Browse the repository at this point in the history
…ct INITIAL_SEGMENT_SIZE
  • Loading branch information
juanbono committed Apr 18, 2024
1 parent 809970e commit 6b5290c
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions vm/src/vm/runners/builtin_runner/segment_arena.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ mod tests {
let mut vm = vm!();
vm.segments.segment_used_sizes = Some(vec![3]);

assert_eq!(builtin.get_used_instances(&vm.segments), Ok(0));
assert_eq!(builtin.get_used_instances(&vm.segments), Ok(1));
}

#[test]
Expand All @@ -130,7 +130,7 @@ mod tests {
let mut vm = vm!();
vm.segments.segment_used_sizes = Some(vec![3]);

assert_eq!(builtin.get_used_instances(&vm.segments), Ok(0));
assert_eq!(builtin.get_used_instances(&vm.segments), Ok(1));
}

#[test]
Expand All @@ -154,7 +154,7 @@ mod tests {
builtin.final_stack(&vm.segments, pointer),
Err(RunnerError::InvalidStopPointer(Box::new((
SEGMENT_ARENA_BUILTIN_NAME,
relocatable!(0, 3),
relocatable!(0, 6),
relocatable!(0, 0)
))))
);
Expand Down Expand Up @@ -245,7 +245,7 @@ mod tests {

assert_eq!(
builtin.get_used_cells_and_allocated_size(&vm),
Ok((0_usize, 0))
Ok((3_usize, 3))
);
}

Expand Down Expand Up @@ -303,7 +303,7 @@ mod tests {
let mut vm = vm!();

vm.segments.segment_used_sizes = Some(vec![4]);
assert_eq!(builtin.get_used_cells(&vm.segments), Ok(1));
assert_eq!(builtin.get_used_cells(&vm.segments), Ok(1 + 3));
}

#[test]
Expand All @@ -325,7 +325,7 @@ mod tests {
let mut memory_segment_manager = MemorySegmentManager::new();
memory_segment_manager.segment_used_sizes = Some(vec![6]);
// SIZE(6) / CELLS_PER_INSTANCE(3)
assert_eq!(builtin.get_used_instances(&memory_segment_manager), Ok(1));
assert_eq!(builtin.get_used_instances(&memory_segment_manager), Ok(2));
}

#[test]
Expand Down

0 comments on commit 6b5290c

Please sign in to comment.