-
Notifications
You must be signed in to change notification settings - Fork 580
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
Wire up simapp to use mock v2 modules #7401
Wire up simapp to use mock v2 modules #7401
Conversation
testing/mock/v2/ibc_module.go
Outdated
func NewIBCModule() IBCModule { | ||
return IBCModule{ | ||
IBCApp: &IBCApp{}, | ||
} | ||
} |
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.
modified signature here to not include the IBCApp as an argument, think it's cleaner this way and don't think it would ever be required to pass a custom IBCApp at module creation time.
"github.com/cosmos/ibc-go/v9/modules/core/exported" | ||
) | ||
|
||
// Keeper defines the IBC connection keeper | ||
type Keeper struct { | ||
Router *types.Router | ||
Router *types.Router | ||
RouterV2 *api.Router |
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.
not sure where this should actually go, we can figure this out later unless someone has a strong opinion about putting it somewhere now!
Quality Gate passed for 'ibc-go'Issues Measures |
Description
Review after #7398
This PR wires up 2 mock applications in the simapp.
closes: #7374
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
).godoc
comments.Files changed
in the GitHub PR explorer.SonarCloud Report
in the comment section below once CI passes.