reset state

x/feemarket/keeper/msg_server.go

@@ -30,10 +30,15 @@ func (ms MsgServer) Params(goCtx context.Context, msg *types.MsgParams) (*types.
 		return nil, fmt.Errorf("invalid authority to execute message")
 	}
 
-	err := ms.k.SetParams(ctx, msg.Params)
-	if err != nil {
+	params := msg.Params
+	if err := ms.k.SetParams(ctx, params); err != nil {
 		return nil, fmt.Errorf("error setting params: %w", err)
 	}
 
+	newState := types.NewState(params.Window, params.MinBaseFee, params.MinLearningRate)
+	if err := ms.k.SetState(ctx, newState); err != nil {
+		return nil, fmt.Errorf("error setting state: %w", err)
+	}
+
 	return &types.MsgParamsResponse{}, nil
 }

x/feemarket/keeper/msg_server_test.go

@@ -12,6 +12,10 @@ func (s *KeeperTestSuite) TestMsgParams() {
 		resp, err := s.msgServer.Params(s.ctx, req)
 		s.Require().NoError(err)
 		s.Require().NotNil(resp)
+
+		params, err := s.feeMarketKeeper.GetParams(s.ctx)
+		s.Require().NoError(err)
+		s.Require().False(params.Enabled)
 	})
 
 	s.Run("accepts a req with params", func() {
@@ -35,4 +39,31 @@ func (s *KeeperTestSuite) TestMsgParams() {
 		_, err := s.msgServer.Params(s.ctx, req)
 		s.Require().Error(err)
 	})
+
+	s.Run("resets state after new params request", func() {
+		params, err := s.feeMarketKeeper.GetParams(s.ctx)
+		s.Require().NoError(err)
+
+		state, err := s.feeMarketKeeper.GetState(s.ctx)
+		s.Require().NoError(err)
+
+		err = state.Update(params.MaxBlockUtilization, params)
+		s.Require().NoError(err)
+
+		err = s.feeMarketKeeper.SetState(s.ctx, state)
+		s.Require().NoError(err)
+
+		params.Window = 100
+		req := &types.MsgParams{
+			Authority: s.authorityAccount.String(),
+			Params:    params,
+		}
+		_, err = s.msgServer.Params(s.ctx, req)
+		s.Require().NoError(err)
+
+		state, err = s.feeMarketKeeper.GetState(s.ctx)
+		s.Require().NoError(err)
+		s.Require().Equal(params.Window, uint64(len(state.Window)))
+		s.Require().Equal(state.Window[0], uint64(0))
+	})
 }