Open csgui opened 1 month ago
stx-transfer-memo?
and stx-transfer
both use the exact same stdlib.transfer-memo
wasm function. Current unit tests already cover the unhappy paths.
Just noticed that. Thanks for pointing @Acaccia ! 👍
https://github.com/stacks-network/clarity-wasm/blob/main/clar2wasm/src/words/stx.rs#L75
https://github.com/stacks-network/clarity-wasm/blob/main/clar2wasm/src/words/stx.rs#L105
Closing this issue as it is no longer necessary.
My comment was not necessarily about this issue being unnecessary, but it was more the fact that the current tests could be copied for stx-transfer-memo?
.
I think it's important to add them in case we refactor those functions later, so I'll reopen this issue.
I think it's important to add them in case we refactor those functions later, so I'll reopen this issue.
Oh, I see! I misunderstood your comment. Thanks!
Cover unhappy path of stx-transfer-memo? function with unit tests.
https://github.com/stacks-network/clarity-wasm/blob/main/clar2wasm/src/words/stx.rs#L131