Skip to content

Commit

Permalink
Fixing lint errors.
Browse files Browse the repository at this point in the history
  • Loading branch information
arhamchordia committed Feb 13, 2023
1 parent bef646a commit 55ff989
Show file tree
Hide file tree
Showing 37 changed files with 259 additions and 233 deletions.
8 changes: 4 additions & 4 deletions wasmbinding/callback.go
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,10 @@ import (
"github.com/gogo/protobuf/proto"
"github.com/tendermint/tendermint/libs/log"

"cosmossdk.io/errors"
"github.com/CosmWasm/wasmd/x/wasm"
wasmk "github.com/CosmWasm/wasmd/x/wasm/keeper"
sdk "github.com/cosmos/cosmos-sdk/types"
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
ibctransfertypes "github.com/cosmos/ibc-go/v5/modules/apps/transfer/types"
gammbalancer "github.com/quasarlabs/quasarnode/osmosis/gamm/pool-models/balancer"
gammtypes "github.com/quasarlabs/quasarnode/osmosis/gamm/types"
Expand Down Expand Up @@ -127,7 +127,7 @@ func (c *CallbackPlugin) doHandle(ctx sdk.Context, seq uint64, channel string, p
resp := new(bytes.Buffer)
err := m.Marshal(resp, response)
if err != nil {
return sdkerrors.Wrap(err, "ibc ack callback marshalling")
return errors.Wrap(err, "ibc ack callback marshalling")
}

data, err := json.Marshal(ContractAck{
Expand All @@ -144,13 +144,13 @@ func (c *CallbackPlugin) doHandle(ctx sdk.Context, seq uint64, channel string, p
})

if err != nil {
return sdkerrors.Wrap(err, "ibc ack callback")
return errors.Wrap(err, "ibc ack callback")
}
c.Logger(ctx).Info(fmt.Sprintf("Preparing callback message: %v", string(data)))

_, err = c.contractKeeper.Execute(ctx, addr, c.callBackAddress, data, nil)
if err != nil {
return sdkerrors.Wrap(err, "ack callback execute")
return errors.Wrap(err, "ack callback execute")
}

return nil
Expand Down
58 changes: 29 additions & 29 deletions wasmbinding/message_plugin.go
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ import (
"encoding/json"
"time"

"cosmossdk.io/errors"
wasmkeeper "github.com/CosmWasm/wasmd/x/wasm/keeper"
wasmvmtypes "github.com/CosmWasm/wasmvm/types"
sdk "github.com/cosmos/cosmos-sdk/types"
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
bankkeeper "github.com/cosmos/cosmos-sdk/x/bank/keeper"

"github.com/quasarlabs/quasarnode/wasmbinding/bindings"
Expand Down Expand Up @@ -41,7 +41,7 @@ func (m *CustomMessenger) DispatchMsg(ctx sdk.Context, contractAddr sdk.AccAddre
// leave everything else for the wrapped version
var contractMsg bindings.QuasarMsg
if err := json.Unmarshal(msg.Custom, &contractMsg); err != nil {
return nil, nil, sdkerrors.Wrap(err, "osmosis msg")
return nil, nil, errors.Wrap(err, "osmosis msg")
}
if contractMsg.TestScenario != nil {
return m.testScenario(ctx, contractAddr, contractMsg.TestScenario)
Expand Down Expand Up @@ -78,7 +78,7 @@ func (m *CustomMessenger) testScenario(ctx sdk.Context, contractAddr sdk.AccAddr
err := PerformTestScenario(m.intergammKeeper, ctx, contractAddr, testScenario)
// err := PerformCreateDenom(m.tokenFactory, m.bank, ctx, contractAddr, createDenom)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "perform test scenario")
return nil, nil, errors.Wrap(err, "perform test scenario")
}
return nil, nil, nil
}
Expand All @@ -95,7 +95,7 @@ func PerformTestScenario(k *intergammkeeper.Keeper, ctx sdk.Context, contractAdd
// msgCreateDenom := tokenfactorytypes.NewMsgCreateDenom(contractAddr.String(), createDenom.Subdenom)

if err := msgTestScenario.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "failed validating MsgTestScenario")
return errors.Wrap(err, "failed validating MsgTestScenario")
}

// Run the test scenario
Expand All @@ -104,15 +104,15 @@ func PerformTestScenario(k *intergammkeeper.Keeper, ctx sdk.Context, contractAdd
msgTestScenario,
)
if err != nil {
return sdkerrors.Wrap(err, "running test scenario")
return errors.Wrap(err, "running test scenario")
}
return nil
}

func (m *CustomMessenger) sendToken(ctx sdk.Context, contractAddr sdk.AccAddress, send *bindings.SendToken) ([]sdk.Event, [][]byte, error) {
err := PerformSendToken(m.intergammKeeper, m.bank, ctx, contractAddr, send, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "send token")
return nil, nil, errors.Wrap(err, "send token")
}
return nil, nil, nil
}
Expand All @@ -123,13 +123,13 @@ func PerformSendToken(k *intergammkeeper.Keeper, b *bankkeeper.BaseKeeper, ctx s
}
sdkMsg := intergammtypes.NewMsgSendToken(contractAddr.String(), send.DestinationLocalZoneId, send.Receiver, send.Coin)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.SendToken(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "sending tokens")
return errors.Wrap(err, "sending tokens")
}

// register the packet as sent with the callback plugin
Expand All @@ -140,7 +140,7 @@ func PerformSendToken(k *intergammkeeper.Keeper, b *bankkeeper.BaseKeeper, ctx s
func (m *CustomMessenger) RegisterICAOnZone(ctx sdk.Context, contractAddr sdk.Address, register *bindings.RegisterICAOnZone) ([]sdk.Event, [][]byte, error) {
err := PerformRegisterICAOnZone(m.intergammKeeper, ctx, contractAddr, register)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "register ica account")
return nil, nil, errors.Wrap(err, "register ica account")
}
return nil, nil, nil
}
Expand All @@ -152,21 +152,21 @@ func PerformRegisterICAOnZone(k *intergammkeeper.Keeper, ctx sdk.Context, contra

sdkMsg := intergammtypes.NewMsgRegisterICAOnZone(contractAddr.String(), register.ZoneId)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
_, err := msgServer.RegisterICAOnZone(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "register interchain account")
return errors.Wrap(err, "register interchain account")
}
return nil
}

