From b0854ae4296c13c6e7fd88ed5c0fd94c58e5de7f Mon Sep 17 00:00:00 2001 From: maksneprik <155268440+deus-labs@users.noreply.github.com> Date: Wed, 8 Oct 2025 17:23:09 +0200 Subject: [PATCH] fix(x/staking): add missing error check in simulation operations test (#25421) Co-authored-by: Alex | Cosmos Labs --- x/staking/simulation/operations_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/x/staking/simulation/operations_test.go b/x/staking/simulation/operations_test.go index 86d5ff3805..c47578fce8 100644 --- a/x/staking/simulation/operations_test.go +++ b/x/staking/simulation/operations_test.go @@ -147,8 +147,8 @@ func (s *SimTestSuite) TestWeightedOperations() { } for i, w := range weightedOps { - operationMsg, _, _ := w.Op()(s.r, s.app.BaseApp, s.ctx, s.accounts, s.ctx.ChainID()) - // require.NoError(t, err) // TODO check if it should be NoError + operationMsg, _, err := w.Op()(s.r, s.app.BaseApp, s.ctx, s.accounts, s.ctx.ChainID()) + s.Require().NoError(err) // the following checks are very much dependent from the ordering of the output given // by WeightedOperations. if the ordering in WeightedOperations changes some tests