Unit tests

config/config_test.go

@@ -118,8 +118,10 @@ func TestGetDestinationRPCEndpoint(t *testing.T) {
 	}
 
 	for i, testCase := range testCases {
-		res := testCase.s.GetNodeRPCEndpoint()
-		require.Equal(t, testCase.expectedResult, res, fmt.Sprintf("test case %d failed", i))
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			res := testCase.s.GetNodeRPCEndpoint()
+			require.Equal(t, testCase.expectedResult, res)
+		})
 	}
 }
 
@@ -182,8 +184,10 @@ func TestGetSourceSubnetWSEndpoint(t *testing.T) {
 	}
 
 	for i, testCase := range testCases {
-		res := testCase.s.GetNodeWSEndpoint()
-		require.Equal(t, testCase.expectedResult, res, fmt.Sprintf("test case %d failed", i))
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			res := testCase.s.GetNodeWSEndpoint()
+			require.Equal(t, testCase.expectedResult, res)
+		})
 	}
 }
 
@@ -240,12 +244,14 @@ func TestGetRelayerAccountInfo(t *testing.T) {
 	}
 
 	for i, testCase := range testCases {
-		pk, addr, err := testCase.s.GetRelayerAccountInfo()
-		require.Equal(t, testCase.expectedResult.err, err, fmt.Sprintf("test case %d had unexpected error", i))
-		if err == nil {
-			require.Equal(t, testCase.expectedResult.pk.D.Int64(), pk.D.Int64(), fmt.Sprintf("test case %d had mismatched pk", i))
-			require.Equal(t, testCase.expectedResult.addr, addr, fmt.Sprintf("test case %d had mismatched address", i))
-		}
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			pk, addr, err := testCase.s.GetRelayerAccountInfo()
+			require.Equal(t, testCase.expectedResult.err, err)
+			if err == nil {
+				require.Equal(t, testCase.expectedResult.pk.D.Int64(), pk.D.Int64())
+				require.Equal(t, testCase.expectedResult.addr, addr)
+			}
+		})
 	}
 }
 
@@ -280,7 +286,7 @@ func runGetRelayerAccountPrivateKeyTest(t *testing.T, testCase getRelayerAccount
 	require.NoError(t, err)
 	require.Equal(t, optionOverwritten, testCase.expectedOverwritten)
 
-	require.True(t, testCase.resultVerifier(parsedCfg), "unexpected config")
+	require.True(t, testCase.resultVerifier(parsedCfg))
 }
 
 func TestGetRelayerAccountPrivateKey_set_pk_in_config(t *testing.T) {

utils/utils_test.go

@@ -4,6 +4,7 @@
 package utils
 
 import (
+	"fmt"
 	"math/big"
 	"testing"
 
@@ -50,14 +51,16 @@ func TestConvertProtocol(t *testing.T) {
 	}
 
 	for i, testCase := range testCases {
-		actualUrl, err := ConvertProtocol(testCase.urlString, testCase.protocol)
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			actualUrl, err := ConvertProtocol(testCase.urlString, testCase.protocol)
 
-		if testCase.expectedError {
-			require.Error(t, err, "Test case %d failed", i)
-		} else {
-			require.NoError(t, err, "Test case %d failed", i)
-		}
-		require.Equal(t, testCase.expectedUrl, actualUrl, "Test case %d failed", i)
+			if testCase.expectedError {
+				require.Error(t, err)
+			} else {
+				require.NoError(t, err)
+			}
+			require.Equal(t, testCase.expectedUrl, actualUrl)
+		})
 	}
 }
 
@@ -83,8 +86,10 @@ func TestSanitizeHashString(t *testing.T) {
 		},
 	}
 	for i, testCase := range testCases {
-		actualResult := SanitizeHashString(testCase.hash)
-		require.Equal(t, testCase.expectedResult, actualResult, "Test case %d failed", i)
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			actualResult := SanitizeHashString(testCase.hash)
+			require.Equal(t, testCase.expectedResult, actualResult)
+		})
 	}
 }
 
@@ -133,7 +138,9 @@ func TestCheckStakeWeightExceedsThreshold(t *testing.T) {
 		},
 	}
 	for i, testCase := range testCases {
-		actualResult := CheckStakeWeightExceedsThreshold(new(big.Int).SetUint64(testCase.accumulatedSignatureWeight), testCase.totalWeight, testCase.quorumNumerator, testCase.quorumDenominator)
-		require.Equal(t, testCase.expectedResult, actualResult, "Test case %d failed", i)
+		t.Run(fmt.Sprintf("test_%d", i), func(t *testing.T) {
+			actualResult := CheckStakeWeightExceedsThreshold(new(big.Int).SetUint64(testCase.accumulatedSignatureWeight), testCase.totalWeight, testCase.quorumNumerator, testCase.quorumDenominator)
+			require.Equal(t, testCase.expectedResult, actualResult)
+		})
 	}
 }