style: enable testifylint linter (#1198)

Signed-off-by: Jonathan Oppenheimer <147infiniti@gmail.com>
Signed-off-by: Jonathan Oppenheimer <jonathan.oppenheimer@avalabs.org>
Co-authored-by: Austin Larson <78000745+alarso16@users.noreply.github.com>
Co-authored-by: Stephen Buttolph <stephen@avalabs.org>
Co-authored-by: Ceyhun Onur <ceyhun.onur@avalabs.org>

Author: 4 people

.avalanche-golangci.yml

@@ -77,7 +77,7 @@ linters:
     - spancheck
     - staticcheck
     - tagalign
-    # - testifylint
+    - testifylint
     - unconvert
     - unparam
     - unused
@@ -208,16 +208,16 @@ linters:
         - serialize
       strict: true
     testifylint:
-    # Enable all checkers (https://github.com/Antonboom/testifylint#checkers).
-    # Default: false
-    enable-all: true
-    # Disable checkers by name
-    # (in addition to default
-    #   suite-thelper
-    # ).
-    disable:
-      - go-require
-      - float-compare
+      # Enable all checkers (https://github.com/Antonboom/testifylint#checkers).
+      # Default: false
+      enable-all: true
+      # Disable checkers by name
+      # (in addition to default
+      #   suite-thelper
+      # ).
+      disable:
+        - go-require
+        - float-compare
     unused:
       # Mark all struct fields that have been written to as used.
       # Default: true

accounts/abi/abi_extra_test.go

@@ -4,7 +4,6 @@
 package abi
 
 import (
-	"bytes"
 	"math/big"
 	"strings"
 	"testing"
@@ -82,14 +81,10 @@ func TestUnpackInputIntoInterface(t *testing.T) {
 				err = abi.UnpackInputIntoInterface(&v, "receive", data, test.strictMode) // skips 4 byte selector
 
 				if test.expectedErrorSubstring != "" {
-					require.Error(t, err)
 					require.ErrorContains(t, err, test.expectedErrorSubstring)
 				} else {
 					require.NoError(t, err)
-					// Verify unpacked values match input
-					require.Equal(t, v.Amount, input.Amount)
-					require.EqualValues(t, v.Amount, input.Amount)
-					require.True(t, bytes.Equal(v.Memo, input.Memo))
+					require.Equal(t, input, v)
 				}
 			})
 		}

accounts/abi/abi_test.go

@@ -1319,8 +1319,8 @@ func TestABI_PackEvent(t *testing.T) {
 				t.Fatal(err)
 			}
 
-			assert.EqualValues(t, test.expectedTopics, topics)
-			assert.EqualValues(t, test.expectedData, data)
+			assert.Equal(t, test.expectedTopics, topics)
+			assert.Equal(t, test.expectedData, data)
 		})
 	}
 }

core/genesis_extra_test.go

@@ -133,7 +133,7 @@ func TestGenesisToBlockDecoding(t *testing.T) {
 			readHeader := rawdb.ReadHeader(db, block.Hash(), 0)
 			require.Equal(t, block.Hash(), readHeader.Hash())
 			require.Equal(t, previousHashes[fork], block.Hash())
-			require.EqualValues(t, block.Header(), readHeader)
+			require.Equal(t, block.Header(), readHeader)
 		})
 	}
 }

core/state_manager_test.go

