more tests, fix lint

Author: skudasov

chain/evm/provider/confirm_functor.go

@@ -34,6 +34,7 @@ func ConfirmFuncGeth(waitMinedTimeout time.Duration, opts ...func(*confirmFuncGe
 	for _, o := range opts {
 		o(cf)
 	}
+
 	return cf
 }
 

chain/evm/provider/confirm_functor_test.go

@@ -39,39 +39,47 @@ func Test_ConfirmFuncGeth_ConfirmFunc(t *testing.T) {
 		adminTransactor.From: {Balance: prefundAmountWei},
 	}
 
-	tests := []struct {
-		name    string
-		giveTx  func(*testing.T, *SimClient) *types.Transaction
-		wantErr string
-	}{
-		{
-			name: "successful confirmation",
-			giveTx: func(t *testing.T, client *SimClient) *types.Transaction {
-				t.Helper()
+	defaultConfirmFunc := func(t *testing.T, client *SimClient) *types.Transaction {
+		t.Helper()
 
-				// Get the nonce
-				nonce, err := client.PendingNonceAt(t.Context(), adminTransactor.From)
-				require.NoError(t, err)
+		// Get the nonce
+		nonce, err := client.PendingNonceAt(t.Context(), adminTransactor.From)
+		require.NoError(t, err)
 
-				gasPrice, err := client.SuggestGasPrice(t.Context())
-				require.NoError(t, err)
+		gasPrice, err := client.SuggestGasPrice(t.Context())
+		require.NoError(t, err)
 
-				// Create a transaction to send tokens. This will be used to test the confirmation function.
-				tx := types.NewTransaction(
-					nonce, userTransactor.From, big.NewInt(10000000000000000), 21000, gasPrice, nil,
-				)
+		// Create a transaction to send tokens. This will be used to test the confirmation function.
+		tx := types.NewTransaction(
+			nonce, userTransactor.From, big.NewInt(10000000000000000), 21000, gasPrice, nil,
+		)
 
-				signedTx, err := types.SignTx(tx, types.NewCancunSigner(simChainID), adminKey)
-				require.NoError(t, err, "failed to sign transaction")
+		signedTx, err := types.SignTx(tx, types.NewCancunSigner(simChainID), adminKey)
+		require.NoError(t, err, "failed to sign transaction")
 
-				// Send the transaction
-				err = client.SendTransaction(t.Context(), signedTx)
-				require.NoError(t, err)
+		// Send the transaction
+		err = client.SendTransaction(t.Context(), signedTx)
+		require.NoError(t, err)
 
-				client.Commit() // Commit the transaction to the simulated backend
+		client.Commit() // Commit the transaction to the simulated backend
 
-				return signedTx
-			},
+		return signedTx
+	}
+
+	tests := []struct {
+		name          string
+		giveTx        func(*testing.T, *SimClient) *types.Transaction
+		confirmerOpts []func(*confirmFuncGeth)
+		wantErr       string
+	}{
+		{
+			name:   "successful confirmation",
+			giveTx: defaultConfirmFunc,
+		},
+		{
+			name:          "successful confirmation with custom WaitMined ticker",
+			confirmerOpts: []func(*confirmFuncGeth){WithTickInterval(10 * time.Millisecond)},
+			giveTx:        defaultConfirmFunc,
 		},
 		{
 			name: "failed with nil tx",
@@ -121,7 +129,7 @@ func Test_ConfirmFuncGeth_ConfirmFunc(t *testing.T) {
 			tx := tt.giveTx(t, client)
 
 			// Generate the confirm function
-			functor := ConfirmFuncGeth(1 * time.Second)
+			functor := ConfirmFuncGeth(1*time.Second, tt.confirmerOpts...)
 			confirmFunc, err := functor.Generate(
 				t.Context(), chainsel.TEST_1000.Selector, client, adminTransactor.From,
 			)

chain/evm/provider/ctf_anvil_provider_test.go

@@ -295,15 +295,15 @@ func TestCTFAnvilChainProvider_InitializeErrors(t *testing.T) {
 
 func TestCTFAnvilChainProvider_SignerIntegration(t *testing.T) {
 	t.Parallel()
-	
+
 	t.Run("custom tick interval", func(t *testing.T) {
 		t.Parallel()
 		customKey := "ac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80"
 
 		var once sync.Once
 		config := CTFAnvilChainProviderConfig{
 			Once:                  &once,
-			ConfirmFunctor:        ConfirmFuncGeth(2 * time.Minute, WithTickInterval(100*time.Millisecond)),
+			ConfirmFunctor:        ConfirmFuncGeth(2*time.Minute, WithTickInterval(100*time.Millisecond)),
 			DeployerTransactorGen: TransactorFromRaw(customKey),
 			T:                     t,
 		}

engine/cld/legacy/cli/commands/template_input.go

@@ -31,14 +31,15 @@ changesets:
 `, envKey, domainName)
 
 	// Generate each changeset section
+	var yamlTemplateSb34 strings.Builder
 	for i, changesetName := range changesetNames {
 		if changesetName == "" {
 			continue
 		}
 
 		// Add separator between changesets
 		if i > 0 {
-			yamlTemplate += "\n  # ----------------------------------------\n"
+			yamlTemplateSb34.WriteString("\n  # ----------------------------------------\n")
 		}
 
 		// Get changeset configuration
@@ -53,8 +54,9 @@ changesets:
 			return "", fmt.Errorf("generate section for changeset %s: %w", changesetName, err)
 		}
 
-		yamlTemplate += changesetSection
+		yamlTemplateSb34.WriteString(changesetSection)
 	}
+	yamlTemplate += yamlTemplateSb34.String()
 
 	return yamlTemplate, nil
 }