Make CoreGenderClaim a string NewType instead of an enum

ramosbugs · Sep 12, 2019 · c2bccf9 · c2bccf9
1 parent fdf8360
commit c2bccf9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 14 deletions.
diff --git a/src/core/mod.rs b/src/core/mod.rs
@@ -442,21 +442,13 @@ pub enum CoreClientAuthMethod {
 }
 impl ClientAuthMethod for CoreClientAuthMethod {}
 
-///
-/// OpenID Connect Core gender claim.
-///
-#[derive(Clone, Debug, Deserialize, PartialEq, Serialize)]
-#[serde(rename_all = "snake_case")]
-pub enum CoreGenderClaim {
-    ///
-    /// Female
-    ///
-    Female,
+new_type![
     ///
-    /// Male
+    /// OpenID Connect Core gender claim.
     ///
-    Male,
-}
+    #[derive(Deserialize, Eq, Hash, Ord, PartialOrd, Serialize)]
+    CoreGenderClaim(String)
+];
 impl GenderClaim for CoreGenderClaim {}
 
 ///

diff --git a/src/id_token.rs b/src/id_token.rs
@@ -794,7 +794,7 @@ mod tests {
                 ),
                 email: Some(EndUserEmail::new("[email protected]".to_string())),
                 email_verified: Some(true),
-                gender: Some(CoreGenderClaim::Male),
+                gender: Some(CoreGenderClaim::new("male".to_string())),
                 birthday: Some(EndUserBirthday::new("1956-05-12".to_string())),
                 zoneinfo: Some(EndUserTimezone::new("America/Los_Angeles".to_string())),
                 locale: Some(LanguageTag::new("en-US".to_string())),