Skip to content

Commit

Permalink
Merge pull request public-awesome#1052 from costcould/main
Browse files Browse the repository at this point in the history
chore: fix some comments
  • Loading branch information
jhernandezb authored Jun 7, 2024
2 parents 65fabee + 85addb4 commit 3916bb4
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion x/cron/keeper/msg_server_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ func TestDemoteFromPrivilegedContract(t *testing.T) {
true,
},
{
"contract curretly does not have privilege to demote it",
"contract currently does not have privilege to demote it",
func(ctx sdk.Context, keeper keeper.Keeper) *types.MsgDemoteFromPrivilegedContract {
govModuleAddr := keeper.GetAuthority()
msg := types.MsgDemoteFromPrivilegedContract{
Expand Down
6 changes: 3 additions & 3 deletions x/cron/keeper/privileged_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ import (
func Test_SetPrivileged(t *testing.T) {
k, ctx := keeper.CronKeeper(t)

// contract doesnt exist
// contract doesn't exist
acc1 := sample.AccAddress()
err := k.SetPrivileged(ctx, acc1)
if err == nil {
Expand All @@ -29,14 +29,14 @@ func Test_SetPrivileged(t *testing.T) {
func Test_UnsetPrivileged(t *testing.T) {
k, ctx := keeper.CronKeeper(t)

// contract doesnt exist
// contract doesn't exist
acc1 := sample.AccAddress()
err := k.UnsetPrivileged(ctx, acc1)
if err == nil {
t.Errorf("expected %s to not exist, and fail to set privilege", acc1)
}

// contract exists but doesnt have privilege
// contract exists but doesn't have privilege
acc2 := sdk.MustAccAddressFromBech32("cosmos1qyqszqgpqyqszqgpqyqszqgpqyqszqgpjnp7du")
err = k.UnsetPrivileged(ctx, acc2)
if err == nil {
Expand Down
2 changes: 1 addition & 1 deletion x/globalfee/keeper/code_authorization_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ func Test_CodeAuthorization(t *testing.T) {
require.Error(t, err)
})

t.Run("authorization doesnt exist", func(t *testing.T) {
t.Run("authorization doesn't exist", func(t *testing.T) {
found := k.HasCodeAuthorization(ctx, ca.CodeID)
require.False(t, found)
})
Expand Down
2 changes: 1 addition & 1 deletion x/globalfee/keeper/contract_authorization_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ func Test_ContractAuthorization(t *testing.T) {
require.Error(t, err)
})

t.Run("authorization doesnt exist", func(t *testing.T) {
t.Run("authorization doesn't exist", func(t *testing.T) {
found := k.HasContractAuthorization(ctx, sdk.MustAccAddressFromBech32(ca.ContractAddress))
require.False(t, found)
})
Expand Down
2 changes: 1 addition & 1 deletion x/globalfee/keeper/msg_server_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -235,7 +235,7 @@ func TestSetContractAuthorization(t *testing.T) {
true,
},
{
"contract doesnt exist",
"contract doesn't exist",
func(ctx sdk.Context, keeper keeper.Keeper) *types.MsgSetContractAuthorization {
sender := sample.AccAddress()
params := types.NewParams([]string{sender.String()})
Expand Down

0 comments on commit 3916bb4

Please sign in to comment.