Remove message event default constructors

Author: YinMo19

src/models/message/event/audit.rs

@@ -29,10 +29,3 @@ pub struct MessageAudit {
     #[serde(skip)]
     pub event_id: Option<String>,
 }
-
-impl MessageAudit {
-    /// Creates a new message audit.
-    pub fn new() -> Self {
-        Self::default()
-    }
-}

src/models/message/event/c2c.rs

@@ -6,7 +6,7 @@ use serde_json::Value;
 use crate::models::message::C2CMessageParams;
 
 /// Represents a C2C (client-to-client) message.
-#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize, Default)]
 pub struct C2CMessage {
     /// The message's unique ID
     pub id: Option<String>,
@@ -34,22 +34,6 @@ pub struct C2CMessage {
 }
 
 impl C2CMessage {
-    /// Creates a new C2C message.
-    pub fn new() -> Self {
-        Self {
-            id: None,
-            content: None,
-            message_reference: None,
-            mentions: Vec::new(),
-            attachments: Vec::new(),
-            msg_seq: None,
-            timestamp: None,
-            author: None,
-            message_scene: None,
-            event_id: None,
-        }
-    }
-
     /// Reply to this C2C message
     pub async fn reply(
         &self,
@@ -76,9 +60,3 @@ impl C2CMessage {
         }
     }
 }
-
-impl Default for C2CMessage {
-    fn default() -> Self {
-        Self::new()
-    }
-}

src/models/message/event/direct.rs

@@ -33,9 +33,3 @@ impl DirectMessageToCreate {
         }
     }
 }
-impl DirectMessage {
-    /// Creates a new direct-message session.
-    pub fn new() -> Self {
-        Self::default()
-    }
-}

src/models/message/event/group.rs

@@ -5,7 +5,7 @@ use serde::{Deserialize, Serialize};
 use crate::models::message::GroupMessageParams;
 
 /// Represents a group message.
-#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize, Default)]
 pub struct GroupMessage {
     /// The message's unique ID
     pub id: Option<Snowflake>,
@@ -33,22 +33,6 @@ pub struct GroupMessage {
 }
 
 impl GroupMessage {
-    /// Creates a new group message.
-    pub fn new() -> Self {
-        Self {
-            id: None,
-            content: None,
-            message_reference: None,
-            mentions: Vec::new(),
-            attachments: Vec::new(),
-            msg_seq: None,
-            timestamp: None,
-            author: None,
-            group_openid: None,
-            event_id: None,
-        }
-    }
-
     /// Reply to this group message
     pub async fn reply(
         &self,
@@ -71,9 +55,3 @@ impl GroupMessage {
         }
     }
 }
-
-impl Default for GroupMessage {
-    fn default() -> Self {
-        Self::new()
-    }
-}

src/models/message/event/message.rs

@@ -5,7 +5,7 @@ use serde::{Deserialize, Serialize};
 use crate::models::message::{Ark, Embed, MessageParams};
 
 /// Represents a message in a guild channel.
-#[derive(Debug, Clone, PartialEq, Serialize, Deserialize)]
+#[derive(Debug, Clone, PartialEq, Serialize, Deserialize, Default)]
 pub struct Message {
     /// The message's unique ID
     pub id: Option<Snowflake>,
@@ -60,35 +60,6 @@ pub struct Message {
 }
 
 impl Message {
-    /// Creates a new message.
-    pub fn new() -> Self {
-        Self {
-            id: None,
-            content: None,
-            channel_id: None,
-            guild_id: None,
-            group_id: None,
-            author: None,
-            member: None,
-            message_reference: None,
-            mentions: Vec::new(),
-            attachments: Vec::new(),
-            embeds: Vec::new(),
-            ark: None,
-            direct_message: None,
-            seq: None,
-            seq_in_channel: None,
-            timestamp: None,
-            edited_timestamp: None,
-            mention_everyone: None,
-            src_guild_id: None,
-            file_info: None,
-            ttl: None,
-            message_scene: None,
-            event_id: None,
-        }
-    }
-
     /// Reply to this message
     pub async fn reply(
         &self,
@@ -111,12 +82,6 @@ impl Message {
     }
 }
 
-impl Default for Message {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl HasId for Message {
     fn id(&self) -> Option<&Snowflake> {
         self.id.as_ref()

src/models/message/tests.rs

@@ -516,7 +516,7 @@ fn messages_pager_omits_empty_query_params() {
 
 #[test]
 fn test_message_creation() {
-    let message = Message::new();
+    let message = Message::default();
     assert!(message.id.is_none());
     assert!(message.content.is_none());
     assert!(message.attachments.is_empty());
@@ -568,8 +568,10 @@ fn direct_message_session_uses_required_zero_value_fields() {
 
 #[test]
 fn test_message_with_content() {
-    let mut message = Message::new();
-    message.content = Some("Hello, world!".to_string());
+    let message = Message {
+        content: Some("Hello, world!".to_string()),
+        ..Default::default()
+    };
     assert!(
         message
             .content
@@ -708,13 +710,15 @@ fn message_attachment_omits_go_zero_values() {
 
 #[test]
 fn test_bot_detection() {
-    let mut message = Message::new();
-    message.author = Some(MessageUser {
-        id: Some("123".to_string()),
-        username: Some("Bot".to_string()),
-        bot: Some(true),
-        avatar: None,
-    });
+    let mut message = Message {
+        author: Some(MessageUser {
+            id: Some("123".to_string()),
+            username: Some("Bot".to_string()),
+            bot: Some(true),
+            avatar: None,
+        }),
+        ..Default::default()
+    };
 
     assert!(
         message