jmap-types 0.4.1

//! Shared macro for string-backed enums with a catch-all `Other(String)` variant.
//!
//! This module is the single canonical home for the [`impl_string_enum!`] macro
//! used across the `jmap-*-types` crate family. JMAP wire-protocol enums often
//! allow vendor-specific extensions (per RFC 8620 §1.6 and the various JMAP
//! extension specs), so each enum carries an `Other(String)` variant that
//! preserves unknown values for round-trip fidelity.
//!
//! Previously every `jmap-*-types` crate carried a byte-identical copy of this
//! macro; this module consolidates them. See bd issue JMAP-wk77.

/// Implement an inherent `as_str()`, [`serde::Serialize`],
/// [`serde::Deserialize`], [`std::fmt::Display`], `From<&str>` and
/// `From<String>` for a string-backed enum with a catch-all `Other(String)`
/// variant.
///
/// # Requirements
///
/// The enum **must** have a variant `Other(String)` that stores the raw wire string
/// for any value not listed in the known-value mapping.  The macro generates match
/// arms only for the listed variants; the `Other` catch-all is generated automatically.
///
/// # Usage
///
/// ```ignore
/// impl_string_enum!(MyEnum, "a my-enum value",
///     "wire-string-1" => Variant1,
///     "wire-string-2" => Variant2,
/// );
/// ```
///
/// This emits:
/// - `pub fn as_str(&self) -> &str` — the single wire mapping: each listed
///   variant yields its wire string, `Other(v)` yields the inner string.
///   Do not hand-write a second copy; `Serialize` and `Display` call this.
/// - `impl Serialize` — serialises the `as_str()` value.
/// - `impl Deserialize` — deserialises a JSON string using a `Visitor`; unknown values
///   become `Other(v.to_owned())`.
/// - `impl Display` — formats the `as_str()` value, identical to `Serialize`.
/// - `impl From<&str>` and `impl From<String>` — the same mapping as
///   `Deserialize`, so a listed wire string always builds its typed variant
///   and never `Other`: `MyEnum::from("wire-string-1") == MyEnum::Variant1`.
///
/// # Forged `Other` values
///
/// `Other(String)` is a public variant, so a caller can write
/// `MyEnum::Other("wire-string-1".into())`. That value compares unequal to
/// `MyEnum::Variant1`, yet serialises to the same wire string and
/// deserialises back as `MyEnum::Variant1`. Build values from a string with
/// `MyEnum::from(..)`, which never yields `Other` for a listed wire string,
/// and construct `Other` directly only for a string the enum does not list.
#[macro_export]
macro_rules! impl_string_enum {
    ($ty:ident, $expecting:literal, $( $s:literal => $variant:ident ),+ $(,)?) => {
        impl $ty {
            /// The wire string for this value: the listed string for a
            /// known variant, the stored string for `Other`.
            pub fn as_str(&self) -> &str {
                match self {
                    $( $ty::$variant => $s, )+
                    $ty::Other(v) => v.as_str(),
                }
            }
        }
        impl ::serde::Serialize for $ty {
            fn serialize<S: ::serde::Serializer>(&self, s: S) -> Result<S::Ok, S::Error> {
                s.serialize_str(self.as_str())
            }
        }
        impl<'de> ::serde::Deserialize<'de> for $ty {
            fn deserialize<D: ::serde::Deserializer<'de>>(d: D) -> Result<Self, D::Error> {
                struct Visitor;
                impl ::serde::de::Visitor<'_> for Visitor {
                    type Value = $ty;
                    fn expecting(
                        &self,
                        f: &mut ::std::fmt::Formatter<'_>,
                    ) -> ::std::fmt::Result {
                        write!(f, $expecting)
                    }
                    fn visit_str<E: ::serde::de::Error>(self, v: &str) -> Result<$ty, E> {
                        Ok($ty::from(v))
                    }
                }
                d.deserialize_str(Visitor)
            }
        }
        impl ::std::convert::From<&str> for $ty {
            fn from(v: &str) -> Self {
                match v {
                    $( $s => $ty::$variant, )+
                    _ => $ty::Other(v.to_owned()),
                }
            }
        }
        impl ::std::convert::From<::std::string::String> for $ty {
            fn from(v: ::std::string::String) -> Self {
                match v.as_str() {
                    $( $s => $ty::$variant, )+
                    _ => $ty::Other(v),
                }
            }
        }
        impl ::std::fmt::Display for $ty {
            fn fmt(&self, f: &mut ::std::fmt::Formatter<'_>) -> ::std::fmt::Result {
                f.write_str(self.as_str())
            }
        }
    };
}

#[cfg(test)]
mod tests {
    // Test enum exercising the catch-all `Other(String)` pattern. The test
    // oracle is hand-written JSON (independent of the code under test): we
    // assert the macro emits the exact wire bytes a JMAP client would receive
    // and that an unknown wire value round-trips losslessly via `Other`.
    #[derive(Debug, PartialEq)]
    enum TestKind {
        Foo,
        Bar,
        Other(String),
    }

    impl_string_enum!(
        TestKind,
        "a test kind string",
        "foo" => Foo,
        "bar" => Bar,
    );

    #[test]
    fn known_variant_serializes_to_wire_string() {
        // Hand-written oracle: the JSON form of TestKind::Foo is the literal
        // string "foo" (3 bytes plus surrounding quotes = 5 bytes total).
        assert_eq!(serde_json::to_string(&TestKind::Foo).unwrap(), r#""foo""#);
        assert_eq!(serde_json::to_string(&TestKind::Bar).unwrap(), r#""bar""#);
    }

    #[test]
    fn known_wire_string_deserializes_to_variant() {
        // Inverse oracle: feeding the canonical wire bytes yields the typed variant.
        let foo: TestKind = serde_json::from_str(r#""foo""#).unwrap();
        let bar: TestKind = serde_json::from_str(r#""bar""#).unwrap();
        assert_eq!(foo, TestKind::Foo);
        assert_eq!(bar, TestKind::Bar);
    }

    #[test]
    fn forged_other_is_unequal_but_round_trips_to_variant() {
        // The documented caveat: `Other("foo")` is not `Foo`, serialises to
        // the wire string "foo" and deserialises back as `Foo`.
        let forged = TestKind::Other("foo".to_owned());
        assert_ne!(forged, TestKind::Foo);
        let wire = serde_json::to_string(&forged).unwrap();
        assert_eq!(wire, r#""foo""#);
        let back: TestKind = serde_json::from_str(&wire).unwrap();
        assert_eq!(back, TestKind::Foo);
        assert_eq!(TestKind::from("foo"), TestKind::Foo);
    }

    #[test]
    fn unknown_wire_string_round_trips_via_other() {
        // The defining JMAP property: a value the spec does not know must be
        // preserved verbatim through deserialize -> serialize. This is what
        // `Other(String)` exists to guarantee.
        let original = r#""vendor-extension-value""#;
        let parsed: TestKind = serde_json::from_str(original).unwrap();
        assert_eq!(parsed, TestKind::Other("vendor-extension-value".to_owned()));
        let reserialized = serde_json::to_string(&parsed).unwrap();
        assert_eq!(reserialized, original);
    }

    #[test]
    fn display_matches_serialize_form() {
        // Display must emit the same wire string as Serialize. A divergence
        // here would silently corrupt log output relative to JSON output.
        assert_eq!(format!("{}", TestKind::Foo), "foo");
        assert_eq!(format!("{}", TestKind::Bar), "bar");
        assert_eq!(
            format!("{}", TestKind::Other("custom".to_owned())),
            "custom",
        );
    }

    #[test]
    fn as_str_yields_wire_string() {
        // Hand-written oracle: the listed wire strings, and the stored
        // string verbatim for `Other`.
        assert_eq!(TestKind::Foo.as_str(), "foo");
        assert_eq!(TestKind::Bar.as_str(), "bar");
        assert_eq!(TestKind::Other("custom".to_owned()).as_str(), "custom");
    }

    #[test]
    fn from_str_matches_deserialize() {
        // A listed wire string builds its typed variant, never `Other`,
        // so a value built in code compares equal to the same value
        // decoded from the wire.
        assert_eq!(TestKind::from("foo"), TestKind::Foo);
        assert_eq!(TestKind::from(String::from("bar")), TestKind::Bar);
        assert_eq!(
            TestKind::from("vendor-extension-value"),
            TestKind::Other("vendor-extension-value".to_owned())
        );
        assert_eq!(
            TestKind::from(String::from("vendor-extension-value")),
            serde_json::from_str::<TestKind>(r#""vendor-extension-value""#).unwrap()
        );
    }

    #[test]
    fn empty_string_round_trips_as_other() {
        // Empty wire string is unknown to the registered list, so it lands in
        // `Other("")`. The macro must not panic or treat it specially.
        let parsed: TestKind = serde_json::from_str(r#""""#).unwrap();
        assert_eq!(parsed, TestKind::Other(String::new()));
        assert_eq!(serde_json::to_string(&parsed).unwrap(), r#""""#);
    }

    #[test]
    fn deserialize_rejects_non_string_json() {
        // Visitor only implements visit_str. Numbers, booleans, objects must
        // fail with a serde error rather than silently coercing.
        assert!(serde_json::from_str::<TestKind>("42").is_err());
        assert!(serde_json::from_str::<TestKind>("true").is_err());
        assert!(serde_json::from_str::<TestKind>("null").is_err());
        assert!(serde_json::from_str::<TestKind>("{}").is_err());
    }
}