Complete SignedSSVMessage tests (#441)

* Add SignedSSVMessage missing tests * Rename file according to test name * Add missing SignedSSVMessages tests in qbft * Add missing SignedSSVMessages tests in ssv postconsensus * Add missing SignedSSVMessages tests in ssv preconsensus * Generate JSON tests * Remove SignedSSVMessage tests from QBFT/messages * Add SignedSSVMessage validation tests in SSV/Runner/Consensus * Generate JSON tests * Generate JSON tests
ssvlabs · Jul 3, 2024 · 981a403 · 981a403
1 parent c31ad31
commit 981a403
Show file tree

Hide file tree

Showing 148 changed files with 38,835 additions and 12 deletions.
diff --git a/qbft/spectest/all_tests.go b/qbft/spectest/all_tests.go
@@ -96,6 +96,7 @@ var AllTests = []tests.TestF{
 	messages.ProposeDataEncoding,
 	messages.SignedMsgNoSigners,
 	messages.SignedMsgDuplicateSigners,
+
 	messages.SignedMsgMultiSigners,
 	messages.GetRoot,
 	messages.RoundChangeJustificationsUnmarshalling,

diff --git a/ssv/spectest/all_tests.go b/ssv/spectest/all_tests.go
@@ -56,6 +56,10 @@ var AllTests = []tests.TestF{
 	postconsensus.PartialInvalidRootQuorumThenValidQuorum,
 	postconsensus.PartialInvalidSigQuorumThenValidQuorum,
 	postconsensus.MixedCommittees,
+	postconsensus.InvalidSignedMessageNoSignature,
+	postconsensus.InvalidSignedMessageEmptySignature,
+	postconsensus.InvalidSignedMessageDifferentLength,
+	postconsensus.InvalidSignedMessageNoSigners,
 
 	newduty.ConsensusNotStarted,
 	newduty.NotDecided,
@@ -102,6 +106,13 @@ var AllTests = []tests.TestF{
 	consensus.ValidMessage,
 	consensus.InvalidSignature,
 	consensus.DecidedSlashableAttestation,
+	consensus.NoSigners,
+	consensus.NonUniqueSigners,
+	consensus.ZeroSigner,
+	consensus.NoSignatures,
+	consensus.EmptySignature,
+	consensus.SignersAndSignaturesWithDifferentLength,
+	consensus.NilSSVMessage,
 
 	synccommitteeaggregator.SomeAggregatorQuorum,
 	synccommitteeaggregator.NoneAggregatorQuorum,
@@ -141,6 +152,10 @@ var AllTests = []tests.TestF{
 	preconsensus.InvalidQuorumThenValidQuorum,
 	preconsensus.InconsistentOperatorSigner,
 	preconsensus.NilSSVMessage,
+	preconsensus.InvalidSignedMessageNoSigners,
+	preconsensus.InvalidSignedMessageNoSignatures,
+	preconsensus.InvalidSignedMessageEmptySignature,
+	preconsensus.InvalidSignedMessageDifferentLength,
 
 	valcheckduty.WrongValidatorIndex,
 	valcheckduty.WrongValidatorPK,

diff --git a/...sts_MultiMsgProcessingSpecTest/different_length_of_signers_and_signatures/aggregator.json b/...sts_MultiMsgProcessingSpecTest/different_length_of_signers_and_signatures/aggregator.json