@@ -9,7 +9,7 @@ import (
 
 	"github.com/ava-labs/libevm/common"
 	"github.com/ava-labs/libevm/core/types"
-	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 // Default state history size
@@ -42,7 +42,7 @@ func TestCappedMemoryTrieWriter(t *testing.T) {
 	m := &MockTrieDB{}
 	cacheConfig := &CacheConfig{Pruning: true, CommitInterval: 4096, StateHistory: uint64(tipBufferSize)}
 	w := NewTrieWriter(m, cacheConfig)
-	assert := assert.New(t)
+	require := require.New(t)
 	for i := 0; i < int(cacheConfig.CommitInterval)+1; i++ {
 		bigI := big.NewInt(int64(i))
 		block := types.NewBlock(
@@ -53,35 +53,35 @@ func TestCappedMemoryTrieWriter(t *testing.T) {
 			nil, nil, nil, nil,
 		)
 
-		assert.NoError(w.InsertTrie(block))
-		assert.Equal(common.Hash{}, m.LastDereference, "should not have dereferenced block on insert")
-		assert.Equal(common.Hash{}, m.LastCommit, "should not have committed block on insert")
+		require.NoError(w.InsertTrie(block))
+		require.Zero(m.LastDereference, "should not have dereferenced block on insert")
+		require.Zero(m.LastCommit, "should not have committed block on insert")
 
 		w.AcceptTrie(block)
 		if i <= tipBufferSize {
-			assert.Equal(common.Hash{}, m.LastDereference, "should not have dereferenced block on accept")
+			require.Zero(m.LastDereference, "should not have dereferenced block on accept")
 		} else {
-			assert.Equal(common.BigToHash(big.NewInt(int64(i-tipBufferSize))), m.LastDereference, "should have dereferenced old block on last accept")
+			require.Equal(common.BigToHash(big.NewInt(int64(i-tipBufferSize))), m.LastDereference, "should have dereferenced old block on last accept")
 			m.LastDereference = common.Hash{}
 		}
 		if i < int(cacheConfig.CommitInterval) {
-			assert.Equal(common.Hash{}, m.LastCommit, "should not have committed block on accept")
+			require.Zero(m.LastCommit, "should not have committed block on accept")
 		} else {
-			assert.Equal(block.Root(), m.LastCommit, "should have committed block after CommitInterval")
+			require.Equal(block.Root(), m.LastCommit, "should have committed block after CommitInterval")
 			m.LastCommit = common.Hash{}
 		}
 
 		w.RejectTrie(block)
-		assert.Equal(block.Root(), m.LastDereference, "should have dereferenced block on reject")
-		assert.Equal(common.Hash{}, m.LastCommit, "should not have committed block on reject")
+		require.Equal(block.Root(), m.LastDereference, "should have dereferenced block on reject")
+		require.Zero(m.LastCommit, "should not have committed block on reject")
 		m.LastDereference = common.Hash{}
 	}
 }
 
 func TestNoPruningTrieWriter(t *testing.T) {
 	m := &MockTrieDB{}
 	w := NewTrieWriter(m, &CacheConfig{})
-	assert := assert.New(t)
+	require := require.New(t)
 	for i := 0; i < tipBufferSize+1; i++ {
 		bigI := big.NewInt(int64(i))
 		block := types.NewBlock(
@@ -92,18 +92,18 @@ func TestNoPruningTrieWriter(t *testing.T) {
 			nil, nil, nil, nil,
 		)
 
-		assert.NoError(w.InsertTrie(block))
-		assert.Equal(common.Hash{}, m.LastDereference, "should not have dereferenced block on insert")
-		assert.Equal(common.Hash{}, m.LastCommit, "should not have committed block on insert")
+		require.NoError(w.InsertTrie(block))
+		require.Zero(m.LastDereference, "should not have dereferenced block on insert")
+		require.Zero(m.LastCommit, "should not have committed block on insert")
 
 		w.AcceptTrie(block)
-		assert.Equal(common.Hash{}, m.LastDereference, "should not have dereferenced block on accept")
-		assert.Equal(block.Root(), m.LastCommit, "should have committed block on accept")
+		require.Zero(m.LastDereference, "should not have dereferenced block on accept")
+		require.Equal(block.Root(), m.LastCommit, "should have committed block on accept")
 		m.LastCommit = common.Hash{}
 
 		w.RejectTrie(block)
-		assert.Equal(block.Root(), m.LastDereference, "should have dereferenced block on reject")
-		assert.Equal(common.Hash{}, m.LastCommit, "should not have committed block on reject")
+		require.Equal(block.Root(), m.LastDereference, "should have dereferenced block on reject")
+		require.Zero(m.LastCommit, "should not have committed block on reject")
 		m.LastDereference = common.Hash{}
 	}
 }

eth/filters/filter_system_test.go

@@ -892,5 +892,5 @@ func TestGetLogsRegression(t *testing.T) {
 	test := FilterCriteria{BlockHash: &common.Hash{}, FromBlock: big.NewInt(rpc.LatestBlockNumber.Int64())}
 
 	_, err := api.GetLogs(context.Background(), test)
-	require.Error(t, err, "unknown block")
+	require.ErrorContains(t, err, "unknown block")
 }

go.mod

@@ -26,7 +26,7 @@ require (
 	github.com/spf13/cast v1.5.0
 	github.com/spf13/pflag v1.0.5
 	github.com/spf13/viper v1.12.0
-	github.com/stretchr/testify v1.10.0
+	github.com/stretchr/testify v1.11.1
 	github.com/tyler-smith/go-bip39 v1.1.0
 	github.com/urfave/cli/v2 v2.25.7
 	go.uber.org/goleak v1.3.0

go.sum

@@ -579,8 +579,8 @@ github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/
 github.com/stretchr/testify v1.7.2/go.mod h1:R6va5+xMeoiuVRoj+gSkQ7d3FALtqAAGI1FQKckRals=
 github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
 github.com/stretchr/testify v1.8.1/go.mod h1:w2LPCIKwWwSfY2zedu0+kehJoqGctiVI29o6fzry7u4=
-github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
-github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
+github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
+github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
 github.com/subosito/gotenv v1.3.0 h1:mjC+YW8QpAdXibNi+vNWgzmgBH4+5l5dCXv8cNysBLI=
 github.com/subosito/gotenv v1.3.0/go.mod h1:YzJjq/33h7nrwdY+iHMhEOEEbW0ovIz0tB6t6PwAXzs=
 github.com/supranational/blst v0.3.14 h1:xNMoHRJOTwMn63ip6qoWJ2Ymgvj7E2b9jY2FAwY+qRo=

nativeasset/contract_test.go

@@ -13,6 +13,7 @@ import (
 	"github.com/ava-labs/libevm/core/vm"
 	"github.com/holiman/uint256"
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 
 	// Force import core to register the VM hooks.
 	// This allows testing the precompiles by exercising the EVM.
@@ -48,11 +49,11 @@ func TestPackNativeAssetCallInput(t *testing.T) {
 	input := PackNativeAssetCallInput(addr, assetID, assetAmount, callData)
 
 	unpackedAddr, unpackedAssetID, unpackedAssetAmount, unpackedCallData, err := UnpackNativeAssetCallInput(input)
-	assert.NoError(t, err)
-	assert.Equal(t, addr, unpackedAddr, "address")
-	assert.Equal(t, assetID, unpackedAssetID, "assetID")
-	assert.Equal(t, assetAmount, unpackedAssetAmount, "assetAmount")
-	assert.Equal(t, callData, unpackedCallData, "callData")
+	require.NoError(t, err)
+	require.Equal(t, addr, unpackedAddr, "address")
+	require.Equal(t, assetID, unpackedAssetID, "assetID")
+	require.Equal(t, assetAmount, unpackedAssetAmount, "assetAmount")
+	require.Equal(t, callData, unpackedCallData, "callData")
 }
 
 func TestStatefulPrecompile(t *testing.T) {
@@ -254,10 +255,10 @@ func TestStatefulPrecompile(t *testing.T) {
 				user2AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr2, assetID)
 
 				expectedBalance := big.NewInt(50)
-				assert.Equal(t, u256Hundred, user1Balance, "user 1 balance")
-				assert.Equal(t, big0, user2Balance, "user 2 balance")
-				assert.Equal(t, expectedBalance, user1AssetBalance, "user 1 asset balance")
-				assert.Equal(t, expectedBalance, user2AssetBalance, "user 2 asset balance")
+				require.Equal(t, u256Hundred, user1Balance, "user 1 balance")
+				require.Equal(t, big0, user2Balance, "user 2 balance")
+				require.Equal(t, expectedBalance, user1AssetBalance, "user 1 asset balance")
+				require.Equal(t, expectedBalance, user2AssetBalance, "user 2 asset balance")
 			},
 		},
 		{
@@ -290,11 +291,11 @@ func TestStatefulPrecompile(t *testing.T) {
 				user2AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr2, assetID)
 				expectedBalance := big.NewInt(50)
 
-				assert.Equal(t, uint256.NewInt(51), user1Balance, "user 1 balance")
-				assert.Equal(t, big0, user2Balance, "user 2 balance")
-				assert.Equal(t, uint256.NewInt(49), nativeAssetCallAddrBalance, "native asset call addr balance")
-				assert.Equal(t, expectedBalance, user1AssetBalance, "user 1 asset balance")
-				assert.Equal(t, expectedBalance, user2AssetBalance, "user 2 asset balance")
+				require.Equal(t, uint256.NewInt(51), user1Balance, "user 1 balance")
+				require.Equal(t, big0, user2Balance, "user 2 balance")
+				require.Equal(t, uint256.NewInt(49), nativeAssetCallAddrBalance, "native asset call addr balance")
+				require.Equal(t, expectedBalance, user1AssetBalance, "user 1 asset balance")
+				require.Equal(t, expectedBalance, user2AssetBalance, "user 2 asset balance")
 			},
 		},
 		{
@@ -325,10 +326,10 @@ func TestStatefulPrecompile(t *testing.T) {
 				user1AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr1, assetID)
 				user2AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr2, assetID)
 
-				assert.Equal(t, bigHundred, user1Balance, "user 1 balance")
-				assert.Equal(t, big0, user2Balance, "user 2 balance")
-				assert.Equal(t, big.NewInt(51), user1AssetBalance, "user 1 asset balance")
-				assert.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
+				require.Equal(t, bigHundred, user1Balance, "user 1 balance")
+				require.Equal(t, big0, user2Balance, "user 2 balance")
+				require.Equal(t, big.NewInt(51), user1AssetBalance, "user 1 asset balance")
+				require.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
 			},
 		},
 		{
@@ -359,10 +360,10 @@ func TestStatefulPrecompile(t *testing.T) {
 				user1AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr1, assetID)
 				user2AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr2, assetID)
 
-				assert.Equal(t, big.NewInt(50), user1Balance, "user 1 balance")
-				assert.Equal(t, big0, user2Balance, "user 2 balance")
-				assert.Equal(t, big.NewInt(50), user1AssetBalance, "user 1 asset balance")
-				assert.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
+				require.Equal(t, big.NewInt(50), user1Balance, "user 1 balance")
+				require.Equal(t, big0, user2Balance, "user 2 balance")
+				require.Equal(t, big.NewInt(50), user1AssetBalance, "user 1 asset balance")
+				require.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
 			},
 		},
 		{
@@ -415,10 +416,10 @@ func TestStatefulPrecompile(t *testing.T) {
 				user1AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr1, assetID)
 				user2AssetBalance := wrappedStateDB.GetBalanceMultiCoin(userAddr2, assetID)
 
-				assert.Equal(t, bigHundred, user1Balance, "user 1 balance")
-				assert.Equal(t, big0, user2Balance, "user 2 balance")
-				assert.Equal(t, bigHundred, user1AssetBalance, "user 1 asset balance")
-				assert.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
+				require.Equal(t, bigHundred, user1Balance, "user 1 balance")
+				require.Equal(t, big0, user2Balance, "user 2 balance")
+				require.Equal(t, bigHundred, user1AssetBalance, "user 1 asset balance")
+				require.Equal(t, big0, user2AssetBalance, "user 2 asset balance")
 			},
 		},
 		{

network/network.go

@@ -36,6 +36,7 @@ const (
 var (
 	errAcquiringSemaphore                      = errors.New("error acquiring semaphore")
 	errExpiredRequest                          = errors.New("expired request")
+	errNoPeersFound                            = errors.New("no peers found matching version")
 	_                     Network              = (*network)(nil)
 	_                     validators.Connector = (*network)(nil)
 	_                     common.AppHandler    = (*network)(nil)
@@ -180,7 +181,7 @@ func (n *network) SendAppRequestAny(ctx context.Context, minVersion *version.App
 	}
 
 	n.activeAppRequests.Release(1)
-	return ids.EmptyNodeID, fmt.Errorf("no peers found matching version %s out of %d peers", minVersion, n.peers.Size())
+	return ids.EmptyNodeID, fmt.Errorf("%w: version: %s, numPeers: %d", errNoPeersFound, minVersion, n.peers.Size())
 }
 
 // SendAppRequest sends request message bytes to specified nodeID, notifying the responseHandler on response or failure