From ceeb01a310eab34a25182eb87226b0f1c6e71fcf Mon Sep 17 00:00:00 2001
From: MadReza <mohammadreza@prismprotocol.app>
Date: Fri, 8 Sep 2023 17:29:19 +0330
Subject: [PATCH 1/2] fix: set name, display and symbol of denom metadata in
 tokenfactory's CreateDenom

---
 x/tokenfactory/keeper/createdenom.go      |  5 +++-
 x/tokenfactory/keeper/createdenom_test.go |  5 +++-
 x/tokenfactory/keeper/genesis_test.go     | 28 ++++++++++++++++++++---
 3 files changed, 33 insertions(+), 5 deletions(-)

diff --git a/x/tokenfactory/keeper/createdenom.go b/x/tokenfactory/keeper/createdenom.go
index 8ec1b6b83cf..cc07e453ab0 100644
--- a/x/tokenfactory/keeper/createdenom.go
+++ b/x/tokenfactory/keeper/createdenom.go
@@ -35,7 +35,10 @@ func (k Keeper) createDenomAfterValidation(ctx sdk.Context, creatorAddr string,
 				Denom:    denom,
 				Exponent: 0,
 			}},
-			Base: denom,
+			Base:    denom,
+			Name:    denom,
+			Symbol:  denom,
+			Display: denom,
 		}
 
 		k.bankKeeper.SetDenomMetaData(ctx, denomMetaData)
diff --git a/x/tokenfactory/keeper/createdenom_test.go b/x/tokenfactory/keeper/createdenom_test.go
index 2d3bc7c1dd1..48697a50650 100644
--- a/x/tokenfactory/keeper/createdenom_test.go
+++ b/x/tokenfactory/keeper/createdenom_test.go
@@ -175,7 +175,10 @@ func (s *KeeperTestSuite) TestCreateDenom() {
 						Denom:    res.GetNewTokenDenom(),
 						Exponent: 0,
 					}},
-					Base: res.GetNewTokenDenom(),
+					Base:    res.GetNewTokenDenom(),
+					Display: res.GetNewTokenDenom(),
+					Name:    res.GetNewTokenDenom(),
+					Symbol:  res.GetNewTokenDenom(),
 				}, metadata)
 			} else {
 				s.Require().Error(err)
diff --git a/x/tokenfactory/keeper/genesis_test.go b/x/tokenfactory/keeper/genesis_test.go
index 28f223c4986..9862f14a32a 100644
--- a/x/tokenfactory/keeper/genesis_test.go
+++ b/x/tokenfactory/keeper/genesis_test.go
@@ -38,7 +38,16 @@ func (s *KeeperTestSuite) TestGenesis() {
 	for i, denom := range genesisState.FactoryDenoms {
 		// hacky, sets bank metadata to exist if i != 0, to cover both cases.
 		if i != 0 {
-			app.BankKeeper.SetDenomMetaData(s.Ctx, banktypes.Metadata{Base: denom.GetDenom(), Display: "test"})
+			app.BankKeeper.SetDenomMetaData(s.Ctx, banktypes.Metadata{
+				DenomUnits: []*banktypes.DenomUnit{{
+					Denom:    denom.GetDenom(),
+					Exponent: 0,
+				}},
+				Base:    denom.GetDenom(),
+				Display: denom.GetDenom(),
+				Name:    denom.GetDenom(),
+				Symbol:  denom.GetDenom(),
+			})
 		}
 	}
 
@@ -57,14 +66,27 @@ func (s *KeeperTestSuite) TestGenesis() {
 	s.Require().NotNil(exportedGenesis)
 	s.Require().Equal(genesisState, *exportedGenesis)
 
+	// verify that the exported bank genesis is valid
 	app.BankKeeper.SetParams(s.Ctx, banktypes.DefaultParams())
-	app.BankKeeper.InitGenesis(s.Ctx, app.BankKeeper.ExportGenesis(s.Ctx))
+	exportedBankGenesis := app.BankKeeper.ExportGenesis(s.Ctx)
+	s.Require().NoError(exportedBankGenesis.Validate())
+
+	app.BankKeeper.InitGenesis(s.Ctx, exportedBankGenesis)
 	for i, denom := range genesisState.FactoryDenoms {
 		// hacky, check whether bank metadata is not replaced if i != 0, to cover both cases.
 		if i != 0 {
 			metadata, found := app.BankKeeper.GetDenomMetaData(s.Ctx, denom.GetDenom())
 			s.Require().True(found)
-			s.Require().Equal(metadata, banktypes.Metadata{Base: denom.GetDenom(), Display: "test"})
+			s.Require().EqualValues(metadata, banktypes.Metadata{
+				DenomUnits: []*banktypes.DenomUnit{{
+					Denom:    denom.GetDenom(),
+					Exponent: 0,
+				}},
+				Base:    denom.GetDenom(),
+				Display: denom.GetDenom(),
+				Name:    denom.GetDenom(),
+				Symbol:  denom.GetDenom(),
+			})
 		}
 	}
 }

From b7593d5a3a003cf90c3dcc985fb69c1dc982d8c3 Mon Sep 17 00:00:00 2001
From: MadReza <mohammadreza@prismprotocol.app>
Date: Fri, 8 Sep 2023 18:03:30 +0330
Subject: [PATCH 2/2] docs: add changelog entry

---
 CHANGELOG.md | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index c40898e540f..58fd9dc0beb 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -42,6 +42,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+### State Breaking
+
+* [#6344](https://github.com/osmosis-labs/osmosis/pull/6344) fix: set name, display and symbol of denom metadata in tokenfactory's CreateDenom
+
 ### Bug Fixes
 
 * [#6334](https://github.com/osmosis-labs/osmosis/pull/6334) fix: enable taker fee cli