mirror of
https://gitlab.com/pulsechaincom/prysm-pulse.git
synced 2024-12-26 05:17:22 +00:00
addb3cd665
* Starting, looking for feedbacks * Update error.go * Wrap invalid blocks to rest of the processings * More tests * More tests * Fix tests * Update process_block_test.go * Update execution_engine_test.go * Update BUILD.bazel * Nishant's feedback and Kasey's recommendation * Add comments on what an invalid block is * Update beacon-chain/blockchain/error.go Co-authored-by: Potuz <potuz@prysmaticlabs.com> * Update beacon-chain/blockchain/error.go Co-authored-by: Potuz <potuz@prysmaticlabs.com> * Rm faulty invalid conditions Co-authored-by: Potuz <potuz@prysmaticlabs.com> Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
214 lines
7.1 KiB
Go
214 lines
7.1 KiB
Go
package blockchain
|
|
|
|
import (
|
|
"context"
|
|
"fmt"
|
|
"math/big"
|
|
"testing"
|
|
|
|
"github.com/holiman/uint256"
|
|
testDB "github.com/prysmaticlabs/prysm/beacon-chain/db/testing"
|
|
"github.com/prysmaticlabs/prysm/beacon-chain/forkchoice/protoarray"
|
|
mocks "github.com/prysmaticlabs/prysm/beacon-chain/powchain/testing"
|
|
"github.com/prysmaticlabs/prysm/beacon-chain/state/stategen"
|
|
fieldparams "github.com/prysmaticlabs/prysm/config/fieldparams"
|
|
"github.com/prysmaticlabs/prysm/config/params"
|
|
"github.com/prysmaticlabs/prysm/consensus-types/wrapper"
|
|
"github.com/prysmaticlabs/prysm/encoding/bytesutil"
|
|
enginev1 "github.com/prysmaticlabs/prysm/proto/engine/v1"
|
|
ethpb "github.com/prysmaticlabs/prysm/proto/prysm/v1alpha1"
|
|
"github.com/prysmaticlabs/prysm/testing/require"
|
|
)
|
|
|
|
func Test_validTerminalPowBlock(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
currentDifficulty *uint256.Int
|
|
parentDifficulty *uint256.Int
|
|
ttd uint64
|
|
want bool
|
|
}{
|
|
{
|
|
name: "current > ttd, parent > ttd",
|
|
currentDifficulty: uint256.NewInt(2),
|
|
parentDifficulty: uint256.NewInt(2),
|
|
ttd: 1,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "current < ttd, parent < ttd",
|
|
currentDifficulty: uint256.NewInt(2),
|
|
parentDifficulty: uint256.NewInt(2),
|
|
ttd: 3,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "current == ttd, parent == ttd",
|
|
currentDifficulty: uint256.NewInt(2),
|
|
parentDifficulty: uint256.NewInt(2),
|
|
ttd: 2,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "current > ttd, parent == ttd",
|
|
currentDifficulty: uint256.NewInt(2),
|
|
parentDifficulty: uint256.NewInt(1),
|
|
ttd: 1,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "current == ttd, parent < ttd",
|
|
currentDifficulty: uint256.NewInt(2),
|
|
parentDifficulty: uint256.NewInt(1),
|
|
ttd: 2,
|
|
want: true,
|
|
},
|
|
{
|
|
name: "current > ttd, parent < ttd",
|
|
currentDifficulty: uint256.NewInt(3),
|
|
parentDifficulty: uint256.NewInt(1),
|
|
ttd: 2,
|
|
want: true,
|
|
},
|
|
}
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
cfg := params.BeaconConfig()
|
|
cfg.TerminalTotalDifficulty = fmt.Sprint(tt.ttd)
|
|
params.OverrideBeaconConfig(cfg)
|
|
got, err := validateTerminalBlockDifficulties(tt.currentDifficulty, tt.parentDifficulty)
|
|
require.NoError(t, err)
|
|
if got != tt.want {
|
|
t.Errorf("validateTerminalBlockDifficulties() = %v, want %v", got, tt.want)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_validTerminalPowBlockSpecConfig(t *testing.T) {
|
|
cfg := params.BeaconConfig()
|
|
cfg.TerminalTotalDifficulty = "115792089237316195423570985008687907853269984665640564039457584007913129638912"
|
|
params.OverrideBeaconConfig(cfg)
|
|
|
|
i, _ := new(big.Int).SetString("115792089237316195423570985008687907853269984665640564039457584007913129638912", 10)
|
|
current, of := uint256.FromBig(i)
|
|
require.Equal(t, of, false)
|
|
i, _ = new(big.Int).SetString("115792089237316195423570985008687907853269984665640564039457584007913129638911", 10)
|
|
parent, of := uint256.FromBig(i)
|
|
require.Equal(t, of, false)
|
|
|
|
got, err := validateTerminalBlockDifficulties(current, parent)
|
|
require.NoError(t, err)
|
|
require.Equal(t, true, got)
|
|
}
|
|
|
|
func Test_validateMergeBlock(t *testing.T) {
|
|
cfg := params.BeaconConfig()
|
|
cfg.TerminalTotalDifficulty = "2"
|
|
params.OverrideBeaconConfig(cfg)
|
|
|
|
ctx := context.Background()
|
|
beaconDB := testDB.SetupDB(t)
|
|
fcs := protoarray.New(0, 0, [32]byte{'a'})
|
|
opts := []Option{
|
|
WithDatabase(beaconDB),
|
|
WithStateGen(stategen.New(beaconDB)),
|
|
WithForkChoiceStore(fcs),
|
|
}
|
|
service, err := NewService(ctx, opts...)
|
|
require.NoError(t, err)
|
|
|
|
engine := &mocks.EngineClient{BlockByHashMap: map[[32]byte]*enginev1.ExecutionBlock{}}
|
|
service.cfg.ExecutionEngineCaller = engine
|
|
engine.BlockByHashMap[[32]byte{'a'}] = &enginev1.ExecutionBlock{
|
|
ParentHash: bytesutil.PadTo([]byte{'b'}, fieldparams.RootLength),
|
|
TotalDifficulty: "0x2",
|
|
}
|
|
engine.BlockByHashMap[[32]byte{'b'}] = &enginev1.ExecutionBlock{
|
|
ParentHash: bytesutil.PadTo([]byte{'3'}, fieldparams.RootLength),
|
|
TotalDifficulty: "0x1",
|
|
}
|
|
blk := ðpb.SignedBeaconBlockBellatrix{
|
|
Block: ðpb.BeaconBlockBellatrix{
|
|
Slot: 1,
|
|
Body: ðpb.BeaconBlockBodyBellatrix{
|
|
ExecutionPayload: &enginev1.ExecutionPayload{
|
|
ParentHash: bytesutil.PadTo([]byte{'a'}, fieldparams.RootLength),
|
|
},
|
|
},
|
|
},
|
|
}
|
|
b, err := wrapper.WrappedSignedBeaconBlock(blk)
|
|
require.NoError(t, err)
|
|
require.NoError(t, service.validateMergeBlock(ctx, b))
|
|
|
|
cfg.TerminalTotalDifficulty = "1"
|
|
params.OverrideBeaconConfig(cfg)
|
|
err = service.validateMergeBlock(ctx, b)
|
|
require.ErrorContains(t, "invalid TTD, configTTD: 1, currentTTD: 2, parentTTD: 1", err)
|
|
require.Equal(t, true, IsInvalidBlock(err))
|
|
}
|
|
|
|
func Test_getBlkParentHashAndTD(t *testing.T) {
|
|
ctx := context.Background()
|
|
beaconDB := testDB.SetupDB(t)
|
|
fcs := protoarray.New(0, 0, [32]byte{'a'})
|
|
opts := []Option{
|
|
WithDatabase(beaconDB),
|
|
WithStateGen(stategen.New(beaconDB)),
|
|
WithForkChoiceStore(fcs),
|
|
}
|
|
service, err := NewService(ctx, opts...)
|
|
require.NoError(t, err)
|
|
|
|
engine := &mocks.EngineClient{BlockByHashMap: map[[32]byte]*enginev1.ExecutionBlock{}}
|
|
service.cfg.ExecutionEngineCaller = engine
|
|
h := [32]byte{'a'}
|
|
p := [32]byte{'b'}
|
|
td := "0x1"
|
|
engine.BlockByHashMap[h] = &enginev1.ExecutionBlock{
|
|
ParentHash: p[:],
|
|
TotalDifficulty: td,
|
|
}
|
|
parentHash, totalDifficulty, err := service.getBlkParentHashAndTD(ctx, h[:])
|
|
require.NoError(t, err)
|
|
require.Equal(t, p, bytesutil.ToBytes32(parentHash))
|
|
require.Equal(t, td, totalDifficulty.String())
|
|
|
|
_, _, err = service.getBlkParentHashAndTD(ctx, []byte{'c'})
|
|
require.ErrorContains(t, "could not get pow block: block not found", err)
|
|
|
|
engine.BlockByHashMap[h] = nil
|
|
_, _, err = service.getBlkParentHashAndTD(ctx, h[:])
|
|
require.ErrorContains(t, "pow block is nil", err)
|
|
|
|
engine.BlockByHashMap[h] = &enginev1.ExecutionBlock{
|
|
ParentHash: p[:],
|
|
TotalDifficulty: "1",
|
|
}
|
|
_, _, err = service.getBlkParentHashAndTD(ctx, h[:])
|
|
require.ErrorContains(t, "could not decode merge block total difficulty: hex string without 0x prefix", err)
|
|
|
|
engine.BlockByHashMap[h] = &enginev1.ExecutionBlock{
|
|
ParentHash: p[:],
|
|
TotalDifficulty: "0XFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF",
|
|
}
|
|
_, _, err = service.getBlkParentHashAndTD(ctx, h[:])
|
|
require.ErrorContains(t, "could not decode merge block total difficulty: hex number > 256 bits", err)
|
|
}
|
|
|
|
func Test_validateTerminalBlockHash(t *testing.T) {
|
|
require.NoError(t, validateTerminalBlockHash(1, &enginev1.ExecutionPayload{}))
|
|
|
|
cfg := params.BeaconConfig()
|
|
cfg.TerminalBlockHash = [32]byte{0x01}
|
|
params.OverrideBeaconConfig(cfg)
|
|
require.ErrorContains(t, "terminal block hash activation epoch not reached", validateTerminalBlockHash(1, &enginev1.ExecutionPayload{}))
|
|
|
|
cfg.TerminalBlockHashActivationEpoch = 0
|
|
params.OverrideBeaconConfig(cfg)
|
|
require.ErrorContains(t, "parent hash does not match terminal block hash", validateTerminalBlockHash(1, &enginev1.ExecutionPayload{}))
|
|
|
|
require.NoError(t, validateTerminalBlockHash(1, &enginev1.ExecutionPayload{ParentHash: cfg.TerminalBlockHash.Bytes()}))
|
|
}
|