check the chainid (#1750)

change the spec name in the tests to create the case where the specs names are the same at the start Co-authored-by: Yaroms <[email protected]>
lavanet · Oct 20, 2024 · 7eb86a4 · 7eb86a4
1 parent 3dc3e32
commit 7eb86a4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/x/rewards/keeper/base_pay.go b/x/rewards/keeper/base_pay.go
@@ -84,8 +84,10 @@ func (k Keeper) popAllBasePayForChain(ctx sdk.Context, chainID string) (list []t
 		k.cdc.MustUnmarshal(iterator.Value(), &val)
 		bs := types.BasePayKeyRecover(string(iterator.Key()))
 		bs.BasePay = val
-		list = append(list, bs)
-		store.Delete(iterator.Key())
+		if bs.ChainId == chainID {
+			list = append(list, bs)
+			store.Delete(iterator.Key())
+		}
 	}
 
 	return

diff --git a/x/rewards/keeper/helpers_test.go b/x/rewards/keeper/helpers_test.go
@@ -34,7 +34,7 @@ var (
 		sdk.NewCoin(commontypes.TokenDenom, sdk.NewInt(1100)),
 		sdk.NewCoin(ibcDenom, sdk.NewInt(500)),
 	)
-	mockSpec2 string = "mock2"
+	mockSpec2 string = "mockspec2"
 )
 
 type tester struct {