mirror of
https://github.com/OffchainLabs/prysm.git
synced 2026-01-08 23:18:15 -05:00
* Use SlotTicker instead of time.Ticker for attestation pool pruning * Offset one second before slot start
148 lines
5.6 KiB
Go
148 lines
5.6 KiB
Go
package attestations
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/OffchainLabs/prysm/v6/async"
|
|
fieldparams "github.com/OffchainLabs/prysm/v6/config/fieldparams"
|
|
"github.com/OffchainLabs/prysm/v6/config/params"
|
|
"github.com/OffchainLabs/prysm/v6/consensus-types/primitives"
|
|
ethpb "github.com/OffchainLabs/prysm/v6/proto/prysm/v1alpha1"
|
|
"github.com/OffchainLabs/prysm/v6/testing/assert"
|
|
"github.com/OffchainLabs/prysm/v6/testing/require"
|
|
"github.com/OffchainLabs/prysm/v6/testing/util"
|
|
"github.com/prysmaticlabs/go-bitfield"
|
|
)
|
|
|
|
func TestPruneExpired_Ticker(t *testing.T) {
|
|
// Need timeout longer than the offset (secondsPerSlot - 1) + some buffer
|
|
timeout := time.Duration(params.BeaconConfig().SecondsPerSlot+5) * time.Second
|
|
ctx, cancel := context.WithTimeout(t.Context(), timeout)
|
|
defer cancel()
|
|
|
|
s, err := NewService(ctx, &Config{
|
|
Pool: NewPool(),
|
|
pruneInterval: 250 * time.Millisecond,
|
|
})
|
|
require.NoError(t, err)
|
|
|
|
ad1 := util.HydrateAttestationData(ðpb.AttestationData{})
|
|
|
|
ad2 := util.HydrateAttestationData(ðpb.AttestationData{Slot: 1})
|
|
|
|
atts := []ethpb.Att{
|
|
ðpb.Attestation{Data: ad1, AggregationBits: bitfield.Bitlist{0b1000, 0b1}, Signature: make([]byte, fieldparams.BLSSignatureLength)},
|
|
ðpb.Attestation{Data: ad2, AggregationBits: bitfield.Bitlist{0b1000, 0b1}, Signature: make([]byte, fieldparams.BLSSignatureLength)},
|
|
}
|
|
require.NoError(t, s.cfg.Pool.SaveUnaggregatedAttestations(atts))
|
|
require.Equal(t, 2, s.cfg.Pool.UnaggregatedAttestationCount(), "Unexpected number of attestations")
|
|
atts = []ethpb.Att{
|
|
ðpb.Attestation{Data: ad1, AggregationBits: bitfield.Bitlist{0b1101, 0b1}, Signature: make([]byte, fieldparams.BLSSignatureLength)},
|
|
ðpb.Attestation{Data: ad2, AggregationBits: bitfield.Bitlist{0b1101, 0b1}, Signature: make([]byte, fieldparams.BLSSignatureLength)},
|
|
}
|
|
require.NoError(t, s.cfg.Pool.SaveAggregatedAttestations(atts))
|
|
assert.Equal(t, 2, s.cfg.Pool.AggregatedAttestationCount())
|
|
for _, att := range atts {
|
|
require.NoError(t, s.cfg.Pool.SaveBlockAttestation(att))
|
|
}
|
|
|
|
// Rewind back one epoch worth of time.
|
|
s.genesisTime = time.Now().Add(-1 * time.Duration(params.BeaconConfig().SlotsPerEpoch) * time.Duration(params.BeaconConfig().SecondsPerSlot) * time.Second)
|
|
|
|
go s.pruneExpired()
|
|
|
|
done := make(chan struct{}, 1)
|
|
async.RunEvery(ctx, 500*time.Millisecond, func() {
|
|
for _, attestation := range s.cfg.Pool.UnaggregatedAttestations() {
|
|
if attestation.GetData().Slot == 0 {
|
|
return
|
|
}
|
|
}
|
|
for _, attestation := range s.cfg.Pool.AggregatedAttestations() {
|
|
if attestation.GetData().Slot == 0 {
|
|
return
|
|
}
|
|
}
|
|
for _, attestation := range s.cfg.Pool.BlockAttestations() {
|
|
if attestation.GetData().Slot == 0 {
|
|
return
|
|
}
|
|
}
|
|
if s.cfg.Pool.UnaggregatedAttestationCount() != 1 || s.cfg.Pool.AggregatedAttestationCount() != 1 {
|
|
return
|
|
}
|
|
done <- struct{}{}
|
|
})
|
|
select {
|
|
case <-done:
|
|
// All checks are passed.
|
|
case <-ctx.Done():
|
|
t.Error("Test case takes too long to complete")
|
|
}
|
|
}
|
|
|
|
func TestPruneExpired_PruneExpiredAtts(t *testing.T) {
|
|
s, err := NewService(t.Context(), &Config{Pool: NewPool()})
|
|
require.NoError(t, err)
|
|
|
|
ad1 := util.HydrateAttestationData(ðpb.AttestationData{})
|
|
|
|
ad2 := util.HydrateAttestationData(ðpb.AttestationData{})
|
|
|
|
att1 := ðpb.Attestation{Data: ad1, AggregationBits: bitfield.Bitlist{0b1101}}
|
|
att2 := ðpb.Attestation{Data: ad1, AggregationBits: bitfield.Bitlist{0b1111}}
|
|
att3 := ðpb.Attestation{Data: ad2, AggregationBits: bitfield.Bitlist{0b1101}}
|
|
att4 := ðpb.Attestation{Data: ad2, AggregationBits: bitfield.Bitlist{0b1110}}
|
|
atts := []ethpb.Att{att1, att2, att3, att4}
|
|
require.NoError(t, s.cfg.Pool.SaveAggregatedAttestations(atts))
|
|
for _, att := range atts {
|
|
require.NoError(t, s.cfg.Pool.SaveBlockAttestation(att))
|
|
}
|
|
|
|
// Rewind back one epoch worth of time.
|
|
s.genesisTime = time.Now().Add(-1 * time.Duration(params.BeaconConfig().SlotsPerEpoch) * time.Duration(params.BeaconConfig().SecondsPerSlot) * time.Second)
|
|
|
|
s.pruneExpiredAtts()
|
|
// All the attestations on slot 0 should be pruned.
|
|
for _, attestation := range s.cfg.Pool.AggregatedAttestations() {
|
|
if attestation.GetData().Slot == 0 {
|
|
t.Error("Should be pruned")
|
|
}
|
|
}
|
|
for _, attestation := range s.cfg.Pool.BlockAttestations() {
|
|
if attestation.GetData().Slot == 0 {
|
|
t.Error("Should be pruned")
|
|
}
|
|
}
|
|
}
|
|
|
|
func TestPruneExpired_Expired(t *testing.T) {
|
|
s, err := NewService(t.Context(), &Config{Pool: NewPool()})
|
|
require.NoError(t, err)
|
|
|
|
// Rewind back one epoch worth of time.
|
|
s.genesisTime = time.Now().Add(-1 * time.Duration(params.BeaconConfig().SlotsPerEpoch) * time.Duration(params.BeaconConfig().SecondsPerSlot) * time.Second)
|
|
assert.Equal(t, true, s.expired(0), "Should be expired")
|
|
assert.Equal(t, false, s.expired(1), "Should not be expired")
|
|
}
|
|
|
|
func TestPruneExpired_ExpiredDeneb(t *testing.T) {
|
|
params.SetupTestConfigCleanup(t)
|
|
cfg := params.BeaconConfig()
|
|
cfg.DenebForkEpoch = 3
|
|
params.OverrideBeaconConfig(cfg)
|
|
|
|
s, err := NewService(t.Context(), &Config{Pool: NewPool()})
|
|
require.NoError(t, err)
|
|
|
|
// Rewind back 4 epochs + 10 slots worth of time.
|
|
s.genesisTime = time.Now().Add(-4*time.Duration(params.BeaconConfig().SlotsPerEpoch*primitives.Slot(params.BeaconConfig().SecondsPerSlot))*time.Second - 10*time.Duration(params.BeaconConfig().SecondsPerSlot)*time.Second)
|
|
secondEpochStart := primitives.Slot(2 * uint64(params.BeaconConfig().SlotsPerEpoch))
|
|
thirdEpochStart := primitives.Slot(3 * uint64(params.BeaconConfig().SlotsPerEpoch))
|
|
|
|
assert.Equal(t, true, s.expired(secondEpochStart), "Should be expired")
|
|
assert.Equal(t, false, s.expired(thirdEpochStart), "Should not be expired")
|
|
}
|