func (m *CustomMessenger) OsmosisJoinPool(ctx sdk.Context, contractAddr sdk.AccAddress, join *bindings.OsmosisJoinPool) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisJoinPool(m.intergammKeeper, ctx, contractAddr, join, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "join pool")
return nil, nil, errors.Wrap(err, "join pool")
}
return nil, nil, nil
}
Expand All @@ -179,13 +179,13 @@ func PerformOsmosisJoinPool(k *intergammkeeper.Keeper, ctx sdk.Context, contract
// TODO see if hardcoding creator like this works
sdkMsg := intergammtypes.NewMsgTransmitIbcJoinPool(contractAddr.String(), join.ConnectionId, join.TimeoutTimestamp, join.PoolId, join.ShareOutAmount, join.TokenInMaxs)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcJoinPool(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "join pool")
return errors.Wrap(err, "join pool")
}

cb.OnSendPacket(ctx, res.Seq, res.Channel, res.PortId, contractAddr)
Expand All @@ -195,7 +195,7 @@ func PerformOsmosisJoinPool(k *intergammkeeper.Keeper, ctx sdk.Context, contract
func (m *CustomMessenger) OsmosisExitPool(ctx sdk.Context, contractAddr sdk.AccAddress, exit *bindings.OsmosisExitPool) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisExitPool(m.intergammKeeper, ctx, contractAddr, exit, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "exit pool")
return nil, nil, errors.Wrap(err, "exit pool")
}
return nil, nil, nil
}
Expand All @@ -207,13 +207,13 @@ func PerformOsmosisExitPool(k *intergammkeeper.Keeper, ctx sdk.Context, contract

sdkMsg := intergammtypes.NewMsgTransmitIbcExitPool(contractAddr.String(), exit.ConnectionId, exit.TimeoutTimestamp, exit.PoolId, exit.ShareInAmount, exit.TokenOutMins)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcExitPool(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "exit pool")
return errors.Wrap(err, "exit pool")
}

cb.OnSendPacket(ctx, res.GetSeq(), res.Channel, res.PortId, contractAddr)
Expand All @@ -223,7 +223,7 @@ func PerformOsmosisExitPool(k *intergammkeeper.Keeper, ctx sdk.Context, contract
func (m *CustomMessenger) OsmosisLockTokens(ctx sdk.Context, contractAddr sdk.AccAddress, withdraw *bindings.OsmosisLockTokens) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisLockTokens(m.intergammKeeper, ctx, contractAddr, withdraw, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "withdraw")
return nil, nil, errors.Wrap(err, "withdraw")
}
return nil, nil, nil
}
Expand All @@ -236,13 +236,13 @@ func PerformOsmosisLockTokens(k *intergammkeeper.Keeper, ctx sdk.Context, contra
// TODO: lets make sure the way we do durations is correct
sdkMsg := intergammtypes.NewMsgTransmitIbcLockTokens(contractAddr.String(), lock.ConnectionId, lock.TimeoutTimestamp, time.Duration(lock.Duration), lock.Coins)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcLockTokens(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "lock tokens")
return errors.Wrap(err, "lock tokens")
}

