From cc9633cc9536c5a1bb70fe0535793cc9b34b5ccb Mon Sep 17 00:00:00 2001 From: "Helmut K. C. Tessarek" Date: Mon, 4 Sep 2023 05:09:29 -0400 Subject: [PATCH] refactor(server): change response to 'file deleted' (#137) --- fixtures/test-file-delete/test.sh | 2 +- src/server.rs | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fixtures/test-file-delete/test.sh b/fixtures/test-file-delete/test.sh index 75fb087..3ab68a8 100755 --- a/fixtures/test-file-delete/test.sh +++ b/fixtures/test-file-delete/test.sh @@ -9,7 +9,7 @@ setup() { run_test() { file_url=$(curl -s -F "file=@file" localhost:8000) test "$file_url" = "http://localhost:8000/file.txt" - test "the file is deleted" = "$(curl -s -H "Authorization: may_the_force_be_with_you" -X DELETE http://localhost:8000/file.txt)" + test "file deleted" = "$(curl -s -H "Authorization: may_the_force_be_with_you" -X DELETE http://localhost:8000/file.txt)" test "file is not found or expired :(" = "$(curl -s -H "Authorization: may_the_force_be_with_you" -X DELETE http://localhost:8000/file.txt)" } diff --git a/src/server.rs b/src/server.rs index 819881f..28788a3 100644 --- a/src/server.rs +++ b/src/server.rs @@ -176,7 +176,7 @@ async fn delete( return Err(error::ErrorInternalServerError("cannot delete file")); } } - Ok(HttpResponse::Ok().body(String::from("the file is deleted\n"))) + Ok(HttpResponse::Ok().body(String::from("file deleted\n"))) } /// Expose version endpoint @@ -788,6 +788,7 @@ mod tests { let response = test::call_service(&app, request).await; assert_eq!(StatusCode::OK, response.status()); + assert_body(response.into_body(), "file deleted\n").await?; let path = PathBuf::from(file_name); assert!(!path.exists());