fix : Fix CairoRunner::get_memory_holes (#1027)

* Add test

* Ignore builtin segments in get_memory_holes

* Improve test

* Add memory holes check for cairo_run_test tests

* Fix broken condition

* Make memory_holes check optional & customizable

* Fix bounds

* Add a test with deliberately created memory holes

* Fix test value

* Remove duplicated tests + add memory hole value to some tests

* Add memory holes value + remove duplicated tests + fix typo

* Fix test values

* Add changelog entry

* Link PR in Changelog

* Mark breaking change in changelog

* fmt

* Fix test value

* Fix codecov-patch diff

Author: fmoletta

CHANGELOG.md

@@ -2,6 +2,13 @@
 
 #### Upcoming Changes
 
+* BREAKING CHANGE: Fix `CairoRunner::get_memory_holes` [#1027](https://github.com/lambdaclass/cairo-rs/pull/1027):
+
+  * Skip builtin segements when counting memory holes
+  * Check amount of memory holes for all tests in cairo_run_test
+  * Remove duplicated tests in cairo_run_test
+  * BREAKING CHANGE: `MemorySegmentManager.get_memory_holes` now also receives the amount of builtins in the vm. Signature is now `pub fn get_memory_holes(&self, builtin_count: usize) -> Result<usize, MemoryError>`
+
 * Add missing hint on uint256_improvements lib [#1025](https://github.com/lambdaclass/cairo-rs/pull/1025):
 
     `BuiltinHintProcessor` now supports the following hint:

cairo_programs/memory_holes.cairo

@@ -0,0 +1,5 @@
+func main() {
+    // Deliberately create memory holes
+    assert [ap + 5] = 1;
+    return ();
+}