Skip to content

Commit

Permalink
fix: export x/collection params into genesis (#1268)
Browse files Browse the repository at this point in the history
* Export params

* Update CHANGELOG.md

* Move position
  • Loading branch information
0Tech authored Mar 22, 2024
1 parent 819a938 commit e86b980
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
* (x/collection) [\#1276](https://github.com/Finschia/finschia-sdk/pull/1276) eliminates potential risk for Insufficient Sanity Check of tokenID in Genesis
* (x/foundation) [\#1277](https://github.com/Finschia/finschia-sdk/pull/1277) add init logic of foundation module accounts to InitGenesis in order to eliminate potential panic
* (x/collection, x/token) [\#1288](https://github.com/Finschia/finschia-sdk/pull/1288) use accAddress to compare in validatebasic function in collection & token modules
* (x/collection) [\#1268](https://github.com/Finschia/finschia-sdk/pull/1268) export x/collection params into genesis

### Removed

Expand Down
1 change: 1 addition & 0 deletions x/collection/keeper/genesis.go
Original file line number Diff line number Diff line change
Expand Up @@ -211,6 +211,7 @@ func (k Keeper) ExportGenesis(ctx sdk.Context) *collection.GenesisState {
contracts := k.getContracts(ctx)

return &collection.GenesisState{
Params: k.GetParams(ctx),
Contracts: contracts,
NextClassIds: k.getAllNextClassIDs(ctx),
Classes: k.getClasses(ctx, contracts),
Expand Down
8 changes: 8 additions & 0 deletions x/collection/keeper/genesis_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,3 +28,11 @@ func (s *KeeperTestSuite) TestImportExportGenesis() {
newGenesis := s.keeper.ExportGenesis(s.ctx)
s.Require().Equal(genesis, newGenesis)
}

func (s *KeeperTestSuite) TestExportGenesis() {
genesis := s.keeper.ExportGenesis(s.ctx)

params := genesis.Params
s.Require().NotZero(params.DepthLimit)
s.Require().NotZero(params.WidthLimit)
}

0 comments on commit e86b980

Please sign in to comment.