feat: Skip some field in user serialization and added test

Author: mitsuhiko

src/protocol/v7.rs

@@ -292,12 +292,16 @@ impl Default for Breadcrumb {
 #[serde(default)]
 pub struct User {
     /// The ID of the user.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub id: Option<String>,
     /// The email address of the user.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub email: Option<String>,
     /// The remote ip address of the user.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub ip_address: Option<IpAddr>,
     /// A human readable username of the user.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub username: Option<String>,
     /// Additional data that should be send along.
     #[serde(flatten)]

tests/test_protocol_v7.rs

@@ -182,6 +182,47 @@ fn test_repos() {
     );
 }
 
+#[test]
+fn test_user() {
+    let event = v7::Event {
+        user: Some(v7::User {
+            id: Some("8fd5a33b-5b0e-45b2-aff2-9e4f067756ba".into()),
+            email: Some("[email protected]".into()),
+            ip_address: Some("127.0.0.1".parse().unwrap()),
+            username: Some("john-doe".into()),
+            data: {
+                let mut hm = HashMap::new();
+                hm.insert("foo".into(), "bar".into());
+                hm
+            }
+        }),
+        ..Default::default()
+    };
+
+    assert_eq!(
+        serde_json::to_string(&event).unwrap(),
+        "{\"user\":{\"id\":\"8fd5a33b-5b0e-45b2-aff2-9e4f067756ba\",\
+         \"email\":\"[email protected]\",\"ip_address\":\"127.0.0.1\",\
+         \"username\":\"john-doe\",\"foo\":\"bar\"}}"
+    );
+
+    let event = v7::Event {
+        user: Some(v7::User {
+            id: Some("8fd5a33b-5b0e-45b2-aff2-9e4f067756ba".into()),
+            email: None,
+            ip_address: None,
+            username: None,
+            ..Default::default()
+        }),
+        ..Default::default()
+    };
+
+    assert_eq!(
+        serde_json::to_string(&event).unwrap(),
+        "{\"user\":{\"id\":\"8fd5a33b-5b0e-45b2-aff2-9e4f067756ba\"}}"
+    );
+}
+
 #[test]
 fn test_canonical_exception() {
     let mut event: v7::Event = Default::default();