Get private key of batch authenticator owner from .env.

Author: philippecamacho

espresso/.env

@@ -37,6 +37,8 @@ OPERATOR_ADDRESS=0xf39Fd6e51aad88F6F4ce6aB8827279cffFb92266
 # BatchAuthenticator owner address for contract ownership (index 3 from mnemonic)
 # cast wallet address --mnemonic "test test ... junk" --hd-path "m/44'/60'/0'/0/3"
 BATCH_AUTHENTICATOR_OWNER_ADDRESS=0x90F79bf6EB2c4f870365E785982E1f101E93b906
+# cast wallet private-key --mnemonic "test test ... junk" --hd-path "m/44'/60'/0'/0/3"
+BATCH_AUTHENTICATOR_OWNER_PRIVATE_KEY=0x7c852118294e51e653712a81e05800f419141751be58f605c371e15141b007a6
 
 # cast wallet address --mnemonic "test test ... junk" --hd-path "m/44'/60'/0'/0/1"
 PROPOSER_ADDRESS=0x70997970C51812dc3A010C7d01b50e0d17dc79C8

espresso/devnet-tests/devnet_tools.go

@@ -9,6 +9,7 @@ import (
 	"math/big"
 	"os"
 	"os/exec"
+	"path/filepath"
 	"reflect"
 	"strconv"
 	"strings"
@@ -27,6 +28,7 @@ import (
 	"github.com/ethereum/go-ethereum/crypto"
 	"github.com/ethereum/go-ethereum/ethclient"
 	"github.com/ethereum/go-ethereum/log"
+	"github.com/joho/godotenv"
 
 	env "github.com/ethereum-optimism/optimism/espresso/environment"
 	"github.com/ethereum-optimism/optimism/op-e2e/config/secrets"
@@ -44,6 +46,18 @@ type Devnet struct {
 	L2VerifRollup *sources.RollupClient
 }
 
+// LoadEnvFile loads environment variables from a .env file
+func LoadEnvFile(filename string) error {
+	return godotenv.Load(filename)
+}
+
+// LoadDevnetEnv loads the espresso/.env file for devnet tests
+func LoadDevnetEnv() error {
+	// Get the path to the espresso/.env file relative to the test directory
+	envPath := filepath.Join("..", ".env")
+	return LoadEnvFile(envPath)
+}
+
 func NewDevnet(ctx context.Context, t *testing.T) *Devnet {
 
 	if testing.Short() {

espresso/devnet-tests/key_rotation_test.go

@@ -2,16 +2,20 @@ package devnet_tests
 
 import (
 	"context"
+	"os"
+	"strings"
 	"testing"
 
 	"github.com/ethereum-optimism/optimism/op-batcher/bindings"
 	"github.com/ethereum-optimism/optimism/op-e2e/e2eutils/wait"
 	"github.com/ethereum-optimism/optimism/op-service/eth"
 	"github.com/ethereum/go-ethereum/accounts/abi/bind"
+	"github.com/ethereum/go-ethereum/crypto"
 	"github.com/stretchr/testify/require"
 )
 
 func TestRotateBatcherKey(t *testing.T) {
+
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
@@ -53,6 +57,10 @@ func TestRotateBatcherKey(t *testing.T) {
 }
 
 func TestChangeBatchInboxOwner(t *testing.T) {
+	// Load environment variables from .env file
+	err := LoadDevnetEnv()
+	require.NoError(t, err, "Failed to load .env file")
+
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
@@ -80,36 +88,34 @@ func TestChangeBatchInboxOwner(t *testing.T) {
 	// Check current owner first
 	currentOwner, err := batchAuthenticator.Owner(&bind.CallOpts{})
 	require.NoError(t, err)
-	t.Logf("Current BatchAuthenticator owner: %s", currentOwner.Hex())
-	t.Logf("Deployer address: %s", d.secrets.Addresses().Deployer.Hex())
-	t.Logf("Bob address: %s", d.secrets.Addresses().Bob.Hex())
 
-	// Use deployer key to sign the transaction
-	deployerOpts, err := bind.NewKeyedTransactorWithChainID(d.secrets.Deployer, l1ChainID)
+	// Check that the new owner is different from the current one
+	bobAddress := d.secrets.Addresses().Bob
+	require.NotEqual(t, currentOwner, bobAddress)
+
+	// Use batch authenticator owner key to sign the transaction
+	batchAuthenticatorPrivateKeyHex := os.Getenv("BATCH_AUTHENTICATOR_OWNER_PRIVATE_KEY")
+	require.NotEmpty(t, batchAuthenticatorPrivateKeyHex, "BATCH_AUTHENTICATOR_OWNER_PRIVATE_KEY must be set")
+	t.Logf("Using BATCH_AUTHENTICATOR_OWNER_PRIVATE_KEY from environment: %s...", batchAuthenticatorPrivateKeyHex[:10])
+
+	batchAuthenticatorKey, err := crypto.HexToECDSA(strings.TrimPrefix(batchAuthenticatorPrivateKeyHex, "0x"))
 	require.NoError(t, err)
 
-	// Verify we're transferring to the right address
-	bobAddress := d.secrets.Addresses().Bob
-	t.Logf("Attempting to transfer ownership from %s to %s", currentOwner.Hex(), bobAddress.Hex())
+	batchAuthenticatorOwnerOpts, err := bind.NewKeyedTransactorWithChainID(batchAuthenticatorKey, l1ChainID)
+	require.NoError(t, err)
 
 	// Call TransferOwnership
-	tx, err := batchAuthenticator.TransferOwnership(deployerOpts, bobAddress)
+	tx, err := batchAuthenticator.TransferOwnership(batchAuthenticatorOwnerOpts, bobAddress)
 	require.NoError(t, err)
-	t.Logf("TransferOwnership transaction hash: %s", tx.Hash().Hex())
 
 	// Wait for transaction receipt and check if it succeeded
-	receipt, err := wait.ForReceiptOK(ctx, d.L1, tx.Hash())
+	_, err = wait.ForReceiptOK(ctx, d.L1, tx.Hash())
 	require.NoError(t, err)
-	t.Logf("TransferOwnership transaction receipt status: %d", receipt.Status)
-	t.Logf("TransferOwnership transaction gas used: %d", receipt.GasUsed)
-	t.Logf("TransferOwnership transaction completed successfully")
 
 	// Ensure the owner has been changed
 	newOwner, err := batchAuthenticator.Owner(&bind.CallOpts{})
 	require.NoError(t, err)
-	t.Logf("New owner after transfer: %s", newOwner.Hex())
-	t.Logf("Expected Bob address: %s", d.secrets.Addresses().Bob.Hex())
-	require.Equal(t, newOwner, d.secrets.Addresses().Bob)
+	require.Equal(t, newOwner, bobAddress)
 
 	// Check that everything still functions
 	require.NoError(t, d.RunSimpleL2Burn())

go.mod

@@ -79,6 +79,8 @@ require (
 	gopkg.in/yaml.v3 v3.0.1
 )
 
+require github.com/joho/godotenv v1.5.1 // indirect
+
 require (
 	codeberg.org/go-fonts/liberation v0.5.0 // indirect
 	codeberg.org/go-latex/latex v0.1.0 // indirect

go.sum

@@ -473,6 +473,8 @@ github.com/jbenet/goprocess v0.1.4/go.mod h1:5yspPrukOVuOLORacaBi858NqyClJPQxYZl
 github.com/jellevandenhooff/dkim v0.0.0-20150330215556-f50fe3d243e1/go.mod h1:E0B/fFc00Y+Rasa88328GlI/XbtyysCtTHZS8h7IrBU=
 github.com/jessevdk/go-flags v0.0.0-20141203071132-1679536dcc89/go.mod h1:4FA24M0QyGHXBuZZK/XkWh8h0e1EYbRYJSGM75WSRxI=
 github.com/jessevdk/go-flags v1.4.0/go.mod h1:4FA24M0QyGHXBuZZK/XkWh8h0e1EYbRYJSGM75WSRxI=
+github.com/joho/godotenv v1.5.1 h1:7eLL/+HRGLY0ldzfGMeQkb7vMd0as4CfYvUVzLqw0N0=
+github.com/joho/godotenv v1.5.1/go.mod h1:f4LDr5Voq0i2e/R5DDNOoa2zzDfwtkZa6DnEwAbqwq4=
 github.com/josharian/intern v1.0.0/go.mod h1:5DoeVV0s6jJacbCEi61lwdGj/aVlrQvzHFFd8Hwg//Y=
 github.com/jpillora/backoff v1.0.0/go.mod h1:J/6gKK9jxlEcS3zixgDgUAsiuZ7yrSoa/FX5e0EB2j4=
 github.com/jrick/logrotate v1.0.0/go.mod h1:LNinyqDIJnpAur+b8yyulnQw/wDuN1+BYKlTRt3OuAQ=