removing solomachine consensus state nil check and test cases (#1895)

cosmos · Aug 8, 2022 · 87bae33 · 87bae33
1 parent 9a4ed68
commit 87bae33
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 18 deletions.
diff --git a/modules/light-clients/06-solomachine/client_state.go b/modules/light-clients/06-solomachine/client_state.go
@@ -231,10 +231,6 @@ func produceVerificationArgs(
 		return nil, nil, 0, 0, err
 	}
 
-	if cs.ConsensusState == nil {
-		return nil, nil, 0, 0, sdkerrors.Wrap(clienttypes.ErrInvalidConsensus, "consensus state cannot be empty")
-	}
-
 	// sequence is encoded in the revision height of height struct
 	sequence := height.GetRevisionHeight()
 	latestSequence := cs.GetLatestHeight().GetRevisionHeight()

diff --git a/modules/light-clients/06-solomachine/client_state_test.go b/modules/light-clients/06-solomachine/client_state_test.go
@@ -447,13 +447,6 @@ func (suite *SoloMachineTestSuite) TestVerifyMembership() {
 				},
 				true,
 			},
-			{
-				"consensus state in client state is nil",
-				func() {
-					clientState = solomachine.NewClientState(1, nil, false)
-				},
-				false,
-			},
 			{
 				"client state latest height is less than sequence",
 				func() {
@@ -665,13 +658,6 @@ func (suite *SoloMachineTestSuite) TestVerifyNonMembership() {
 				},
 				true,
 			},
-			{
-				"consensus state in client state is nil",
-				func() {
-					clientState = solomachine.NewClientState(1, nil, false)
-				},
-				false,
-			},
 			{
 				"client state latest height is less than sequence",
 				func() {