-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Start global capability index at 1 #6047
Conversation
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.
utACK
Codecov Report
@@ Coverage Diff @@
## master #6047 +/- ##
=======================================
Coverage 54.69% 54.69%
=======================================
Files 424 424
Lines 25808 25814 +6
=======================================
+ Hits 14115 14119 +4
- Misses 10716 10717 +1
- Partials 977 978 +1 |
@@ -98,7 +98,10 @@ func (am AppModule) RegisterInvariants(_ sdk.InvariantRegistry) {} | |||
|
|||
// InitGenesis performs the capability module's genesis initialization It returns | |||
// no validator updates. | |||
func (am AppModule) InitGenesis(_ sdk.Context, _ codec.JSONMarshaler, _ json.RawMessage) []abci.ValidatorUpdate { | |||
func (am AppModule) InitGenesis(ctx sdk.Context, _ codec.JSONMarshaler, _ json.RawMessage) []abci.ValidatorUpdate { |
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.
Won't this be run after port binding in app.go
? That seems problematic
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.
... never mind - however, somehow, this is not being called
* start index at 1 * Update x/capability/keeper/keeper.go Co-authored-by: Federico Kunze <[email protected]>
Closes: #6039
Description
An annoying quirk in the capability testing setup prevented us from catching the bug reported above. Created a new testcase
TestOriginalCapabilityKeeper
that reproduced the failure above and is now passing.Created
InitGenesis
logic to set initial global index to 1For contributor use:
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerFor admin use:
WIP
,R4R
,docs
, etc)