-
Notifications
You must be signed in to change notification settings - Fork 628
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
docs(simapp): fixed inline middleware wiring comments #4361
Merged
Merged
Changes from 4 commits
Commits
Show all changes
6 commits
Select commit
Hold shift + click to select a range
3f86fcf
docs(callbacks/simapp): fixed inline stack documentation
srdtrk 68e7bdb
docs(testing/simapp): fixed inline stack documentation
srdtrk b753cf4
merge: branch 'main' into serdar/fix-inline-callbacks-wiring-comments
srdtrk 1ed8f2c
docs(testing/simapp): fixed icacontroller inline stack documentation
srdtrk 483a380
docs(simapp): improved icaController stack's documentation
srdtrk dbfc59c
docs(callbacks/simapp): review fixes
srdtrk 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
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
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -522,7 +522,7 @@ func NewSimApp( | |||||
|
||||||
// Create Transfer Stack | ||||||
// SendPacket, since it is originating from the application to core IBC: | ||||||
// transferKeeper.SendPacket -> fee.SendPacket -> callbacks.SendPacket -> channel.SendPacket | ||||||
// transferKeeper.sendTransfer -> callbacks.SendPacket -> feeKeeper.SendPacket -> channel.SendPacket | ||||||
|
||||||
// RecvPacket, message that originates from core IBC and goes down to app, the flow is the other way | ||||||
// channel.RecvPacket -> callbacks.OnRecvPacket -> fee.OnRecvPacket -> transfer.OnRecvPacket | ||||||
|
@@ -545,7 +545,7 @@ func NewSimApp( | |||||
|
||||||
// Create Interchain Accounts Stack | ||||||
// SendPacket, since it is originating from the application to core IBC: | ||||||
// icaAuthModuleKeeper.SendTx -> icaController.SendPacket -> fee.SendPacket -> callbacks.SendPacket -> channel.SendPacket | ||||||
// icaControllerKeeper.SendTx -> callbacks.SendPacket -> fee.SendPacket -> channel.SendPacket | ||||||
|
||||||
// initialize ICA module with mock module as the authentication module on the controller side | ||||||
var icaControllerStack porttypes.IBCModule | ||||||
|
@@ -574,14 +574,13 @@ func NewSimApp( | |||||
AddRoute(ibcmock.ModuleName+icacontrollertypes.SubModuleName, icaControllerStack) // ica with mock auth module stack route to ica (top level of middleware stack) | ||||||
|
||||||
// Create Mock IBC Fee module stack for testing | ||||||
// SendPacket, since it is originating from the application to core IBC: | ||||||
// mockModule.SendPacket -> fee.SendPacket -> channel.SendPacket | ||||||
// SendPacket, mock module cannot send packets | ||||||
|
||||||
// OnRecvPacket, message that originates from core IBC and goes down to app, the flow is the otherway | ||||||
// channel.RecvPacket -> fee.OnRecvPacket -> mockModule.OnRecvPacket | ||||||
// channel.RecvPacket -> callbacks.OnRecvPacket -> fee.OnRecvPacket -> mockModule.OnRecvPacket | ||||||
|
||||||
// OnAcknowledgementPacket as this is where fee's are paid out | ||||||
// mockModule.OnAcknowledgementPacket -> fee.OnAcknowledgementPacket -> channel.OnAcknowledgementPacket | ||||||
// mockModule.OnAcknowledgementPacket -> callbacks.OnRecvPacket -> fee.OnAcknowledgementPacket -> channel.OnAcknowledgementPacket | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
|
||||||
// create fee wrapped mock module | ||||||
feeMockModule := ibcmock.NewIBCModule(&mockModule, ibcmock.NewIBCApp(MockFeePort, scopedFeeMockKeeper)) | ||||||
|
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
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.
might be best considering its the public name available
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'd be happy with keeping it as
SendPacket
to avoid any confusion