cb.OnSendPacket(ctx, res.GetSeq(), res.Channel, res.PortId, contractAddr)
Expand All @@ -252,7 +252,7 @@ func PerformOsmosisLockTokens(k *intergammkeeper.Keeper, ctx sdk.Context, contra
func (m *CustomMessenger) OsmosisBeginUnlocking(ctx sdk.Context, contractAddr sdk.AccAddress, begin *bindings.OsmosisBeginUnlocking) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisBeginUnlocking(m.intergammKeeper, ctx, contractAddr, begin, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "begin unlocking")
return nil, nil, errors.Wrap(err, "begin unlocking")
}
return nil, nil, nil
}
Expand All @@ -264,13 +264,13 @@ func PerformOsmosisBeginUnlocking(k *intergammkeeper.Keeper, ctx sdk.Context, co

sdkMsg := intergammtypes.NewMsgTransmitIbcBeginUnlocking(contractAddr.String(), begin.ConnectionId, begin.TimeoutTimestamp, begin.Id, begin.Coins)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcBeginUnlocking(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "begin unlocking")
return errors.Wrap(err, "begin unlocking")
}

cb.OnSendPacket(ctx, res.GetSeq(), res.Channel, res.PortId, contractAddr)
Expand All @@ -280,7 +280,7 @@ func PerformOsmosisBeginUnlocking(k *intergammkeeper.Keeper, ctx sdk.Context, co
func (m *CustomMessenger) OsmosisJoinSwapExternAmountIn(ctx sdk.Context, contractAddr sdk.AccAddress, join *bindings.OsmosisJoinSwapExternAmountIn) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisJoinSwapExternAmountIn(m.intergammKeeper, ctx, contractAddr, join, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "join swap extern amount in")
return nil, nil, errors.Wrap(err, "join swap extern amount in")
}
return nil, nil, nil
}
Expand All @@ -292,13 +292,13 @@ func PerformOsmosisJoinSwapExternAmountIn(k *intergammkeeper.Keeper, ctx sdk.Con

sdkMsg := intergammtypes.NewMsgTransmitIbcJoinSwapExternAmountIn(contractAddr.String(), join.ConnectionId, join.TimeoutTimestamp, join.PoolId, join.ShareOutMinAmount, join.TokenIn)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcJoinSwapExternAmountIn(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "join swap extern amount in")
return errors.Wrap(err, "join swap extern amount in")
}

