cleaned up tests

state/helpers_test.go

@@ -29,6 +29,7 @@ func (m mockEvPoolAlwaysCommitted) PendingEvidence(int64) []types.Evidence { ret
 func (m mockEvPoolAlwaysCommitted) AddEvidence(types.Evidence) error       { return nil }
 func (m mockEvPoolAlwaysCommitted) Update(*types.Block, sm.State)          {}
 func (m mockEvPoolAlwaysCommitted) IsCommitted(types.Evidence) bool        { return true }
+func (m mockEvPoolAlwaysCommitted) IsPending(types.Evidence) bool          { return false }
 
 func newTestApp() proxy.AppConns {
 	app := &testApp{}

state/services.go

@@ -43,6 +43,7 @@ type EvidencePool interface {
 	Update(*types.Block, State)
 	// IsCommitted indicates if this evidence was already marked committed in another block.
 	IsCommitted(types.Evidence) bool
+	IsPending(types.Evidence) bool
 }
 
 // MockEvidencePool is an empty implementation of EvidencePool, useful for testing.
@@ -52,3 +53,4 @@ func (m MockEvidencePool) PendingEvidence(int64) []types.Evidence { return nil }
 func (m MockEvidencePool) AddEvidence(types.Evidence) error       { return nil }
 func (m MockEvidencePool) Update(*types.Block, State)             {}
 func (m MockEvidencePool) IsCommitted(types.Evidence) bool        { return false }
+func (m MockEvidencePool) IsPending(types.Evidence) bool          { return false }

state/validation.go

@@ -136,6 +136,9 @@ func validateBlock(evidencePool EvidencePool, stateDB dbm.DB, state State, block
 			if evidencePool.IsCommitted(ev) {
 				return types.NewErrEvidenceInvalid(ev, errors.New("evidence was already committed"))
 			}
+			if evidencePool.IsPending(ev) {
+				return nil
+			}
 		}
 		if err := VerifyEvidence(stateDB, state, ev, &block.Header); err != nil {
 			return types.NewErrEvidenceInvalid(ev, err)