feat(makefile,ci): add cairo_test_suite_programs target and CI integration#2380
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Benchmark Results for unmodified programs 🚀
|
cf572eb to
646dd9e
Compare
421425b to
2b921a1
Compare
646dd9e to
2bc600b
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## naor/feat/add_test_helpers #2380 +/- ##
===========================================================
Coverage 96.07% 96.07%
===========================================================
Files 107 107
Lines 37852 37852
===========================================================
Hits 36366 36366
Misses 1486 1486 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
2bc600b to
6fd8964
Compare
6ec5328 to
ce3aeb1
Compare
6fd8964 to
c106a57
Compare
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
106cbb4 to
ff56720
Compare
OmriEshhar1
left a comment
There was a problem hiding this comment.
@OmriEshhar1 reviewed 3 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on naor-starkware).
e97f3a8 to
3e8d6ce
Compare
6c03214 to
d11f204
Compare
3e8d6ce to
5e2dd11
Compare
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
d11f204 to
b2799c2
Compare
YairVaknin-starkware
left a comment
There was a problem hiding this comment.
@YairVaknin-starkware reviewed 3 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on naor-starkware).
…ation Makefile: - Add CAIRO_TEST_SUITE_ROOT, CAIRO_TEST_SUITE_FILES, COMPILED_CAIRO_TEST_SUITE variables to track cairo test suite sources - Add pattern rule to compile each .cairo to .json via cairo-compile - Add cairo_test_suite_programs target - Hook cairo_test_suite_programs into the test target - Add cairo_test_suite_programs to .PHONY CI (rust.yml): - Add cairo_test_suite_programs to the build-programs matrix - Add vm/src/tests/cairo_test_suite/**/*.json to CAIRO_PROGRAMS_PATH cache - Update all program cache keys to hash test suite source files - Add function_runner feature flag to the tests job Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
b2799c2 to
f0bb2d0
Compare

TITLE
Description
Description of the pull request changes and motivation.
Checklist
This change is