fix: require and satisfy hasbeginblocker interface correclty in x/mint

x/mint/abci.go

@@ -161,11 +161,11 @@ func predictTotalMinted(totalMinted sdkmath.Uint, normTimePassed, timeAhead sdkm
 }
 
 // BeginBlocker mints new tokens for the previous block.
-func BeginBlocker(ctx context.Context, k keeper.Keeper) {
+func BeginBlocker(ctx context.Context, k keeper.Keeper) error {
 	c := sdk.UnwrapSDKContext(ctx)
 	minter := k.GetMinter(ctx)
 	if minter.TotalMinted.GTE(types.MintingCap) {
-		return
+		return errors.New("minting cap has been reached")
 	}
 
 	defer telemetry.ModuleMeasureSince(types.ModuleName, time.Now(), telemetry.MetricKeyBeginBlocker)
@@ -206,4 +206,6 @@ func BeginBlocker(ctx context.Context, k keeper.Keeper) {
 			sdk.NewAttribute(sdk.AttributeKeyAmount, coinAmount.String()),
 		),
 	)
+
+	return nil
 }

x/mint/module.go

@@ -32,8 +32,9 @@ var (
 	_ module.AppModuleSimulation = (*AppModule)(nil)
 	_ module.HasABCIGenesis      = (*AppModule)(nil)
 
-	_ appmodule.AppModule    = (*AppModule)(nil)
-	_ module.HasABCIEndBlock = (*AppModule)(nil)
+	_ appmodule.AppModule       = (*AppModule)(nil)
+	_ module.HasABCIEndBlock    = (*AppModule)(nil)
+	_ appmodule.HasBeginBlocker = (*AppModule)(nil)
 )
 
 // AppModuleBasic defines the basic application module used by the mint module.
@@ -166,8 +167,8 @@ func (am AppModule) ExportGenesis(ctx sdk.Context, cdc codec.JSONCodec) json.Raw
 func (AppModule) ConsensusVersion() uint64 { return ConsensusVersion }
 
 // BeginBlock returns the begin blocker for the mint module.
-func (am AppModule) BeginBlock(ctx context.Context) {
-	BeginBlocker(ctx, am.keeper)
+func (am AppModule) BeginBlock(ctx context.Context) error {
+	return BeginBlocker(ctx, am.keeper)
 }
 
 // EndBlock returns the end blocker for the mint module. It returns no validator