-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: unit test external call + msg::sender mocks #14
Merged
Merged
Changes from 38 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
d6c909b
add motsu multiple contract deployment
qalisander feb635a
bump stylus sdk to 0.7.0-rc.1
qalisander 270e2be
add docs
qalisander 09e7a1a
++
qalisander f29671f
++
qalisander 4f06a58
++
qalisander 1799ae3
use thread id
qalisander 32d59b5
++
qalisander 6947aa6
test three_proxies works
qalisander 7d6ddd2
++
qalisander 08c6530
++
qalisander 233e970
++
qalisander fbaeafe
++
qalisander fa44a57
++
qalisander cb5d91d
++
qalisander dedb194
++
qalisander 1888d3c
use context as a key for storage
qalisander a4ea0ea
impl Drop for Contract
qalisander 7bf0dd3
sender accept generic account
qalisander 366a6f5
init accept generic account
qalisander 7be9b86
++
qalisander 1ffba32
update stylus sdk
qalisander e6b5196
fmt
qalisander 2fe7239
Merge remote-tracking branch 'origin/main' into unit-tests/multiple-c…
qalisander f1cd300
++
qalisander 8ae0ebd
make storage_flush_cache available for linking
qalisander 50c3971
init function is capable of returning value
qalisander 41e9882
call random function for test arguments
qalisander a9c402f
update example with account injection
qalisander 01985a0
rename new(address) -> new_at(address) and add new() method
qalisander efa1be4
Merge branch 'main' into unit-tests/multiple-contract-deployment
bidzyyys 6c52351
update three_proxies test
qalisander 8ec42e6
inject contracts through arguments
qalisander 66c962e
remove get prefix
qalisander bc9cf09
update docs
qalisander 28f1d22
++
qalisander 2766a43
++
qalisander 8f662c8
rename `contract` field to `storage`
qalisander ada172f
update changelog
qalisander d3e60a5
prevent Contract<..> drop restricting ContractCall<..> with reference
qalisander 74e16fa
add static call + test
qalisander f1a5e78
add error call test
qalisander efc7e73
elide lifetimes
qalisander 7b330c3
fix fmt
qalisander File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left my feedback on
motsu::test
macro in our group chat, but the gist of it is that with the new context design in this PR, contracts/EOAs can be safely manually instantiated with little effort, and fuzzing crates likeproptest
work with motsu out-of-the-box (no need for special macro wrappers).However, there are arguments to be made that we need to keep this attribute: