diff options
| author | Justus Winter <justus@sequoia-pgp.org> | 2020-04-15 11:46:04 +0200 |
|---|---|---|
| committer | Justus Winter <justus@sequoia-pgp.org> | 2020-04-15 11:46:04 +0200 |
| commit | 9e70dc664bbb100eb0e1a3df7336c5d64d94f862 (patch) | |
| tree | 7e61a6a2612731a0b05eadf0b09d40914d00798c /openpgp/src/cert/builder.rs | |
| parent | e6e6319da128a275d0a45577afd7e4b0dcea124e (diff) | |
openpgp: Make test more robust.
Diffstat (limited to 'openpgp/src/cert/builder.rs')
| -rw-r--r-- | openpgp/src/cert/builder.rs | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/openpgp/src/cert/builder.rs b/openpgp/src/cert/builder.rs index 5d74d727..a633ab7a 100644 --- a/openpgp/src/cert/builder.rs +++ b/openpgp/src/cert/builder.rs @@ -689,6 +689,8 @@ mod tests { #[test] fn designated_revokers() -> Result<()> { + use std::collections::HashSet; + let p = &P::new(); let fpr1 = "C03F A641 1B03 AE12 5764 6118 7223 B566 78E0 2528"; @@ -708,16 +710,16 @@ mod tests { .generate()?; let cert = cert.with_policy(p, None)?; - assert_eq!(cert.revocation_keys().collect::<Vec<_>>(), - revokers.iter().collect::<Vec<_>>()); + assert_eq!(cert.revocation_keys().collect::<HashSet<_>>(), + revokers.iter().collect::<HashSet<_>>()); // The designated revokers on the direct signature should also // be returned when querying components for designated // revokers. - assert_eq!(cert.primary_key().revocation_keys().collect::<Vec<_>>(), - revokers.iter().collect::<Vec<_>>()); - assert_eq!(cert.primary_userid()?.revocation_keys().collect::<Vec<_>>(), - revokers.iter().collect::<Vec<_>>()); + assert_eq!(cert.primary_key().revocation_keys().collect::<HashSet<_>>(), + revokers.iter().collect::<HashSet<_>>()); + assert_eq!(cert.primary_userid()?.revocation_keys().collect::<HashSet<_>>(), + revokers.iter().collect::<HashSet<_>>()); // Do it again, with a key that has no User IDs. @@ -727,14 +729,14 @@ mod tests { let cert = cert.with_policy(p, None)?; assert!(cert.primary_userid().is_err()); - assert_eq!(cert.revocation_keys().collect::<Vec<_>>(), - revokers.iter().collect::<Vec<_>>()); + assert_eq!(cert.revocation_keys().collect::<HashSet<_>>(), + revokers.iter().collect::<HashSet<_>>()); // The designated revokers on the direct signature should also // be returned when querying components for designated // revokers. - assert_eq!(cert.primary_key().revocation_keys().collect::<Vec<_>>(), - revokers.iter().collect::<Vec<_>>()); + assert_eq!(cert.primary_key().revocation_keys().collect::<HashSet<_>>(), + revokers.iter().collect::<HashSet<_>>()); Ok(()) } } |