Add Eq traits for BLS newtypes

beacon_chain/utils/bls/src/aggregate_signature.rs

@@ -6,7 +6,7 @@ use bls_aggregates::AggregateSignature as RawAggregateSignature;
 ///
 /// This struct is a wrapper upon a base type and provides helper functions (e.g., SSZ
 /// serialization).
-#[derive(Debug, PartialEq, Clone, Default)]
+#[derive(Debug, PartialEq, Clone, Default, Eq)]
 pub struct AggregateSignature(RawAggregateSignature);
 
 impl AggregateSignature {

beacon_chain/utils/bls/src/keypair.rs

@@ -1,6 +1,6 @@
 use super::{PublicKey, SecretKey};
 
-#[derive(Debug, Clone, PartialEq)]
+#[derive(Debug, Clone, PartialEq, Eq)]
 pub struct Keypair {
     pub sk: SecretKey,
     pub pk: PublicKey,

beacon_chain/utils/bls/src/public_key.rs

@@ -1,12 +1,13 @@
 use super::SecretKey;
 use bls_aggregates::PublicKey as RawPublicKey;
-use ssz::{decode_ssz_list, Decodable, DecodeError, Encodable, SszStream};
+use ssz::{decode_ssz_list, ssz_encode, Decodable, DecodeError, Encodable, SszStream};
+use std::hash::{Hash, Hasher};
 
 /// A single BLS signature.
 ///
 /// This struct is a wrapper upon a base type and provides helper functions (e.g., SSZ
 /// serialization).
-#[derive(Debug, PartialEq, Clone)]
+#[derive(Debug, PartialEq, Clone, Eq)]
 pub struct PublicKey(RawPublicKey);
 
 impl PublicKey {
@@ -34,6 +35,12 @@ impl Decodable for PublicKey {
     }
 }
 
+impl Hash for PublicKey {
+    fn hash<H: Hasher>(&self, state: &mut H) {
+        ssz_encode(self).hash(state)
+    }
+}
+
 #[cfg(test)]
 mod tests {
     use super::super::ssz::ssz_encode;

beacon_chain/utils/bls/src/secret_key.rs

@@ -5,7 +5,7 @@ use ssz::{decode_ssz_list, Decodable, DecodeError, Encodable, SszStream};
 ///
 /// This struct is a wrapper upon a base type and provides helper functions (e.g., SSZ
 /// serialization).
-#[derive(Debug, PartialEq, Clone)]
+#[derive(Debug, PartialEq, Clone, Eq)]
 pub struct SecretKey(RawSecretKey);
 
 impl SecretKey {

beacon_chain/utils/bls/src/signature.rs

@@ -6,7 +6,7 @@ use bls_aggregates::Signature as RawSignature;
 ///
 /// This struct is a wrapper upon a base type and provides helper functions (e.g., SSZ
 /// serialization).
-#[derive(Debug, PartialEq, Clone)]
+#[derive(Debug, PartialEq, Clone, Eq)]
 pub struct Signature(RawSignature);
 
 impl Signature {