fix up via interface

beacon-chain/core/blocks/proposer_slashing.go

@@ -12,7 +12,6 @@ import (
 	"github.com/prysmaticlabs/prysm/config/params"
 	types "github.com/prysmaticlabs/prysm/consensus-types/primitives"
 	ethpb "github.com/prysmaticlabs/prysm/proto/prysm/v1alpha1"
-	"github.com/prysmaticlabs/prysm/runtime/version"
 	"github.com/prysmaticlabs/prysm/time/slots"
 	"google.golang.org/protobuf/proto"
 )
@@ -75,16 +74,10 @@ func ProcessProposerSlashing(
 		return nil, errors.Wrap(err, "could not verify proposer slashing")
 	}
 	cfg := params.BeaconConfig()
-	var slashingQuotient uint64
+
-	switch {
+	slashingQuotient, err := beaconState.MinSlashingPenaltyQuotient()
-	case beaconState.Version() == version.Phase0:
+	if err != nil {
-		slashingQuotient = cfg.MinSlashingPenaltyQuotient
+		return nil, err
-	case beaconState.Version() == version.Altair:
-		slashingQuotient = cfg.MinSlashingPenaltyQuotientAltair
-	case beaconState.Version() == version.Bellatrix:
-		slashingQuotient = cfg.MinSlashingPenaltyQuotientBellatrix
-	default:
-		return nil, errors.New("unknown state version")
 	}
 	beaconState, err = slashFunc(ctx, beaconState, slashing.Header_1.Header.ProposerIndex, slashingQuotient, cfg.ProposerRewardQuotient)
 	if err != nil {

beacon-chain/state/interfaces.go

@@ -26,6 +26,7 @@ type BeaconState interface {
 // SpecParametersProvider provides fork-specific configuration parameters as
 // defined in the consensus specification for the beacon chain.
 type SpecParametersProvider interface {
+	MinSlashingPenaltyQuotient() (uint64, error)
 	InactivityPenaltyQuotient() (uint64, error)
 	ProportionalSlashingMultiplier() (uint64, error)
 }

beacon-chain/state/state-native/spec_parameters.go

@@ -28,3 +28,15 @@ func (b *BeaconState) InactivityPenaltyQuotient() (uint64, error) {
 	}
 	return 0, errNotSupported("InactivityPenaltyQuotient()", b.version)
 }
+
+func (b *BeaconState) MinSlashingPenaltyQuotient() (uint64, error) {
+	switch b.version {
+	case version.Bellatrix:
+		return params.BeaconConfig().MinSlashingPenaltyQuotientBellatrix, nil
+	case version.Altair:
+		return params.BeaconConfig().MinSlashingPenaltyQuotientAltair, nil
+	case version.Phase0:
+		return params.BeaconConfig().MinSlashingPenaltyQuotient, nil
+	}
+	return 0, errNotSupported("MinSlashingPenaltyQuotient()", b.version)
+}

beacon-chain/state/v1/spec_parameters.go

@@ -9,3 +9,7 @@ func (b *BeaconState) ProportionalSlashingMultiplier() (uint64, error) {
 func (b *BeaconState) InactivityPenaltyQuotient() (uint64, error) {
 	return params.BeaconConfig().InactivityPenaltyQuotient, nil
 }
+
+func (b *BeaconState) MinSlashingPenaltyQuotient() (uint64, error) {
+	return params.BeaconConfig().MinSlashingPenaltyQuotient, nil
+}

beacon-chain/state/v2/spec_parameters.go

@@ -9,3 +9,7 @@ func (b *BeaconState) ProportionalSlashingMultiplier() (uint64, error) {
 func (b *BeaconState) InactivityPenaltyQuotient() (uint64, error) {
 	return params.BeaconConfig().InactivityPenaltyQuotientAltair, nil
 }
+
+func (b *BeaconState) MinSlashingPenaltyQuotient() (uint64, error) {
+	return params.BeaconConfig().MinSlashingPenaltyQuotientAltair, nil
+}

beacon-chain/state/v3/spec_parameters.go

@@ -9,3 +9,7 @@ func (b *BeaconState) ProportionalSlashingMultiplier() (uint64, error) {
 func (b *BeaconState) InactivityPenaltyQuotient() (uint64, error) {
 	return params.BeaconConfig().InactivityPenaltyQuotientBellatrix, nil
 }
+
+func (b *BeaconState) MinSlashingPenaltyQuotient() (uint64, error) {
+	return params.BeaconConfig().MinSlashingPenaltyQuotientBellatrix, nil
+}