test: Tests for deposit_vault #496
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
akhercha
changed the title
sparqet
approved these changes
Oct 5, 2023
StarkFishinator
pushed a commit
to StarkFishinator/satoru
that referenced
this pull request
Oct 5, 2023
* test(deposit_vault_contract): Can start doing some unit tests from that * test(deposit_vault_contract): Better format * test(deposit_vault_contract): All test passes! 🥳 * test(deposit_vault_contract): Added unit test for transfer_out not enough balance --------- Co-authored-by: akhercha <akherchache@pm.me> Co-authored-by: sparqet <37338401+sparqet@users.noreply.github.com>
zarboq
added a commit
that referenced
this pull request
Nov 5, 2023
* exp function, no tested (failed setting up runner) * not working :( * error comes from 1e18? * exp2 working git add . * exp is working ! * fmt * Feat: Implement the function in the market_utils library #3 (#463) * 10 functions done * almost finished, debug next * debug time * debuging * pushing recent changes/ still bug because missing functions * debuging finished * adding comments on functions * almost clean * Emit bug * programm compile 🎉 * resolving last test * All test passed * resolve request * 1 test failed because of max swap path lenght exceed test * resolving failed test * resolve * solving * compilation resolved --------- Co-authored-by: Michel <105498726+Sk8erboi84@users.noreply.github.com> * test: Tests for deposit_vault (#496) * test(deposit_vault_contract): Can start doing some unit tests from that * test(deposit_vault_contract): Better format * test(deposit_vault_contract): All test passes! 🥳 * test(deposit_vault_contract): Added unit test for transfer_out not enough balance --------- Co-authored-by: akhercha <akherchache@pm.me> Co-authored-by: sparqet <37338401+sparqet@users.noreply.github.com> * test: Improve tests for referral_storage contract (#499) * test(referral_storage_contract): Refacto test architecture before adding tests * test(referral_storage_contract): Quick refacto of base tests * test(referral_storage_contract): Added more tests * test(referral_storage_contract): Finished tests for referral_storage contract * test(referral_storage_contract): Removed custom teardown * test(referral_storage_contract): Unused imports * test(referral_storage_contract): Updated top comment * test(referral_storage_contract): from review; added init check in Governable * test(referral_storage_contract): Removed useless line * test(referral_storage_contract): Removed useless line x2 💀 --------- Co-authored-by: akhercha <akherchache@pm.me> * test: Improve tests of referral_utils library. #483 (#498) * update referalla-util tests * allow changing refferal * remove role_store --------- Co-authored-by: zarboq <37303126+zarboq@users.noreply.github.com> * Implementing StrictBank functions and tests (#426) * strict bank start * strict bank contract and tests * All strict bank tests running * formatted --------- Co-authored-by: Michel <105498726+Sk8erboi84@users.noreply.github.com> Co-authored-by: sparqet <37338401+sparqet@users.noreply.github.com> Co-authored-by: zarboq <37303126+zarboq@users.noreply.github.com> * ✨ execute_deposit_utils functions (#449) * ✨ execute_deposit_utils fn * fix: clone fees and rm extra line * fix: prevent BadMergeBaseMismatch by adding else to if- * fix: fmt issue * Feat: Adding a Contirbutor profil (#501) * 10 functions done * almost finished, debug next * debug time * debuging * pushing recent changes/ still bug because missing functions * debuging finished * adding comments on functions * almost clean * Emit bug * programm compile 🎉 * resolving last test * All test passed * resolve request * 1 test failed because of max swap path lenght exceed test * resolving failed test * resolve * solving * compilation resolved * Added as a Contributor * Adding profil on ReadMe --------- Co-authored-by: Michel <105498726+Sk8erboi84@users.noreply.github.com> * test: Added tests for record_transfer_in (#502) * test(record_transfer_in_function): Added unit tests for record_transfer_in * test(record_transfer_in_function): Added unit tests * test(record_transfer_in_function): Better error message * test(record_transfer_in_function): Mock + Overflow prevented * test(record_transfer_in_function): Removed useless import * test(record_transfer_in_function): record_transfer_in panic on sub overflow * test(record_transfer_in_function): Quick test refacto * test(record_transfer_in_function): Unused variables in tests --------- Co-authored-by: akhercha <akherchache@pm.me> Co-authored-by: zarboq <37303126+zarboq@users.noreply.github.com> * Improve tests of governable contract (#503) * add test to gov * fmt fix * add new contributors (#505) * change panic error * change quote * log2 function is working * fmt * fmt * pow working * fmt * convert type * fix * lock * fmt + lock * works --------- Co-authored-by: Tbelleng <117627242+Tbelleng@users.noreply.github.com> Co-authored-by: Michel <105498726+Sk8erboi84@users.noreply.github.com> Co-authored-by: akhercha <adelkherchache@gmail.com> Co-authored-by: akhercha <akherchache@pm.me> Co-authored-by: sparqet <37338401+sparqet@users.noreply.github.com> Co-authored-by: tevrat aksoy <tevrataksoy@gmail.com> Co-authored-by: zarboq <37303126+zarboq@users.noreply.github.com> Co-authored-by: VictorONN <73134512+VictorONN@users.noreply.github.com> Co-authored-by: kasteph <ayo@kasteph.com> Co-authored-by: Axel Izsak <98711930+axelizsak@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please add the labels corresponding to the type of changes your PR introduces:
What is the current behavior?
Resolves: Issue #477
What is the new behavior?
DepositVaultso it's the same order asStrictBank& follows theinitialize()function ordertest/deposit/deposit_vault.cairoDoes this introduce a breaking change?
Should be a no, but there is that swap of parameters order in the
DepositVaultconstructor. I can revert it if it's dangerous/out of scope!Other information
I have no implemented tests for
record_transfer_in()as the function is not implemented yet. Happy to do it in this PR if you have more informations! 😁I've this test that should fail:
For when a
transfer_outhappens and there's not enough token in theDepositVaultbut I'm really not sure if this is the "right" expected message for theshould_panicline.Maybe we can handle that condition in
src/token/token_utils.cairo -> transfer()to have a relevant error but not sure if that check/computation is what we want.Thanks!