diff --git a/app/upgrades/v15/upgrade_test.go b/app/upgrades/v15/upgrade_test.go index 080a3a9400a..bf02bf4bc4e 100644 --- a/app/upgrades/v15/upgrade_test.go +++ b/app/upgrades/v15/upgrade_test.go @@ -57,7 +57,7 @@ func (suite *UpgradeTestSuite) TestMigrateNextPoolIdAndCreatePool() { gammKeeper := suite.App.GAMMKeeper poolmanagerKeeper := suite.App.PoolManagerKeeper - nextPoolId := gammKeeper.GetNextPoolId(ctx) + nextPoolId := gammKeeper.GetNextPoolId(ctx) //nolint:staticcheck // we're using the deprecated version for testing. suite.Require().Equal(expectedNextPoolId, nextPoolId) // system under test. diff --git a/tests/e2e/e2e_setup_test.go b/tests/e2e/e2e_setup_test.go index d38cfcee66e..2f4632519cf 100644 --- a/tests/e2e/e2e_setup_test.go +++ b/tests/e2e/e2e_setup_test.go @@ -18,7 +18,7 @@ const ( // Environment variable name to skip the IBC tests skipIBCEnv = "OSMOSIS_E2E_SKIP_IBC" // Environment variable name to skip state sync testing - skipStateSyncEnv = "OSMOSIS_E2E_SKIP_STATE_SYNC" + skipStateSyncEnv = "OSMOSIS_E2E_SKIP_STATE_SYNC" //nolint:unused // this is used in the code // Environment variable name to determine if this upgrade is a fork forkHeightEnv = "OSMOSIS_E2E_FORK_HEIGHT" // Environment variable name to skip cleaning up Docker resources in teardown diff --git a/tests/e2e/e2e_test.go b/tests/e2e/e2e_test.go index 5ad1b14daba..10de4704993 100644 --- a/tests/e2e/e2e_test.go +++ b/tests/e2e/e2e_test.go @@ -662,7 +662,7 @@ func (s *IntegrationTestSuite) TestConcentratedLiquidity() { // Withdraw Position // Withdraw Position parameters - var defaultLiquidityRemoval string = "1000" + defaultLiquidityRemoval := "1000" chainA.WaitForNumHeights(2) diff --git a/tests/ibc-hooks/ibc_middleware_test.go b/tests/ibc-hooks/ibc_middleware_test.go index e2653295374..15933a9d8b8 100644 --- a/tests/ibc-hooks/ibc_middleware_test.go +++ b/tests/ibc-hooks/ibc_middleware_test.go @@ -732,7 +732,7 @@ func (suite *HooksTestSuite) SetupCrosschainSwaps(chainName Chain) (sdk.AccAddre ctx := chain.GetContext() // Configuring two prefixes for the same channel here. This is so that we can test bad acks when the receiver can't handle the receiving addr - msg := fmt.Sprintf(`{ + msg := `{ "modify_bech32_prefixes": { "operations": [ {"operation": "set", "chain_name": "osmosis", "prefix": "osmo"}, @@ -742,7 +742,7 @@ func (suite *HooksTestSuite) SetupCrosschainSwaps(chainName Chain) (sdk.AccAddre ] } } - `) + ` _, err = contractKeeper.Execute(ctx, registryAddr, owner, []byte(msg), sdk.NewCoins()) suite.Require().NoError(err) @@ -958,7 +958,7 @@ func (suite *HooksTestSuite) TestUnwrapToken() { contractKeeper := wasmkeeper.NewDefaultPermissionKeeper(osmosisApp.WasmKeeper) - msg := fmt.Sprintf(`{ + msg := `{ "modify_bech32_prefixes": { "operations": [ {"operation": "set", "chain_name": "osmosis", "prefix": "osmo"}, @@ -968,7 +968,7 @@ func (suite *HooksTestSuite) TestUnwrapToken() { ] } } - `) + ` _, err := contractKeeper.Execute(ctx, registryAddr, owner, []byte(msg), sdk.NewCoins()) suite.Require().NoError(err) @@ -1559,7 +1559,7 @@ func (suite *HooksTestSuite) TestCrosschainSwapsViaIBCMultiHop() { // Now the swwap can actually execute on A via the callback and generate a new packet with the swapped token to B packet, err = ibctesting.ParsePacketFromEvents(res.GetEvents()) suite.Require().NoError(err) - res = suite.RelayPacketNoAck(packet, AtoB) + _ = suite.RelayPacketNoAck(packet, AtoB) balanceToken0After := osmosisAppB.BankKeeper.GetBalance(suite.chainB.GetContext(), accountB, token0ACB) suite.Require().Equal(int64(1000), balanceToken0.Amount.Sub(balanceToken0After.Amount).Int64()) diff --git a/wasmbinding/query_plugin_test.go b/wasmbinding/query_plugin_test.go index cda69b4cec7..5f56e86e355 100644 --- a/wasmbinding/query_plugin_test.go +++ b/wasmbinding/query_plugin_test.go @@ -14,7 +14,7 @@ import ( "github.com/cosmos/cosmos-sdk/types/query" authtypes "github.com/cosmos/cosmos-sdk/x/auth/types" banktypes "github.com/cosmos/cosmos-sdk/x/bank/types" - proto "github.com/golang/protobuf/proto" + proto "github.com/golang/protobuf/proto" //nolint:staticcheck // we're intentionally using this deprecated package to be compatible with cosmos protos "github.com/stretchr/testify/suite" "github.com/tendermint/tendermint/crypto/ed25519" tmproto "github.com/tendermint/tendermint/proto/tendermint/types" @@ -84,7 +84,7 @@ func (suite *StargateTestSuite) TestStargateQuerier() { suite.NoError(err) }, requestData: func() []byte { - queryrequest := gammv2types.QuerySpotPriceRequest{ + queryrequest := gammv2types.QuerySpotPriceRequest{ //nolint:staticcheck // we're intentionally using this deprecated package for testing PoolId: 1, BaseAssetDenom: "bar", QuoteAssetDenom: "uosmo", @@ -94,7 +94,7 @@ func (suite *StargateTestSuite) TestStargateQuerier() { return bz }, checkResponseStruct: true, - responseProtoStruct: &gammv2types.QuerySpotPriceResponse{ + responseProtoStruct: &gammv2types.QuerySpotPriceResponse{ //nolint:staticcheck // we're intentionally using this deprecated package for testing SpotPrice: sdk.NewDecWithPrec(5, 1).String(), }, },