fix: improve VecLogErr messages
This commit is contained in:
parent
9d243fa0ad
commit
9a652d851f
1 changed files with 10 additions and 19 deletions
|
|
@ -20,13 +20,6 @@ where
|
|||
where
|
||||
D: serde::Deserializer<'de>,
|
||||
{
|
||||
#[derive(serde::Deserialize)]
|
||||
#[serde(untagged)]
|
||||
enum GoodOrError<T> {
|
||||
Good(T),
|
||||
Error(serde_json::Value),
|
||||
}
|
||||
|
||||
struct SeqVisitor<T>(PhantomData<T>);
|
||||
|
||||
impl<'de, T> Visitor<'de> for SeqVisitor<T>
|
||||
|
|
@ -46,16 +39,14 @@ where
|
|||
let mut values = Vec::with_capacity(seq.size_hint().unwrap_or_default());
|
||||
let mut warnings = Vec::new();
|
||||
|
||||
while let Some(value) = seq.next_element()? {
|
||||
match value {
|
||||
GoodOrError::<T>::Good(value) => {
|
||||
values.push(value);
|
||||
}
|
||||
GoodOrError::<T>::Error(value) => {
|
||||
warnings.push(format!(
|
||||
"error deserializing item: {}",
|
||||
serde_json::to_string(&value).unwrap_or_default()
|
||||
));
|
||||
loop {
|
||||
match seq.next_element::<T>() {
|
||||
Ok(val) => match val {
|
||||
Some(val) => values.push(val),
|
||||
None => break,
|
||||
},
|
||||
Err(e) => {
|
||||
warnings.push(format!("error deserializing item: {e}"));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -186,8 +177,8 @@ mod tests {
|
|||
|
||||
insta::assert_debug_snapshot!(res.items.warnings, @r###"
|
||||
[
|
||||
"error deserializing item: {\"xyz\":\"i2\"}",
|
||||
"error deserializing item: {\"namra\":\"i4\"}",
|
||||
"error deserializing item: missing field `name` at line 1 column 40",
|
||||
"error deserializing item: missing field `name` at line 1 column 73",
|
||||
]
|
||||
"###);
|
||||
}
|
||||
|
|
|
|||
Reference in a new issue