mirror of
https://gitlab.com/pulsechaincom/prysm-pulse.git
synced 2024-12-22 03:30:35 +00:00
5a66807989
* First take at updating everything to v5 * Patch gRPC gateway to use prysm v5 Fix patch * Update go ssz --------- Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com>
330 lines
11 KiB
Go
330 lines
11 KiB
Go
package sync
|
|
|
|
import (
|
|
"bytes"
|
|
"context"
|
|
"fmt"
|
|
"testing"
|
|
"time"
|
|
|
|
pubsub "github.com/libp2p/go-libp2p-pubsub"
|
|
pubsubpb "github.com/libp2p/go-libp2p-pubsub/pb"
|
|
"github.com/prysmaticlabs/go-bitfield"
|
|
mockChain "github.com/prysmaticlabs/prysm/v5/beacon-chain/blockchain/testing"
|
|
"github.com/prysmaticlabs/prysm/v5/beacon-chain/core/helpers"
|
|
"github.com/prysmaticlabs/prysm/v5/beacon-chain/core/signing"
|
|
dbtest "github.com/prysmaticlabs/prysm/v5/beacon-chain/db/testing"
|
|
p2ptest "github.com/prysmaticlabs/prysm/v5/beacon-chain/p2p/testing"
|
|
"github.com/prysmaticlabs/prysm/v5/beacon-chain/startup"
|
|
mockSync "github.com/prysmaticlabs/prysm/v5/beacon-chain/sync/initial-sync/testing"
|
|
lruwrpr "github.com/prysmaticlabs/prysm/v5/cache/lru"
|
|
fieldparams "github.com/prysmaticlabs/prysm/v5/config/fieldparams"
|
|
"github.com/prysmaticlabs/prysm/v5/config/params"
|
|
"github.com/prysmaticlabs/prysm/v5/encoding/bytesutil"
|
|
ethpb "github.com/prysmaticlabs/prysm/v5/proto/prysm/v1alpha1"
|
|
"github.com/prysmaticlabs/prysm/v5/testing/require"
|
|
"github.com/prysmaticlabs/prysm/v5/testing/util"
|
|
)
|
|
|
|
func TestService_validateCommitteeIndexBeaconAttestation(t *testing.T) {
|
|
p := p2ptest.NewTestP2P(t)
|
|
db := dbtest.SetupDB(t)
|
|
chain := &mockChain.ChainService{
|
|
// 1 slot ago.
|
|
Genesis: time.Now().Add(time.Duration(-1*int64(params.BeaconConfig().SecondsPerSlot)) * time.Second),
|
|
ValidatorsRoot: [32]byte{'A'},
|
|
ValidAttestation: true,
|
|
DB: db,
|
|
Optimistic: true,
|
|
}
|
|
ctx, cancel := context.WithCancel(context.Background())
|
|
defer cancel()
|
|
s := &Service{
|
|
ctx: ctx,
|
|
cfg: &config{
|
|
initialSync: &mockSync.Sync{IsSyncing: false},
|
|
p2p: p,
|
|
beaconDB: db,
|
|
chain: chain,
|
|
clock: startup.NewClock(chain.Genesis, chain.ValidatorsRoot),
|
|
attestationNotifier: (&mockChain.ChainService{}).OperationNotifier(),
|
|
},
|
|
blkRootToPendingAtts: make(map[[32]byte][]*ethpb.SignedAggregateAttestationAndProof),
|
|
seenUnAggregatedAttestationCache: lruwrpr.New(10),
|
|
signatureChan: make(chan *signatureVerifier, verifierLimit),
|
|
}
|
|
s.initCaches()
|
|
go s.verifierRoutine()
|
|
|
|
invalidRoot := [32]byte{'A', 'B', 'C', 'D'}
|
|
s.setBadBlock(ctx, invalidRoot)
|
|
|
|
digest, err := s.currentForkDigest()
|
|
require.NoError(t, err)
|
|
|
|
blk := util.NewBeaconBlock()
|
|
blk.Block.Slot = 1
|
|
util.SaveBlock(t, ctx, db, blk)
|
|
|
|
validBlockRoot, err := blk.Block.HashTreeRoot()
|
|
require.NoError(t, err)
|
|
chain.FinalizedCheckPoint = ðpb.Checkpoint{
|
|
Root: validBlockRoot[:],
|
|
Epoch: 0,
|
|
}
|
|
|
|
validators := uint64(64)
|
|
savedState, keys := util.DeterministicGenesisState(t, validators)
|
|
require.NoError(t, savedState.SetSlot(1))
|
|
require.NoError(t, db.SaveState(context.Background(), savedState, validBlockRoot))
|
|
chain.State = savedState
|
|
|
|
tests := []struct {
|
|
name string
|
|
msg *ethpb.Attestation
|
|
topic string
|
|
validAttestationSignature bool
|
|
want bool
|
|
}{
|
|
{
|
|
name: "valid attestation signature",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 0,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{
|
|
Epoch: 0,
|
|
Root: validBlockRoot[:],
|
|
},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: true,
|
|
},
|
|
{
|
|
name: "valid attestation signature with nil topic",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 0,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{
|
|
Epoch: 0,
|
|
Root: validBlockRoot[:],
|
|
},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: "",
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "bad target epoch",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 0,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{
|
|
Epoch: 10,
|
|
Root: validBlockRoot[:],
|
|
},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "already seen",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 0,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "invalid beacon block",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: invalidRoot[:],
|
|
CommitteeIndex: 0,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "committee index exceeds committee length",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 4,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_2", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "wrong committee index",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 2,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_2", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "already aggregated",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b1011},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 1,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "missing block",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: bytesutil.PadTo([]byte("missing"), fieldparams.RootLength),
|
|
CommitteeIndex: 1,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: true,
|
|
want: false,
|
|
},
|
|
{
|
|
name: "invalid attestation",
|
|
msg: ðpb.Attestation{
|
|
AggregationBits: bitfield.Bitlist{0b101},
|
|
Data: ðpb.AttestationData{
|
|
BeaconBlockRoot: validBlockRoot[:],
|
|
CommitteeIndex: 1,
|
|
Slot: 1,
|
|
Target: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
Source: ðpb.Checkpoint{Root: make([]byte, fieldparams.RootLength)},
|
|
},
|
|
},
|
|
topic: fmt.Sprintf("/eth2/%x/beacon_attestation_1", digest),
|
|
validAttestationSignature: false,
|
|
want: false,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
helpers.ClearCache()
|
|
chain.ValidAttestation = tt.validAttestationSignature
|
|
if tt.validAttestationSignature {
|
|
com, err := helpers.BeaconCommitteeFromState(context.Background(), savedState, tt.msg.Data.Slot, tt.msg.Data.CommitteeIndex)
|
|
require.NoError(t, err)
|
|
domain, err := signing.Domain(savedState.Fork(), tt.msg.Data.Target.Epoch, params.BeaconConfig().DomainBeaconAttester, savedState.GenesisValidatorsRoot())
|
|
require.NoError(t, err)
|
|
attRoot, err := signing.ComputeSigningRoot(tt.msg.Data, domain)
|
|
require.NoError(t, err)
|
|
for i := 0; ; i++ {
|
|
if tt.msg.AggregationBits.BitAt(uint64(i)) {
|
|
tt.msg.Signature = keys[com[i]].Sign(attRoot[:]).Marshal()
|
|
break
|
|
}
|
|
}
|
|
} else {
|
|
tt.msg.Signature = make([]byte, 96)
|
|
}
|
|
buf := new(bytes.Buffer)
|
|
_, err := p.Encoding().EncodeGossip(buf, tt.msg)
|
|
require.NoError(t, err)
|
|
m := &pubsub.Message{
|
|
Message: &pubsubpb.Message{
|
|
Data: buf.Bytes(),
|
|
Topic: &tt.topic,
|
|
},
|
|
}
|
|
if tt.topic == "" {
|
|
m.Message.Topic = nil
|
|
}
|
|
|
|
res, err := s.validateCommitteeIndexBeaconAttestation(ctx, "" /*peerID*/, m)
|
|
received := res == pubsub.ValidationAccept
|
|
if received != tt.want {
|
|
t.Fatalf("Did not received wanted validation. Got %v, wanted %v", !tt.want, tt.want)
|
|
}
|
|
if tt.want && err != nil {
|
|
t.Errorf("Non nil error returned: %v", err)
|
|
}
|
|
if tt.want && m.ValidatorData == nil {
|
|
t.Error("Expected validator data to be set")
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestService_setSeenCommitteeIndicesSlot(t *testing.T) {
|
|
s := NewService(context.Background(), WithP2P(p2ptest.NewTestP2P(t)))
|
|
s.initCaches()
|
|
|
|
// Empty cache
|
|
b0 := []byte{9} // 1001
|
|
require.Equal(t, false, s.hasSeenCommitteeIndicesSlot(0, 0, b0))
|
|
|
|
// Cache some entries but same key
|
|
s.setSeenCommitteeIndicesSlot(0, 0, b0)
|
|
require.Equal(t, true, s.hasSeenCommitteeIndicesSlot(0, 0, b0))
|
|
b1 := []byte{14} // 1110
|
|
s.setSeenCommitteeIndicesSlot(0, 0, b1)
|
|
require.Equal(t, true, s.hasSeenCommitteeIndicesSlot(0, 0, b0))
|
|
require.Equal(t, true, s.hasSeenCommitteeIndicesSlot(0, 0, b1))
|
|
|
|
// Cache some entries with diff keys
|
|
s.setSeenCommitteeIndicesSlot(1, 2, b1)
|
|
require.Equal(t, false, s.hasSeenCommitteeIndicesSlot(1, 0, b1))
|
|
require.Equal(t, false, s.hasSeenCommitteeIndicesSlot(0, 2, b1))
|
|
require.Equal(t, true, s.hasSeenCommitteeIndicesSlot(1, 2, b1))
|
|
}
|