-
Notifications
You must be signed in to change notification settings - Fork 94
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: missing tokenfactory bindings #431
Conversation
suite.Nil(events) | ||
suite.Equal(uint64(1), response.SequenceId) | ||
suite.Equal("channel-2", response.Channel) | ||
// huh? |
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.
as far as i understand, the reflect contract does not let through response from a binding handler. If binding returns non empty result we should be able to test it somehow. But i'm not insist
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.
Yeah, i left these comments to not forget to think about that. And i found a solution - now reflect contract returns response of message execution thus it's visible to the test
@@ -5,6 +5,9 @@ import ( | |||
"fmt" | |||
"testing" | |||
|
|||
types2 "github.com/CosmWasm/wasmd/x/wasm/types" |
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.
rename alias to something meaningful?
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.
Fixed
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.
lgtm, please fix minor comments
* add unit tests for tokenfactory ForceTransfer method
This PR adds missing
ForceTransfer
andSetDenomMetadata
messages for the TokenFactory moduleRelated PRs: