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

PoS transaction tests #22

Closed
tzemanovic opened this issue Sep 13, 2021 · 0 comments · Fixed by #254
Closed

PoS transaction tests #22

tzemanovic opened this issue Sep 13, 2021 · 0 comments · Fixed by #254
Assignees
@tzemanovic @junkicide
Labels
ledger PoS prio:high testing wasm

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Sep 13, 2021 •
edited
Loading

Add tests for the PoS transaction WASMs using the tests crate, which provides access to tx host env and wasm tx_prelude.

With the tests, we can remove or replace the log_string calls in tx_bond, tx_unbond and tx_withdraw with a debug_log!, which will be compiled away in release built WASM.

┆Issue is synchronized with this Asana task by Unito
@tzemanovic tzemanovic mentioned this issue Sep 13, 2021
Merged
@tzemanovic tzemanovic added the PoS label Sep 13, 2021
@tzemanovic tzemanovic mentioned this issue Nov 8, 2021
Closed
@sync-by-unito sync-by-unito bot closed this as completed Feb 3, 2022
@juped juped reopened this Feb 3, 2022
@tzemanovic tzemanovic mentioned this issue May 6, 2022
Closed
@tzemanovic tzemanovic mentioned this issue Jun 8, 2022
Closed
tzemanovic referenced this issue in anoma/anoma Jun 8, 2022
@tzemanovic
Changelog: add #462
be8b186
tzemanovic referenced this issue in anoma/anoma Jun 8, 2022
@tzemanovic
Changelog: add #462
8d3048d
tzemanovic referenced this issue in anoma/anoma Jun 8, 2022
@tzemanovic
Changelog: add #462
a2075da
tzemanovic referenced this issue in anoma/anoma Jul 1, 2022
@tzemanovic
Changelog: add #462
48471c8
@tzemanovic tzemanovic transferred this issue from anoma/anoma Jul 7, 2022
@tzemanovic tzemanovic mentioned this issue Aug 2, 2022
Merged
Repository owner moved this from Pending Devnet to Tested in Devnet in Namada-Old Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tzemanovic tzemanovic

@junkicide junkicide

Labels
ledger PoS prio:high testing wasm
Projects
No open projects
Namada-Old
Status: Tested in Devnet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

PoS tx tests anoma/namada
PoS tx tests anoma/anoma
3 participants
@juped @tzemanovic @junkicide