addressed vishal PR comments

x/stakeibc/keeper/host_zone.go

@@ -276,10 +276,14 @@ func (k Keeper) AddValidatorToHostZone(ctx sdk.Context, chainId string, validato
 		SlashQueryCheckpoint:      checkpoint,
 	})
 
+	// Finally, confirm none of the validator's exceed the weight cap
+	if err := k.CheckValidatorWeightsBelowCap(ctx, hostZone.Validators); err != nil {
+		return err
+	}
+
 	k.SetHostZone(ctx, hostZone)
 
-	// Finally, confirm none of the validator's exceed the weight cap
-	return k.CheckValidatorWeightsBelowCap(ctx, hostZone.Validators)
+	return nil
 }
 
 // Removes a validator from a host zone
@@ -325,6 +329,7 @@ func (k Keeper) CheckValidatorWeightsBelowCap(ctx sdk.Context, validators []*typ
 		return nil
 	}
 
+	// The weight cap in params is an int representing a percentage (e.g. 10 is 10%)
 	params := k.GetParams(ctx)
 	validatorWeightCap := float64(params.ValidatorWeightCap)
 

x/stakeibc/keeper/host_zone_test.go

@@ -382,15 +382,15 @@ func (s *KeeperTestSuite) TestCheckValidatorWeightsBelowCap() {
 			name:      "not enough validators",
 			weightCap: 10,
 			validators: []*types.Validator{
-				{Address: "val1", Weight: 0},
-				{Address: "val2", Weight: 0},
-				{Address: "val3", Weight: 0},
-				{Address: "val4", Weight: 0},
-				{Address: "val5", Weight: 0},
-				{Address: "val6", Weight: 0},
-				{Address: "val7", Weight: 0},
-				{Address: "val8", Weight: 0},
-				{Address: "val9", Weight: 0},
+				{Address: "val1", Weight: 1},
+				{Address: "val2", Weight: 1},
+				{Address: "val3", Weight: 1},
+				{Address: "val4", Weight: 1},
+				{Address: "val5", Weight: 1},
+				{Address: "val6", Weight: 1},
+				{Address: "val7", Weight: 1},
+				{Address: "val8", Weight: 1},
+				{Address: "val9", Weight: 1},
 			},
 			exceedsCap: false,
 		},