Tidy validation signatures

lighthouse/state/validation/signatures.rs

@@ -18,11 +18,12 @@ pub enum SignatureVerificationError {
     DBError(String),
 }
 
-fn verify_aggregate_signature_for_indices<T>(message: &[u8],
+pub fn verify_aggregate_signature_for_indices<T>(
-                                             agg_sig: &AggregateSignature,
+    message: &[u8],
-                                             attestation_indices: &[usize],
+    agg_sig: &AggregateSignature,
-                                             bitfield: &Bitfield,
+    attestation_indices: &[usize],
-                                             validator_store: &ValidatorStore<T>)
+    bitfield: &Bitfield,
+    validator_store: &ValidatorStore<T>)
     -> Result<(bool, Option<HashSet<usize>>), SignatureVerificationError>
     where T: ClientDB + Sized
 {
@@ -68,6 +69,15 @@ mod tests {
     use super::super::db::MemoryDB;
     use std::sync::Arc;
 
+    /*
+     * Cases that still need testing:
+     *
+     * - No signatures.
+     * - Database failure.
+     * - Unknown validator index.
+     * - Extra validator on signature.
+     */
+
     #[test]
     fn test_signature_verification() {
         let message = "cats".as_bytes();