ruma-api-macros: Avoid empty POST/PUT request bodys

crates/ruma-api-macros/src/request/outgoing.rs

@@ -176,8 +176,10 @@ impl Request {
             quote! {
                 #ruma_serde::json_to_buf(&RequestBody { #initializers })?
             }
-        } else {
+        } else if method == "GET" {
             quote! { <T as ::std::default::Default>::default() }
+        } else {
+            quote! { #ruma_serde::slice_to_buf(b"{}") }
         };
 
         let (impl_generics, ty_generics, where_clause) = self.generics.split_for_impl();

crates/ruma-api/tests/no_fields.rs

@@ -1,33 +1,72 @@
-use ruma_api::{ruma_api, OutgoingRequest as _, OutgoingResponse as _, SendAccessToken};
+use ruma_api::{OutgoingRequest as _, OutgoingResponse as _, SendAccessToken};
 
-ruma_api! {
+mod get {
-    metadata: {
+    ruma_api::ruma_api! {
-        description: "Does something.",
+        metadata: {
-        method: GET,
+            description: "Does something.",
-        name: "no_fields",
+            method: GET,
-        path: "/_matrix/my/endpoint",
+            name: "no_fields",
-        rate_limited: false,
+            path: "/_matrix/my/endpoint",
-        authentication: None,
+            rate_limited: false,
+            authentication: None,
+        }
+
+        request: {}
+        response: {}
     }
+}
 
-    request: {}
+mod post {
-    response: {}
+    ruma_api::ruma_api! {
+        metadata: {
+            description: "Does something.",
+            method: POST,
+            name: "no_fields",
+            path: "/_matrix/my/endpoint",
+            rate_limited: false,
+            authentication: None,
+        }
+
+        request: {}
+        response: {}
+    }
 }
 
 #[test]
-fn empty_request_http_repr() {
+fn empty_post_request_http_repr() {
-    let req = Request {};
+    let req = post::Request {};
     let http_req = req
         .try_into_http_request::<Vec<u8>>("https://homeserver.tld", SendAccessToken::None)
         .unwrap();
 
+    // Empty POST requests should contain an empty dictionary as a body...
+    assert_eq!(http_req.body(), b"{}");
+}
+#[test]
+fn empty_get_request_http_repr() {
+    let req = get::Request {};
+    let http_req = req
+        .try_into_http_request::<Vec<u8>>("https://homeserver.tld", SendAccessToken::None)
+        .unwrap();
+
+    // ... but GET requests' bodies should be empty.
     assert!(http_req.body().is_empty());
 }
 
 #[test]
-fn empty_response_http_repr() {
+fn empty_post_response_http_repr() {
-    let res = Response {};
+    let res = post::Response {};
     let http_res = res.try_into_http_response::<Vec<u8>>().unwrap();
 
+    // For the reponse, the body should be an empty dict again...
+    assert_eq!(http_res.body(), b"{}");
+}
+
+#[test]
+fn empty_get_response_http_repr() {
+    let res = get::Response {};
+    let http_res = res.try_into_http_response::<Vec<u8>>().unwrap();
+
+    // ... even for GET requests.
     assert_eq!(http_res.body(), b"{}");
 }