clippy fixes

andrewpatto · andrewpatto · commit 2ce4842fad51 · 2025-08-29T15:17:40.000+10:00
diff --git a/lambda-events/src/event/vpc_lattice/common.rs b/lambda-events/src/event/vpc_lattice/common.rs
@@ -1,4 +1,5 @@
 use crate::custom_serde::{deserialize_headers, serialize_headers};
+use crate::encodings::Body;
 use http::HeaderMap;
 use serde::{Deserialize, Serialize};
 #[cfg(feature = "catch-all-fields")]
@@ -30,7 +31,7 @@ pub struct VpcLatticeResponse {
 
     /// The response body
     #[serde(default)]
-    pub body: Option<String>,
+    pub body: Option<Body>,
 
     /// Catchall to catch any additional fields that were present but not explicitly defined by this struct.
     /// Enabled with Cargo feature `catch-all-fields`.
diff --git a/lambda-events/src/event/vpc_lattice/serialization_comma_separated_headers.rs b/lambda-events/src/event/vpc_lattice/serialization_comma_separated_headers.rs
@@ -185,8 +185,8 @@ mod tests {
         let r: RequestWithHeaders =
             serde_json::from_str("{ \"headers\": {\"x-foo\": \"z\", \"x-multi\": \"abcd, DEF, w\" }}").unwrap();
 
-        assert_eq!("z", r.headers.get_all("x-foo").iter().nth(0).unwrap());
-        assert_eq!("abcd", r.headers.get_all("x-multi").iter().nth(0).unwrap());
+        assert_eq!("z", r.headers.get_all("x-foo").iter().next().unwrap());
+        assert_eq!("abcd", r.headers.get_all("x-multi").iter().next().unwrap());
         assert_eq!("DEF", r.headers.get_all("x-multi").iter().nth(1).unwrap());
         assert_eq!("w", r.headers.get_all("x-multi").iter().nth(2).unwrap());
     }
diff --git a/lambda-events/src/event/vpc_lattice/v1.rs b/lambda-events/src/event/vpc_lattice/v1.rs
@@ -61,11 +61,11 @@ mod test {
         assert_eq!("POST", parsed.method);
         assert_eq!(
             "curl/7.68.0",
-            parsed.headers.get_all("user-agent").iter().nth(0).unwrap()
+            parsed.headers.get_all("user-agent").iter().next().unwrap()
         );
         assert_eq!("electronics", parsed.query_string_parameters.first("category").unwrap());
         assert_eq!("{\"id\": 5, \"description\": \"TV\"}", parsed.body.unwrap());
-        assert_eq!(false, parsed.is_base64_encoded);
+        assert!(!parsed.is_base64_encoded);
     }
 
     #[test]
@@ -74,7 +74,7 @@ mod test {
         let data = include_bytes!("../../fixtures/example-vpc-lattice-v1-request.json");
         let parsed: VpcLatticeRequestV1 = serde_json::from_slice(data).unwrap();
 
-        assert_eq!("abcd", parsed.headers.get_all("multi").iter().nth(0).unwrap());
+        assert_eq!("abcd", parsed.headers.get_all("multi").iter().next().unwrap());
         assert_eq!("DEF", parsed.headers.get_all("multi").iter().nth(1).unwrap());
     }
 
diff --git a/lambda-events/src/event/vpc_lattice/v2.rs b/lambda-events/src/event/vpc_lattice/v2.rs
@@ -154,23 +154,23 @@ mod test {
         assert_eq!("GET", parsed.method);
         assert_eq!(
             "curl/7.68.0",
-            parsed.headers.get_all("user-agent").iter().nth(0).unwrap()
+            parsed.headers.get_all("user-agent").iter().next().unwrap()
         );
 
         // headers including testing multi-values
         let header_multi = parsed.headers.get_all("multi");
-        assert_eq!("x", header_multi.iter().nth(0).unwrap());
+        assert_eq!("x", header_multi.iter().next().unwrap());
         assert_eq!("y", header_multi.iter().nth(1).unwrap());
 
         // query string including testing multi-values
         assert_eq!("prod", parsed.query_string_parameters.first("state").unwrap());
         let query_multi = parsed.query_string_parameters.all("multi").unwrap();
-        assert_eq!(&"a", query_multi.iter().nth(0).unwrap());
-        assert_eq!(&"DEF", query_multi.iter().nth(1).unwrap());
-        assert_eq!(&"g", query_multi.iter().nth(2).unwrap());
+        assert_eq!(&"a", query_multi.first().unwrap());
+        assert_eq!(&"DEF", query_multi.get(1).unwrap());
+        assert_eq!(&"g", query_multi.get(2).unwrap());
 
         assert!(parsed.body.is_none());
-        assert_eq!(false, parsed.is_base64_encoded);
+        assert!(!parsed.is_base64_encoded);
 
         // nested structure testing
         assert_eq!(
