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

Verify transfer lib more thoroughly #342

Closed

QGarchery opened this issue Aug 17, 2023 · 1 comment

Assignees

Labels

verif

Contributor

QGarchery commented Aug 17, 2023

I also wonder if we want some small unit tests, like:

safeTransfer and safeTransferFrom revert if balance not enough.
safeTransferFrom reverts if not authorized.
safeTransfer does not revert if balance is enough.

Originally posted by @jhoenicke in #321 (review)

QGarchery self-assigned this

QGarchery mentioned this issue

[Certora] Dispatch tokens and verify safe transfer lib #321

Merged

QGarchery added the verif label

QGarchery mentioned this issue

[Certora] More thorough verification #346

Merged

QGarchery changed the title ~~Verify transfer lib more thouroughly~~ Verify transfer lib more thoroughly

Contributor Author

QGarchery commented Aug 17, 2023

Done in #346

QGarchery closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment