-
Notifications
You must be signed in to change notification settings - Fork 134
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
Voting Application B VM logic tests #361
Conversation
…encingEvents"" This reverts commit 23fc757.
@@ -1,17 +1,36 @@ | |||
import { Experimental, Mina, AccountUpdate } from 'snarkyjs'; | |||
import { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this test script be run as part of ./run-integration-tests.sh
, so that voting tested in CI?
(And does this mean the voting/run.ts
script doesn't need to run in CI anymore? Sorry I haven't really looked at either of these yet)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
run.ts
sets the preconditions and parameters for the tests we want to run and then calls into test.ts
to execute all the tests, so it's being run although maybe a bit confusing :D
/** | ||
* Function used to upgrade a Membership contract with slightly modified methods, in order to manipulate the existing verification key | ||
*/ | ||
/* export async function upgradeMembershipContract( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this intentionally commented out? Can you delete it in that case? Are we testing this somewhere else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Woo 🥳
This PR includes a wide range of different tests for the voting application. See the testing app b doc for reference
proofsEnabled: boolean
flag when creating child account updates