Remove redundant tests for duty with no pre-consensus, consensus, or post-consensus

ssv/committee.go

@@ -113,17 +113,19 @@ func (c *Committee) ProcessMessage(signedSSVMessage *types.SignedSSVMessage) err
 			return errors.Wrap(err, "invalid PartialSignatureMessages")
 		}
 
-		if pSigMessages.Type == types.PostConsensusPartialSig {
-			runner, exists := c.Runners[pSigMessages.Slot]
-			if !exists {
-				return errors.New("no runner found for message's slot")
-			}
-			return runner.ProcessPostConsensus(pSigMessages)
+		if pSigMessages.Type != types.PostConsensusPartialSig {
+			return errors.New("no pre consensus phase for committee runner")
 		}
+
+		runner, exists := c.Runners[pSigMessages.Slot]
+		if !exists {
+			return errors.New("no runner found for message's slot")
+		}
+		return runner.ProcessPostConsensus(pSigMessages)
+
 	default:
 		return errors.New("unknown msg")
 	}
-	return nil
 
 }
 

ssv/spectest/all_tests.go

@@ -63,6 +63,7 @@ var AllTests = []tests.TestF{
 	postconsensus.InvalidSignedMessageEmptySignature,
 	postconsensus.InvalidSignedMessageDifferentLength,
 	postconsensus.InvalidSignedMessageNoSigners,
+	postconsensus.NoPostConsensusPhase,
 
 	newduty.ConsensusNotStarted,
 	newduty.NotDecided,
@@ -118,6 +119,7 @@ var AllTests = []tests.TestF{
 	consensus.EmptySignature,
 	consensus.SignersAndSignaturesWithDifferentLength,
 	consensus.NilSSVMessage,
+	consensus.NoConsensusPhase,
 
 	synccommitteeaggregator.SomeAggregatorQuorum,
 	synccommitteeaggregator.NoneAggregatorQuorum,
@@ -161,6 +163,7 @@ var AllTests = []tests.TestF{
 	preconsensus.InvalidSignedMessageNoSignatures,
 	preconsensus.InvalidSignedMessageEmptySignature,
 	preconsensus.InvalidSignedMessageDifferentLength,
+	preconsensus.NoPreConsensusPhase,
 
 	valcheckduty.WrongValidatorIndex,
 	valcheckduty.WrongValidatorPK,