Derive PartialEq, Eq on ProdCommitValidator (#1264)

Also add test code that ensures ProdVerifier supports the common derived traits.
informalsystems · Feb 6, 2023 · 89c8add · 89c8add
1 parent 47e28b5
commit 89c8add
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/light-client-verifier/src/operations/commit_validator.rs b/light-client-verifier/src/operations/commit_validator.rs
@@ -65,7 +65,7 @@ pub trait CommitValidator: Send + Sync {
 }
 
 /// Production-ready implementation of a commit validator.
-#[derive(Copy, Clone, Default, Debug)]
+#[derive(Copy, Clone, Default, Debug, PartialEq, Eq)]
 pub struct ProdCommitValidator;
 
 impl CommitValidator for ProdCommitValidator {}
diff --git a/light-client-verifier/src/verifier.rs b/light-client-verifier/src/verifier.rs
@@ -285,6 +285,12 @@ mod tests {
         Verdict, Verifier,
     };
 
+    #[cfg(feature = "rust-crypto")]
+    #[derive(Clone, Debug, PartialEq, Eq)]
+    struct ProdVerifierSupportsCommonDerivedTraits {
+        verifier: ProdVerifier,
+    }
+
     #[test]
     fn test_verification_failure_on_chain_id_mismatch() {
         let now = Time::now();