chore: bump golangci-lint to v1.54.2 (#17538)

cosmos · Aug 25, 2023 · 5839c3a · 5839c3a
1 parent 03fb7da
commit 5839c3a
Show file tree

Hide file tree

Showing 23 changed files with 50 additions and 58 deletions.
diff --git a/Makefile b/Makefile
@@ -369,7 +369,7 @@ benchmark:
 ###                                Linting                                  ###
 ###############################################################################
 
-golangci_version=v1.53.3
+golangci_version=v1.54.2
 
 lint-install:
 	@echo "--> Installing golangci-lint $(golangci_version)"

diff --git a/crypto/keys/secp256k1/internal/secp256k1/secp256_test.go b/crypto/keys/secp256k1/internal/secp256k1/secp256_test.go
@@ -24,8 +24,7 @@ func generateKeyPair() (pubkey, privkey []byte) {
 	if err != nil {
 		panic(err)
 	}
-	pubkey = elliptic.Marshal(S256(), key.X, key.Y)
-
+	pubkey = elliptic.Marshal(S256(), key.X, key.Y) //nolint:staticcheck // crypto will be refactored soon.
 	privkey = make([]byte, 32)
 	blob := key.D.Bytes()
 	copy(privkey[32-len(blob):], blob)

diff --git a/internal/conv/string.go b/internal/conv/string.go
@@ -1,20 +1,13 @@
 package conv
 
 import (
-	"reflect"
 	"unsafe"
 )
 
 // UnsafeStrToBytes uses unsafe to convert string into byte array. Returned bytes
 // must not be altered after this function is called as it will cause a segmentation fault.
 func UnsafeStrToBytes(s string) []byte {
-	var buf []byte
-	sHdr := (*reflect.StringHeader)(unsafe.Pointer(&s))
-	bufHdr := (*reflect.SliceHeader)(unsafe.Pointer(&buf))
-	bufHdr.Data = sHdr.Data
-	bufHdr.Cap = sHdr.Len
-	bufHdr.Len = sHdr.Len
-	return buf
+	return unsafe.Slice(unsafe.StringData(s), len(s)) // ref https://github.com/golang/go/issues/53003#issuecomment-1140276077
 }
 
 // UnsafeBytesToStr is meant to make a zero allocation conversion

diff --git a/internal/util.go b/internal/util.go
diff --git a/store/internal/conv/string.go b/store/internal/conv/string.go
@@ -1,20 +1,13 @@
 package conv
 
 import (
-	"reflect"
 	"unsafe"
 )
 
 // UnsafeStrToBytes uses unsafe to convert string into byte array. Returned bytes
 // must not be altered after this function is called as it will cause a segmentation fault.
 func UnsafeStrToBytes(s string) []byte {
-	var buf []byte
-	sHdr := (*reflect.StringHeader)(unsafe.Pointer(&s))
-	bufHdr := (*reflect.SliceHeader)(unsafe.Pointer(&buf))
-	bufHdr.Data = sHdr.Data
-	bufHdr.Cap = sHdr.Len
-	bufHdr.Len = sHdr.Len
-	return buf
+	return unsafe.Slice(unsafe.StringData(s), len(s)) // ref https://github.com/golang/go/issues/53003#issuecomment-1140276077
 }
 
 // UnsafeBytesToStr is meant to make a zero allocation conversion

diff --git a/tests/integration/staking/keeper/msg_server_test.go b/tests/integration/staking/keeper/msg_server_test.go
@@ -159,16 +159,18 @@ func TestCancelUnbondingDelegation(t *testing.T) {
 		},
 	}
 
-	for _, testCase := range testCases {
-		t.Run(testCase.name, func(t *testing.T) {
-			_, err := msgServer.CancelUnbondingDelegation(ctx, &testCase.req)
-			if testCase.exceptErr {
-				assert.ErrorContains(t, err, testCase.expErrMsg)
+	for _, tc := range testCases {
+		tc := tc
+
+		t.Run(tc.name, func(t *testing.T) {
+			_, err := msgServer.CancelUnbondingDelegation(ctx, &tc.req)
+			if tc.exceptErr {
+				assert.ErrorContains(t, err, tc.expErrMsg)
 			} else {
 				assert.NilError(t, err)
 				balanceForNotBondedPool := f.bankKeeper.GetBalance(ctx, notBondedPool.GetAddress(), bondDenom)
-				assert.DeepEqual(t, balanceForNotBondedPool, moduleBalance.Sub(testCase.req.Amount))
-				moduleBalance = moduleBalance.Sub(testCase.req.Amount)
+				assert.DeepEqual(t, balanceForNotBondedPool, moduleBalance.Sub(tc.req.Amount))
+				moduleBalance = moduleBalance.Sub(tc.req.Amount)
 			}
 		})
 	}

diff --git a/tools/cosmovisor/args_test.go b/tools/cosmovisor/args_test.go
@@ -764,6 +764,8 @@ func (s *argsTestSuite) TestGetConfigFromEnv() {
 	}
 
 	for _, tc := range tests {
+		tc := tc
+
 		s.T().Run(tc.name, func(t *testing.T) {
 			s.setEnv(t, &tc.envVals)
 			cfg, err := GetConfigFromEnv()

diff --git a/tools/cosmovisor/cmd/cosmovisor/init_test.go b/tools/cosmovisor/cmd/cosmovisor/init_test.go
@@ -325,6 +325,8 @@ func (s *InitTestSuite) TestInitializeCosmovisorNegativeValidation() {
 	}
 
 	for _, tc := range tests {
+		tc := tc
+
 		s.T().Run(tc.name, func(t *testing.T) {
 			s.setEnv(t, &tc.env)
 			buffer, logger := s.NewCapturingLogger()

diff --git a/x/auth/tx/query.go b/x/auth/tx/query.go
@@ -116,6 +116,8 @@ func getBlocksForTxResults(clientCtx client.Context, resTxs []*coretypes.ResultT
 	resBlocks := make(map[int64]*coretypes.ResultBlock)
 
 	for _, resTx := range resTxs {
+		resTx := resTx
+
 		if _, ok := resBlocks[resTx.Height]; !ok {
 			resBlock, err := node.Block(context.Background(), &resTx.Height)
 			if err != nil {

diff --git a/x/authz/keeper/grpc_query_test.go b/x/authz/keeper/grpc_query_test.go
@@ -261,6 +261,8 @@ func (suite *TestSuite) TestGRPCQueryGranteeGrants() {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
+
 		suite.Run(fmt.Sprintf("Case %s", tc.msg), func() {
 			tc.preRun()
 			result, err := queryClient.GranteeGrants(gocontext.Background(), &tc.request)

diff --git a/x/bank/simulation/genesis_test.go b/x/bank/simulation/genesis_test.go
@@ -79,6 +79,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/consensus/keeper/keeper_test.go b/x/consensus/keeper/keeper_test.go
@@ -114,6 +114,8 @@ func (s *KeeperTestSuite) TestGRPCQueryConsensusParams() {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
+
 		s.Run(tc.msg, func() {
 			s.SetupTest() // reset
 

diff --git a/x/distribution/simulation/genesis_test.go b/x/distribution/simulation/genesis_test.go
@@ -73,6 +73,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/gov/migrations/v3/convert_test.go b/x/gov/migrations/v3/convert_test.go
@@ -152,6 +152,8 @@ func TestConvertToLegacyTallyResult(t *testing.T) {
 		},
 	}
 	for name, tc := range testCases {
+		tc := tc
+
 		t.Run(name, func(t *testing.T) {
 			_, err := v3.ConvertToLegacyTallyResult(&tc.tallyResult)
 			if tc.expErr {

diff --git a/x/gov/simulation/genesis_test.go b/x/gov/simulation/genesis_test.go
@@ -91,6 +91,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/group/keeper/grpc_query_test.go b/x/group/keeper/grpc_query_test.go
@@ -119,6 +119,8 @@ func TestQueryGroupInfo(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
+
 		t.Run(tc.name, func(t *testing.T) {
 			_, err := fixture.queryClient.GroupInfo(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {
@@ -157,6 +159,7 @@ func TestQueryGroupPolicyInfo(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			_, err := fixture.queryClient.GroupPolicyInfo(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {
@@ -197,6 +200,7 @@ func TestQueryGroupMembers(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			resp, err := fixture.queryClient.GroupMembers(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {
@@ -242,6 +246,7 @@ func TestQueryGroupsByAdmin(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			resp, err := fixture.queryClient.GroupsByAdmin(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {
@@ -282,6 +287,7 @@ func TestQueryGroupPoliciesByGroup(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			resp, err := fixture.keeper.GroupPoliciesByGroup(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {
@@ -327,6 +333,7 @@ func TestQueryGroupPoliciesByAdmin(t *testing.T) {
 	}
 
 	for _, tc := range testCases {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 			resp, err := fixture.keeper.GroupPoliciesByAdmin(fixture.ctx, &tc.req)
 			if tc.expErrMsg != "" {

diff --git a/x/group/keeper/keeper.go b/x/group/keeper/keeper.go
@@ -373,6 +373,8 @@ func (k Keeper) PruneProposals(ctx sdk.Context) error {
 		return nil
 	}
 	for _, proposal := range proposals {
+		proposal := proposal
+
 		err := k.pruneProposal(ctx, proposal.Id)
 		if err != nil {
 			return err

diff --git a/x/mint/simulation/genesis_test.go b/x/mint/simulation/genesis_test.go
@@ -80,6 +80,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/nft/internal/conv/string.go b/x/nft/internal/conv/string.go
@@ -1,20 +1,13 @@
 package conv
 
 import (
-	"reflect"
 	"unsafe"
 )
 
 // UnsafeStrToBytes uses unsafe to convert string into byte array. Returned bytes
 // must not be altered after this function is called as it will cause a segmentation fault.
 func UnsafeStrToBytes(s string) []byte {
-	var buf []byte
-	sHdr := (*reflect.StringHeader)(unsafe.Pointer(&s))
-	bufHdr := (*reflect.SliceHeader)(unsafe.Pointer(&buf))
-	bufHdr.Data = sHdr.Data
-	bufHdr.Cap = sHdr.Len
-	bufHdr.Len = sHdr.Len
-	return buf
+	return unsafe.Slice(unsafe.StringData(s), len(s)) // ref https://github.com/golang/go/issues/53003#issuecomment-1140276077
 }
 
 // UnsafeBytesToStr is meant to make a zero allocation conversion

diff --git a/x/slashing/simulation/genesis_test.go b/x/slashing/simulation/genesis_test.go
@@ -79,6 +79,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/staking/simulation/genesis_test.go b/x/staking/simulation/genesis_test.go
@@ -105,6 +105,8 @@ func TestRandomizedGenState1(t *testing.T) {
 	}
 
 	for _, tt := range tests {
+		tt := tt
+
 		require.Panicsf(t, func() { simulation.RandomizedGenState(&tt.simState) }, tt.panicMsg)
 	}
 }
diff --git a/x/upgrade/internal/conv/string.go b/x/upgrade/internal/conv/string.go
@@ -1,18 +1,11 @@
 package conv
 
 import (
-	"reflect"
 	"unsafe"
 )
 
 // UnsafeStrToBytes uses unsafe to convert string into byte array. Returned bytes
 // must not be altered after this function is called as it will cause a segmentation fault.
 func UnsafeStrToBytes(s string) []byte {
-	var buf []byte
-	sHdr := (*reflect.StringHeader)(unsafe.Pointer(&s))
-	bufHdr := (*reflect.SliceHeader)(unsafe.Pointer(&buf))
-	bufHdr.Data = sHdr.Data
-	bufHdr.Cap = sHdr.Len
-	bufHdr.Len = sHdr.Len
-	return buf
+	return unsafe.Slice(unsafe.StringData(s), len(s)) // ref https://github.com/golang/go/issues/53003#issuecomment-1140276077
 }
diff --git a/x/upgrade/keeper/grpc_query_test.go b/x/upgrade/keeper/grpc_query_test.go
@@ -194,6 +194,8 @@ func (suite *UpgradeTestSuite) TestModuleVersions() {
 	suite.Require().NoError(err)
 
 	for _, tc := range testCases {
+		tc := tc
+
 		suite.Run(fmt.Sprintf("Case %s", tc.msg), func() {
 			suite.SetupTest() // reset