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.

Sign up for GitHub

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

Jump to bottom

Snapps integration test waits for ledger proof #10259

Merged
merged 25 commits into from
Mar 22, 2022
Merged

Snapps integration test waits for ledger proof #10259

merged 25 commits into from
Mar 22, 2022

Conversation

imeckler
Copy link
Member

@imeckler imeckler commented Feb 16, 2022 •
edited by deepthiskumar
Loading

This is for #10067 (comment) -- basically it updates the snapps integration test to wait for 1 ledger proofs to be produced

  • fixed a snark worker bug
  • a few fixes to reduce the time it takes for the test to complete

@imeckler imeckler requested a review from a team as a code owner February 16, 2022 23:15
@imeckler imeckler added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Feb 16, 2022
Izaak Meckler and others added 22 commits February 16, 2022 18:01
thread config thru
d64f17b
use min fee
2788468
try changing devnet config
3cac4ea
add some logging
db4b7ca
non-zero num_snark_workers
451def1
Merge remote-tracking branch 'origin/develop' into snapps-integration…
26ac08e 
…-test
try to fix snapps test
d2d02d3
@mrmr1993
Merge branch 'develop' into snapps-integration-test
4bf0de4
add snark work debugger
f591941
add hd_party function to parties
c7d0fb2
make patterns more explicit in transaction_snark to ensure correctness
4335960
make snapps test require only one proof
b1ecab3
Merge branch 'snapps-integration-test' of github.com:MinaProtocol/min…
f4fa2c3 
…a into snapps-integration-test
zkapps test fix
00a1db9
Merge remote-tracking branch 'origin/develop' into snapps-integration…
76f10cd 
…-test
@deepthiskumar
reduce snark work fee for transactions to be included
392c1ba
@deepthiskumar
Merge remote-tracking branch 'origin/develop' into snapps-integration…
dcdc2c8 
…-test
@deepthiskumar
use runtime config to set scan state size
2f295bc
@deepthiskumar
zkapps test improvements
ec81647
@deepthiskumar
Merge remote-tracking branch 'origin/develop' into snapps-integration…
932f38d 
…-test
@deepthiskumar
update nonce
cdb9b77
@deepthiskumar
no need to unlock accounts for zkApp txns in intg tests
f73e15c
@deepthiskumar deepthiskumar changed the title WIP: Snapps integration test waits for ledger proof Snapps integration test waits for ledger proof Mar 22, 2022
@deepthiskumar deepthiskumar merged commit d20d88c into develop Mar 22, 2022
@deepthiskumar deepthiskumar deleted the snapps-integration-test branch March 22, 2022 20:38
@deepthiskumar deepthiskumar mentioned this pull request Mar 23, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@QuiteStochastic QuiteStochastic QuiteStochastic approved these changes

@deepthiskumar deepthiskumar deepthiskumar approved these changes

Assignees
No one assigned
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@imeckler @QuiteStochastic @deepthiskumar @mrmr1993