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

Define native function of Test contract in Cadence #2822

Open
SupunS opened this issue Sep 27, 2023 · 3 comments
Open

Define native function of Test contract in Cadence #2822

SupunS opened this issue Sep 27, 2023 · 3 comments

Comments

@SupunS
Copy link
Member

SupunS commented Sep 27, 2023

Issue to be solved

As discussed in #2783 (comment), once native function support is available in the checker (#2821), define the native functions that are currently being 'injected' to the test contract, also in Cadence.

Suggested Solution

No response

@j1010001
Copy link
Member

@m-Peter is this still blocking you ?

@m-Peter
Copy link
Contributor

m-Peter commented Oct 23, 2023

Hey @j1010001, this is not blocking the current version of the testing framework we are about to release.
This is merely a technical debt, for a workaround that is already merged.
I am tackling the implementation of this in #2823, and it is almost done.
I have come across a limitation on defining optional arguments for functions in user-code: #2823 (comment). But we can possibly fix this by changing the API of these 2 methods.

@j1010001
Copy link
Member

j1010001 commented Jan 5, 2024

on hold until EVM is done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants