allow ability to add new cpps without new denoms

x/oracle/keeper/msg_server.go

@@ -240,8 +240,10 @@ func (ms msgServer) GovAddDenoms(
 		plan.Changes.CurrencyDeviationThresholds = append(plan.Changes.CurrencyDeviationThresholds, cdt)
 	}
 
-	// also update RewardBand key
-	plan.Keys = append(plan.Keys, string(types.KeyRewardBands))
+	// also update RewardBand key if new denoms are getting added
+	if len(msg.DenomList) != 0 {
+		plan.Keys = append(plan.Keys, string(types.KeyRewardBands))
+	}
 
 	// validate plan construction before scheduling
 	err := plan.ValidateBasic()

x/oracle/keeper/msg_server_test.go

@@ -554,6 +554,27 @@ func (s *IntegrationTestSuite) TestMsgServer_GovAddDenom() {
 			Threshold: "2.0",
 		},
 	}
+	currencyPairProviders2 := oracletypes.CurrencyPairProvidersList{
+		{
+			BaseDenom:  "FOOBAR",
+			QuoteDenom: "BAR",
+			PairAddress: []oracletypes.PairAddressProvider{
+				{
+					Address:         "address2",
+					AddressProvider: "provider2",
+				},
+			},
+			Providers: []string{
+				"provider2",
+			},
+		},
+	}
+	currencyDeviationThresholds2 := oracletypes.CurrencyDeviationThresholdList{
+		{
+			BaseDenom: "FOOBAR",
+			Threshold: "2.0",
+		},
+	}
 
 	testCases := []struct {
 		name      string
@@ -604,6 +625,22 @@ func (s *IntegrationTestSuite) TestMsgServer_GovAddDenom() {
 			false,
 			"",
 		},
+		{
+			"valid currency pair providers and currency deviation thresholds addition with no new denoms",
+			&types.MsgGovAddDenoms{
+				Authority:   govAccAddr,
+				Title:       "test",
+				Description: "test",
+				Height:      9,
+				DenomList:   types.DenomList{},
+				Mandatory:   true,
+
+				CurrencyPairProviders:       currencyPairProviders2,
+				CurrencyDeviationThresholds: currencyDeviationThresholds2,
+			},
+			false,
+			"",
+		},
 		{
 			"invalid multiple addition",
 			&types.MsgGovAddDenoms{
@@ -784,6 +821,17 @@ func (s *IntegrationTestSuite) TestMsgServer_GovAddDenom() {
 					band, err := rwb.GetBandFromDenom("REWARD")
 					s.Require().Equal(band, sdk.NewDecWithPrec(2, 3))
 					s.Require().NoError(err)
+
+				case "valid currency pair providers and currency deviation thresholds addition with no new denoms":
+					cpp := s.app.OracleKeeper.CurrencyPairProviders(s.ctx)
+					for i := range currencyPairProviders2 {
+						s.Require().Contains(cpp, currencyPairProviders2[i])
+					}
+
+					cdt := s.app.OracleKeeper.CurrencyDeviationThresholds(s.ctx)
+					for i := range currencyDeviationThresholds2 {
+						s.Require().Contains(cdt, currencyDeviationThresholds2[i])
+					}
 				}
 			}
 		})