Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Being able to pass arguments to the executed entrypoint #38

Open
tdelabro opened this issue Oct 31, 2022 · 1 comment
Open

Being able to pass arguments to the executed entrypoint #38

tdelabro opened this issue Oct 31, 2022 · 1 comment
Labels
Context: intricated deep understanding of the codebase required Difficulty: hard require extensive knowledge Duration: 1 day will take about one day Techno: cairo cairo Techno: rust rust Type: feature a new feature to implement

Comments

@tdelabro
Copy link
Collaborator

tdelabro commented Oct 31, 2022

It is a typical pattern in testing frameworks to be able to define a test function that takes arguments and returns a value, and then to use a case feature to call this test multiple times with different inputs and expected output.

In the current state of our tool, tests are defined as Cairo functions and called through the cairo_run function as entry points.

Problem: cairo-rs is built to accept one canonical entry point, the main function, which does not take any argument.

I think we can modify our fork of cairo-rs so it will init some memory for the arguments before entering the test function

@tdelabro tdelabro added Type: feature a new feature to implement Context: intricated deep understanding of the codebase required Difficulty: hard require extensive knowledge Techno: cairo cairo Techno: rust rust Duration: 1 day will take about one day labels Nov 4, 2022
@Eikix Eikix moved this to Todo in Cairo foundry Nov 23, 2022
@bacharif
Copy link
Contributor

I pick.

@tdelabro tdelabro assigned bacharif and unassigned bacharif Jan 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Context: intricated deep understanding of the codebase required Difficulty: hard require extensive knowledge Duration: 1 day will take about one day Techno: cairo cairo Techno: rust rust Type: feature a new feature to implement
Projects
None yet
Development

No branches or pull requests

2 participants