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

feat: pse evm tests #48

Merged
merged 25 commits into from
Aug 10, 2023
Merged

feat: pse evm tests #48

merged 25 commits into from
Aug 10, 2023

Conversation

jp4g
Copy link
Collaborator

@jp4g jp4g commented Aug 1, 2023

Description

Adds simulated EVM unit tests to prove nargo codegen-verifier generated contracts work

  • updates to acvm 0.18.1 which is needed because we added an upstream fix to allow circuit in eth_contract_from_vk to get public inputs
  • uses new eth_contract_from_vk circuit API to programmatically set number of public inputs

Other notes:

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@jp4g jp4g added the enhancement New feature or request label Aug 1, 2023
@jp4g jp4g changed the title Feat/pse evm tests feat: pse evm tests Aug 2, 2023
@Ethan-000 Ethan-000 merged commit 6374c93 into master Aug 10, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants