From 780f3899734d1266abe7957719a4492e13bd2c1c Mon Sep 17 00:00:00 2001 From: Gwen Lg Date: Thu, 29 Jan 2026 00:09:54 +0100 Subject: [PATCH] test: replace some unwrap with expect in tests this add more information in case of failure --- src/api/s3/cors.rs | 3 ++- src/api/s3/website.rs | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/api/s3/cors.rs b/src/api/s3/cors.rs index d746ff8d..1e3f6a0d 100644 --- a/src/api/s3/cors.rs +++ b/src/api/s3/cors.rs @@ -228,7 +228,8 @@ mod tests { * "#; - let conf: CorsConfiguration = from_str(message).unwrap(); + let conf: CorsConfiguration = + from_str(message).expect("failed to deserialize xml into `CorsConfiguration` struct"); let ref_value = CorsConfiguration { xmlns: (), cors_rules: vec![ diff --git a/src/api/s3/website.rs b/src/api/s3/website.rs index 18da3670..d4f0e37c 100644 --- a/src/api/s3/website.rs +++ b/src/api/s3/website.rs @@ -416,7 +416,8 @@ mod tests { "#; - let conf: WebsiteConfiguration = from_str(message).unwrap(); + let conf: WebsiteConfiguration = + from_str(message).expect("failed to deserialize xml in `WebsiteConfiguration`"); let ref_value = WebsiteConfiguration { xmlns: (), error_document: Some(Key {