cb.OnSendPacket(ctx, res.GetSeq(), res.Channel, res.PortId, contractAddr)
Expand All @@ -309,7 +309,7 @@ func PerformOsmosisJoinSwapExternAmountIn(k *intergammkeeper.Keeper, ctx sdk.Con
func (m *CustomMessenger) OsmosisExitSwapExternAmountOut(ctx sdk.Context, contractAddr sdk.AccAddress, exit *bindings.OsmosisExitSwapExternAmountOut) ([]sdk.Event, [][]byte, error) {
err := PerformOsmosisExitSwapExternAmountOut(m.intergammKeeper, ctx, contractAddr, exit, m.callback)
if err != nil {
return nil, nil, sdkerrors.Wrap(err, "exit swap extern amount out")
return nil, nil, errors.Wrap(err, "exit swap extern amount out")
}
return nil, nil, nil
}
Expand All @@ -321,13 +321,13 @@ func PerformOsmosisExitSwapExternAmountOut(k *intergammkeeper.Keeper, ctx sdk.Co

sdkMsg := intergammtypes.NewMsgTransmitIbcExitSwapExternAmountOut(contractAddr.String(), exit.ConnectionId, exit.TimeoutTimestamp, exit.PoolId, exit.ShareInAmount, exit.TokenOutMins)
if err := sdkMsg.ValidateBasic(); err != nil {
return sdkerrors.Wrap(err, "basic validate msg")
return errors.Wrap(err, "basic validate msg")
}

msgServer := intergammkeeper.NewMsgServerImpl(k)
res, err := msgServer.TransmitIbcExitSwapExternAmountOut(sdk.WrapSDKContext(ctx), sdkMsg)
if err != nil {
return sdkerrors.Wrap(err, "join swap extern amount out")
return errors.Wrap(err, "join swap extern amount out")
}

cb.OnSendPacket(ctx, res.GetSeq(), res.Channel, res.PortId, contractAddr)
Expand Down
13 changes: 7 additions & 6 deletions wasmbinding/query_plugin.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package wasmbinding
import (
"encoding/json"

"cosmossdk.io/errors"
wasmvmtypes "github.com/CosmWasm/wasmvm/types"
sdk "github.com/cosmos/cosmos-sdk/types"
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
Expand All @@ -14,7 +15,7 @@ func CustomQuerier(qk qoraclekeeper.Keeper) func(ctx sdk.Context, request json.R
return func(ctx sdk.Context, request json.RawMessage) ([]byte, error) {
var contractQuery bindings.QuasarQuery
if err := json.Unmarshal(request, &contractQuery); err != nil {
return nil, sdkerrors.Wrap(err, "osmosis query")
return nil, errors.Wrap(err, "osmosis query")
}

switch {
Expand All @@ -23,29 +24,29 @@ func CustomQuerier(qk qoraclekeeper.Keeper) func(ctx sdk.Context, request json.R

bz, err := json.Marshal(pools)
if err != nil {
return nil, sdkerrors.Wrap(err, "failed to marshal quasar qoracle pools")
return nil, errors.Wrap(err, "failed to marshal quasar qoracle pools")
}
return bz, nil
case contractQuery.Pool != nil:
pool, found := qk.GetPool(ctx, contractQuery.Pool.Id)
if !found {
return nil, sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "pool not found")
return nil, errors.Wrap(sdkerrors.ErrInvalidRequest, "pool not found")
}

bz, err := json.Marshal(pool)
if err != nil {
return nil, sdkerrors.Wrap(err, "failed to marshal quasar pool")
return nil, errors.Wrap(err, "failed to marshal quasar pool")
}
return bz, nil
case contractQuery.TokenPrice != nil:
price, err := qk.GetDenomPrice(ctx, contractQuery.TokenPrice.Denom)
if err != nil {
return nil, sdkerrors.Wrap(err, "failed to get token price")
return nil, errors.Wrap(err, "failed to get token price")
}

bz, err := price.MarshalJSON()
if err != nil {
return nil, sdkerrors.Wrap(err, "failed to marshal quasar token price")
return nil, errors.Wrap(err, "failed to marshal quasar token price")
}
return bz, nil
default:
Expand Down
3 changes: 2 additions & 1 deletion x/epochs/handler.go
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package epochs

import (
"cosmossdk.io/errors"
"fmt"

sdk "github.com/cosmos/cosmos-sdk/types"
Expand All @@ -15,7 +16,7 @@ func NewHandler(k *keeper.Keeper) sdk.Handler {
switch msg := msg.(type) {
default:
errMsg := fmt.Sprintf("unrecognized %s message type: %T", types.ModuleName, msg)
return nil, sdkerrors.Wrap(sdkerrors.ErrUnknownRequest, errMsg)
return nil, errors.Wrap(sdkerrors.ErrUnknownRequest, errMsg)
}
}
}
4 changes: 2 additions & 2 deletions x/epochs/types/errors.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,10 @@ package types
// DONTCOVER

import (
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
"cosmossdk.io/errors"
)

// x/epochs module sentinel errors.
var (
ErrSample = sdkerrors.Register(ModuleName, 1100, "sample error")
ErrSample = errors.Register(ModuleName, 1100, "sample error")
)
3 changes: 2 additions & 1 deletion x/intergamm/handler.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package intergamm
import (
"fmt"

"cosmossdk.io/errors"
sdk "github.com/cosmos/cosmos-sdk/types"
sdkerrors "github.com/cosmos/cosmos-sdk/types/errors"
"github.com/quasarlabs/quasarnode/x/intergamm/keeper"
Expand Down Expand Up @@ -56,7 +57,7 @@ func NewHandler(k *keeper.Keeper) sdk.Handler {
// this line is used by starport scaffolding # 1
default:
errMsg := fmt.Sprintf("unrecognized %s message type: %T", types.ModuleName, msg)
return nil, sdkerrors.Wrap(sdkerrors.ErrUnknownRequest, errMsg)
return nil, errors.Wrap(sdkerrors.ErrUnknownRequest, errMsg)
}
}
}
Loading

0 comments on commit 55ff989

Please sign in to comment.