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

Fixes zemu tests #111

Merged
merged 8 commits into from
Jul 27, 2024
Merged

Fixes zemu tests #111

merged 8 commits into from
Jul 27, 2024

Conversation

neithanmo
Copy link
Contributor

@neithanmo neithanmo commented Jul 26, 2024

changes:

  • Update test vector addresses for sapling senders and outputs
  • Update snapshots
  • feat: Update builder tests in rust to use new addresses fix transactions and add new production makefile rule
  • Update deps in zcashtools package to use ledger-zcash-rs/dev

🔗 zboto Link

Update snapshots

feat: Update builder tests in rust to use new addresses
fix transactions and add new production makefile rule

Update tests and snapshots
Update deps in zcashtools package
@neithanmo neithanmo requested a review from emmanuelm41 July 26, 2024 16:25
@emmanuelm41 emmanuelm41 mentioned this pull request Jul 26, 2024
Increate timeout

Update snapshots

Fix approval for stax target
@emmanuelm41 emmanuelm41 merged commit 0c502e7 into dev Jul 27, 2024
34 checks passed
@emmanuelm41 emmanuelm41 deleted the fix/test_vectors branch July 27, 2024 03:29
emmanuelm41 added a commit that referenced this pull request Jul 27, 2024
* squash dev

* Fixes zemu tests (#111)

* fix: Update test vector addresses for sapling senders and outputs
Update snapshots

feat: Update builder tests in rust to use new addresses
fix transactions and add new production makefile rule

Update tests and snapshots
Update deps in zcashtools package

* Enable all targets

* feat: change app name (#112)

* Update snapshots

Increate timeout

Update snapshots

Fix approval for stax target

* Run test concurrently

* feat: update app version and snapshots

* feat: update a few more snapshots

* feat: update a few more snapshots

---------

Co-authored-by: Emmanuel <[email protected]>

---------

Co-authored-by: Juan Leni <[email protected]>
Co-authored-by: Natanael Mojica <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants