Skip to content

Commit a7df122

Browse files
committed
Fixing failing tests
1 parent d4b2b9b commit a7df122

1 file changed

Lines changed: 17 additions & 3 deletions

File tree

vcs-worker/tests/operations/meta/filtering_update.rs

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,20 @@
11
//! Tests for filtering ignored properties and verbs during object update operations
22
33
use crate::common::*;
4+
use serde_json::Value;
5+
6+
/// Helper function to convert object/get response (list of strings) to a single string
7+
fn list_to_string(response: &Value) -> String {
8+
if let Some(list) = response.get_result_list() {
9+
list.iter()
10+
.filter_map(|v| v.as_str())
11+
.collect::<Vec<_>>()
12+
.join("\n")
13+
} else {
14+
// Fallback for errors which are still strings
15+
response.get_result_str().unwrap_or("").to_string()
16+
}
17+
}
418

519
#[tokio::test]
620
async fn test_object_update_filters_ignored_properties() {
@@ -34,7 +48,7 @@ async fn test_object_update_filters_ignored_properties() {
3448
.await
3549
.expect("Failed to get object");
3650

37-
let content = get_response.require_result_str("Get object");
51+
let content = list_to_string(&get_response);
3852

3953
// Verify test_property was filtered out during update
4054
assert!(
@@ -81,7 +95,7 @@ async fn test_object_update_filters_ignored_verbs() {
8195
.await
8296
.expect("Failed to get object");
8397

84-
let content = get_response.require_result_str("Get object");
98+
let content = list_to_string(&get_response);
8599

86100
// Verify test_verb was filtered out during update
87101
assert!(
@@ -132,7 +146,7 @@ async fn test_object_update_preserves_non_ignored_items() {
132146
.await
133147
.expect("Failed to get object");
134148

135-
let content = get_response.require_result_str("Get object");
149+
let content = list_to_string(&get_response);
136150

137151
// Verify ignored items are not present
138152
assert!(

0 commit comments

Comments
 (0)