properly tested code
This commit is contained in:
@@ -108,15 +108,20 @@ async fn test_update_whitespace_fields() {
|
||||
request.firma = " Updated Company ".into();
|
||||
request.telefon = " +421987654321 ".into();
|
||||
|
||||
let _response = put_adresar(&pool, request).await.unwrap();
|
||||
let response = put_adresar(&pool, request).await.unwrap();
|
||||
|
||||
// Verify trimmed values in response
|
||||
assert_eq!(response.firma, "Updated Company");
|
||||
assert_eq!(response.telefon, "+421987654321");
|
||||
|
||||
// Verify raw values in database
|
||||
let db_record = sqlx::query!("SELECT firma, telefon FROM adresar WHERE id = $1", existing_id)
|
||||
.fetch_one(&pool)
|
||||
.await
|
||||
.unwrap();
|
||||
|
||||
assert_eq!(db_record.firma, " Updated Company ");
|
||||
assert_eq!(db_record.telefon.unwrap(), " +421987654321 ");
|
||||
assert_eq!(db_record.firma, "Updated Company"); // Trimmed
|
||||
assert_eq!(db_record.telefon.unwrap(), "+421987654321"); // Trimmed
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
@@ -127,7 +132,7 @@ async fn test_update_empty_required_field() {
|
||||
|
||||
let result = put_adresar(&pool, request).await;
|
||||
assert!(result.is_err());
|
||||
assert_eq!(result.unwrap_err().code(), tonic::Code::Internal);
|
||||
assert_eq!(result.unwrap_err().code(), tonic::Code::InvalidArgument); // Changed from Internal
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
@@ -162,15 +167,20 @@ async fn test_clear_optional_fields() {
|
||||
request.telefon = String::new();
|
||||
request.ulica = String::new();
|
||||
|
||||
let _response = put_adresar(&pool, request).await.unwrap();
|
||||
let response = put_adresar(&pool, request).await.unwrap();
|
||||
|
||||
// Check response contains empty strings
|
||||
assert!(response.telefon.is_empty());
|
||||
assert!(response.ulica.is_empty());
|
||||
|
||||
// Check database contains NULL
|
||||
let db_record = sqlx::query!("SELECT telefon, ulica FROM adresar WHERE id = $1", existing_id)
|
||||
.fetch_one(&pool)
|
||||
.await
|
||||
.unwrap();
|
||||
|
||||
assert_eq!(db_record.telefon.unwrap(), "");
|
||||
assert_eq!(db_record.ulica.unwrap(), "");
|
||||
assert!(db_record.telefon.is_none());
|
||||
assert!(db_record.ulica.is_none());
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
|
||||
Reference in New Issue
Block a user