diff --git a/types/dec_coin.go b/types/dec_coin.go index 6e400d7a05..74a5a1ae44 100644 --- a/types/dec_coin.go +++ b/types/dec_coin.go @@ -628,7 +628,7 @@ func ParseDecCoin(coinStr string) (coin DecCoin, err error) { } if err := ValidateDenom(denomStr); err != nil { - return DecCoin{}, fmt.Errorf("invalid denom cannot contain upper case characters or spaces: %s", err) + return DecCoin{}, fmt.Errorf("invalid denom cannot contain spaces: %s", err) } return NewDecCoinFromDec(denomStr, amount), nil diff --git a/types/dec_coin_test.go b/types/dec_coin_test.go index c0cf44cd60..2f35075d3c 100644 --- a/types/dec_coin_test.go +++ b/types/dec_coin_test.go @@ -671,7 +671,7 @@ func (s *decCoinTestSuite) TestDecCoins_IsAllPositive() { {"One Coin - Zero value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(0)}}, false}, - {"One Coin - Postive value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(5)}}, true}, + {"One Coin - Positive value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(5)}}, true}, {"One Coin - Negative value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(-15)}}, false}, @@ -789,7 +789,7 @@ func (s *decCoinTestSuite) TestDecCoins_IsZero() { {"One Coin - Zero value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(0)}}, true}, - {"One Coin - Postive value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(5)}}, false}, + {"One Coin - Positive value", sdk.DecCoins{sdk.DecCoin{testDenom1, sdk.NewDec(5)}}, false}, {"Multiple Coins - All zero value", sdk.DecCoins{ sdk.DecCoin{testDenom1, sdk.NewDec(0)}, @@ -1084,7 +1084,7 @@ func (s *decCoinTestSuite) TestDecCoin_Validate() { input sdk.DecCoin expectedPass bool }{ - {"Uninitalized deccoin", empty, false}, + {"Uninitialized deccoin", empty, false}, {"Invalid denom string", sdk.DecCoin{"(){9**&})", sdk.NewDec(33)}, false}, @@ -1121,11 +1121,9 @@ func (s *decCoinTestSuite) TestDecCoin_ParseDecCoin() { {"Precision over limit", "9.11111111111111111111stake", empty, true}, - // TODO - Clarify - According to error message for ValidateDenom call, supposed to - // throw error when upper case characters are used. Currently uppercase denoms are allowed. - {"Invalid denom", "9.3STAKE", sdk.DecCoin{"STAKE", sdk.NewDecWithPrec(93, 1)}, false}, + {"Valid upper case denom", "9.3STAKE", sdk.DecCoin{"STAKE", sdk.NewDecWithPrec(93, 1)}, false}, - {"Valid input - amount and denom seperated by space", "9.3 stake", sdk.DecCoin{"stake", sdk.NewDecWithPrec(93, 1)}, false}, + {"Valid input - amount and denom separated by space", "9.3 stake", sdk.DecCoin{"stake", sdk.NewDecWithPrec(93, 1)}, false}, {"Valid input - amount and denom concatenated", "9.3stake", sdk.DecCoin{"stake", sdk.NewDecWithPrec(93, 1)}, false}, }