test: check extra API

Author: alarso16

internal/ethapi/api_extra_test.go

@@ -4,18 +4,86 @@
 package ethapi
 
 import (
+	"context"
 	"errors"
 	"math/big"
 	"testing"
 
 	"github.com/ava-labs/libevm/common"
+	"github.com/ava-labs/libevm/common/hexutil"
 	"github.com/ava-labs/libevm/core/types"
+	ethparams "github.com/ava-labs/libevm/params"
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 	"go.uber.org/mock/gomock"
 
+	"github.com/ava-labs/coreth/consensus/dummy"
+	"github.com/ava-labs/coreth/core"
+	"github.com/ava-labs/coreth/params"
 	"github.com/ava-labs/coreth/rpc"
 )
 
+func TestBlockchainAPI_GetChainConfig(t *testing.T) {
+	t.Parallel()
+
+	ctrl := gomock.NewController(t)
+	defer ctrl.Finish()
+
+	wantConfig := &params.ChainConfig{
+		ChainID: big.NewInt(43114),
+	}
+	backend := NewMockBackend(ctrl)
+	backend.EXPECT().ChainConfig().Return(wantConfig)
+
+	api := NewBlockChainAPI(backend)
+
+	gotConfig := api.GetChainConfig(context.Background())
+	assert.Equal(t, wantConfig, gotConfig)
+}
+
+// Copy one test case from TestCall
+func TestBlockchainAPI_CallDetailed(t *testing.T) {
+	t.Parallel()
+	// Initialize test accounts
+	var (
+		accounts = newAccounts(2)
+		genesis  = &core.Genesis{
+			Config: params.TestChainConfig,
+			Alloc: types.GenesisAlloc{
+				accounts[0].addr: {Balance: big.NewInt(params.Ether)},
+				accounts[1].addr: {Balance: big.NewInt(params.Ether)},
+			},
+		}
+		genBlocks   = 10
+		signer      = types.HomesteadSigner{}
+		blockNumber = rpc.LatestBlockNumber
+	)
+	api := NewBlockChainAPI(newTestBackend(t, genBlocks, genesis, dummy.NewCoinbaseFaker(), func(i int, b *core.BlockGen) {
+		// Transfer from account[0] to account[1]
+		//    value: 1000 wei
+		//    fee:   0 wei
+		tx, _ := types.SignTx(types.NewTx(&types.LegacyTx{Nonce: uint64(i), To: &accounts[1].addr, Value: big.NewInt(1000), Gas: ethparams.TxGas, GasPrice: b.BaseFee(), Data: nil}), signer, accounts[0].key)
+		b.AddTx(tx)
+	}))
+
+	result, err := api.CallDetailed(
+		context.Background(),
+		TransactionArgs{
+			From:  &accounts[0].addr,
+			To:    &accounts[1].addr,
+			Value: (*hexutil.Big)(big.NewInt(1000)),
+		},
+		rpc.BlockNumberOrHash{BlockNumber: &blockNumber},
+		nil,
+	)
+	require.NoError(t, err)
+	require.NotNil(t, result)
+	require.Equal(t, 0, result.ErrCode)
+	require.Nil(t, result.ReturnData)
+	require.Equal(t, ethparams.TxGas, result.UsedGas)
+	require.Empty(t, result.Err)
+}
+
 func TestBlockChainAPI_stateQueryBlockNumberAllowed(t *testing.T) {
 	t.Parallel()
 

precompile/contracts/warp/predicate_test.go

@@ -193,7 +193,7 @@ func createSnowCtx(tb testing.TB, validatorRanges []validatorRange) *snow.Contex
 		GetSubnetIDF: func(context.Context, ids.ID) (ids.ID, error) {
 			return sourceSubnetID, nil
 		},
-		GetValidatorSetF: func(_ context.Context, _ uint64, _ ids.ID) (map[ids.NodeID]*validators.GetValidatorOutput, error) {
+		GetValidatorSetF: func(context.Context, uint64, ids.ID) (map[ids.NodeID]*validators.GetValidatorOutput, error) {
 			return getValidatorsOutput, nil
 		},
 	}
@@ -669,10 +669,10 @@ func makeWarpPredicateTests(tb testing.TB) map[string]precompiletest.PredicateTe
 
 		snowCtx := snowtest.Context(tb, snowtest.CChainID)
 		state := &validatorstest.State{
-			GetSubnetIDF: func(_ context.Context, _ ids.ID) (ids.ID, error) {
+			GetSubnetIDF: func(context.Context, ids.ID) (ids.ID, error) {
 				return sourceSubnetID, nil
 			},
-			GetValidatorSetF: func(_ context.Context, _ uint64, _ ids.ID) (map[ids.NodeID]*validators.GetValidatorOutput, error) {
+			GetValidatorSetF: func(context.Context, uint64, ids.ID) (map[ids.NodeID]*validators.GetValidatorOutput, error) {
 				return getValidatorsOutput, nil
 			},
 		}