openpgp: Introduce feature flag for quickcheck.

  - Make quickcheck dependency optional.

  - Make quickcheck a dev-dependency for tests.

  - Fix doctests for
    - cert::ValidCert::user_attributes,
    - cert::builder::CertBuilder::add_user_attribute,
    - cert::revoke::UserAttributeRevocationBuilder
    - cert::revoke::UserAttributeRevocationBuilder::build.
    Doctests do not use cfg(test), so we cannot use quickcheck in there.

1 files changed, 4 insertions, 0 deletions

diff --git a/openpgp/src/types/timestamp.rs b/openpgp/src/types/timestamp.rs
index bd4f3bbb..6e4aebbd 100644
--- a/openpgp/src/types/timestamp.rs
+++ b/openpgp/src/types/timestamp.rs
@@ -3,6 +3,8 @@ use std::convert::{TryFrom, TryInto};
 use std::fmt;
 use std::time::{SystemTime, Duration as SystemDuration, UNIX_EPOCH};
 use std::u32;
+
+#[cfg(any(test, feature = "quickcheck"))]
 use quickcheck::{Arbitrary, Gen};
 
 use crate::{
@@ -242,6 +244,7 @@ impl Timestamp {
     }
 }
 
+#[cfg(any(test, feature = "quickcheck"))]
 impl Arbitrary for Timestamp {
     fn arbitrary<G: Gen>(g: &mut G) -> Self {
         Timestamp(u32::arbitrary(g))
@@ -562,6 +565,7 @@ impl Timestamp {
     pub(crate) const Y2106 : Timestamp = Timestamp(4291747200);
 }
 
+#[cfg(any(test, feature = "quickcheck"))]
 impl Arbitrary for Duration {
     fn arbitrary<G: Gen>(g: &mut G) -> Self {
         Duration(u32::arbitrary(g))