Remove unused API error constructor

Author: YinMo19

src/models/api/error.rs

@@ -16,19 +16,6 @@ pub struct ApiError {
     pub trace_id: Option<String>,
 }
 
-impl ApiError {
-    /// Creates a new API error.
-    pub fn new(code: u32, message: impl Into<String>) -> Self {
-        Self {
-            code,
-            err_code: None,
-            message: message.into(),
-            errors: None,
-            trace_id: None,
-        }
-    }
-}
-
 impl<'de> Deserialize<'de> for ApiError {
     fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
     where

src/models/api/tests.rs

@@ -16,11 +16,21 @@ fn test_rate_limit() {
 
 #[test]
 fn test_api_error() {
-    let error = ApiError::new(429, "Rate limited");
+    let error: ApiError = serde_json::from_value(serde_json::json!({
+        "code": 429,
+        "message": "Rate limited"
+    }))
+    .unwrap();
+
     assert_eq!(error.err_code, None);
     assert_eq!(error.code, 429);
+    assert_eq!(error.message, "Rate limited");
 
-    let auth_error = ApiError::new(401, "Unauthorized");
+    let auth_error: ApiError = serde_json::from_value(serde_json::json!({
+        "code": 401,
+        "message": "Unauthorized"
+    }))
+    .unwrap();
     assert_eq!(auth_error.code, 401);
 }