nit

osmosis-labs · Jul 4, 2022 · aa85549 · aa85549
1 parent be7dd7a
commit aa85549
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 13 deletions.
diff --git a/osmoutils/binary_search_test.go b/osmoutils/binary_search_test.go
@@ -59,8 +59,8 @@ func TestBinarySearch(t *testing.T) {
 			require.Error(t, err)
 		} else {
 			require.NoError(t, err)
+			require.True(sdk.IntEq(t, tc.expectedSolvedInput, actualSolvedInput))
 		}
-		require.True(sdk.IntEq(t, tc.expectedSolvedInput, actualSolvedInput))
 	}
 }
 

diff --git a/osmoutils/cli_helpers.go b/osmoutils/cli_helpers.go
@@ -61,5 +61,4 @@ func ConditionalPanic(t *testing.T, expectPanic bool, sut func()) bool {
 
 	require.NotPanics(t, sut)
 	return false
-
 }
diff --git a/x/epochs/client/cli/cli_test.go b/x/epochs/client/cli/cli_test.go
@@ -68,7 +68,6 @@ func (s *IntegrationTestSuite) TestGetCmdCurrentEpoch() {
 			}
 
 			out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, args)
-
 			if tc.expectErr {
 				s.Require().Error(err)
 			} else {

diff --git a/x/mint/keeper/genesis_test.go b/x/mint/keeper/genesis_test.go
@@ -112,16 +112,6 @@ func (suite *KeeperTestSuite) TestMintInitGenesis() {
 			originalVestingCoins := bankKeeper.GetBalance(ctx, developerAccount, tc.mintDenom)
 
 			// Test.
-			// if tc.expectPanic {
-			// 	suite.Panics(func() {
-			// 		mintKeeper.InitGenesis(ctx, tc.mintGenesis)
-			// 	})
-			// 	return
-			// }
-
-			// suite.NotPanics(func() {
-			// 	mintKeeper.InitGenesis(ctx, tc.mintGenesis)
-			// })
 
 			if osmoutils.ConditionalPanic(suite.T(), tc.expectPanic || false, func() { mintKeeper.InitGenesis(ctx, tc.mintGenesis) }) {
 				return
Original file line number	Diff line number	Diff line change
Expand Up		@@ -61,5 +61,4 @@ func ConditionalPanic(t *testing.T, expectPanic bool, sut func()) bool {

		require.NotPanics(t, sut)
		return false

		}