Skip to content

Commit 08fea7b

Browse files
author
Devdutt Shenoi
committed
test: fix expectation
1 parent 5d87407 commit 08fea7b

File tree

1 file changed

+18
-28
lines changed

1 file changed

+18
-28
lines changed

src/event/format/json.rs

Lines changed: 18 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -665,7 +665,7 @@ mod tests {
665665
.to_data(None, None, None, SchemaVersion::V0)
666666
.unwrap();
667667
let (schema, _) =
668-
Event::infer_schema(&data[0], &store_schema, None, false, SchemaVersion::V0).unwrap();
668+
Event::infer_schema(&data[1], &store_schema, None, false, SchemaVersion::V0).unwrap();
669669
let rb =
670670
Event::into_recordbatch(Utc::now(), &data, &schema, None, SchemaVersion::V0).unwrap();
671671

@@ -718,7 +718,7 @@ mod tests {
718718
.to_data(None, None, None, SchemaVersion::V0)
719719
.unwrap();
720720
let (schema, _) =
721-
Event::infer_schema(&data[0], &store_schema, None, false, SchemaVersion::V0).unwrap();
721+
Event::infer_schema(&data[1], &store_schema, None, false, SchemaVersion::V0).unwrap();
722722
let rb =
723723
Event::into_recordbatch(Utc::now(), &data, &schema, None, SchemaVersion::V0).unwrap();
724724

@@ -792,23 +792,12 @@ mod tests {
792792

793793
#[test]
794794
fn arr_schema_mismatch() {
795-
let json = json!([
796-
{
797-
"a": null,
798-
"b": "hello",
799-
"c": 1.24
800-
},
801-
{
802-
"a": 1,
803-
"b": "hello",
804-
"c": 1
805-
},
806-
{
807-
"a": 1,
808-
"b": "hello",
809-
"c": null
810-
},
811-
]);
795+
let json = json!(
796+
{
797+
"a": 1,
798+
"b": "hello",
799+
"c": 1
800+
});
812801

813802
let store_schema = fields_to_map(
814803
[
@@ -819,13 +808,14 @@ mod tests {
819808
.into_iter(),
820809
);
821810

822-
let data = Event::new(json, 0 /* doesn't matter */, LogSource::Json)
823-
.to_data(None, None, None, SchemaVersion::V0)
824-
.unwrap();
825-
826-
assert!(
827-
Event::infer_schema(&data[0], &store_schema, None, false, SchemaVersion::V0).is_err()
828-
);
811+
assert!(Event::infer_schema(
812+
json.as_object().unwrap(),
813+
&store_schema,
814+
None,
815+
false,
816+
SchemaVersion::V0
817+
)
818+
.is_err());
829819
}
830820

831821
#[test]
@@ -857,7 +847,7 @@ mod tests {
857847
.to_data(None, None, None, SchemaVersion::V0)
858848
.unwrap();
859849
let (schema, _) =
860-
Event::infer_schema(&data[0], &store_schema, None, false, SchemaVersion::V0).unwrap();
850+
Event::infer_schema(&data[3], &store_schema, None, false, SchemaVersion::V0).unwrap();
861851
let rb =
862852
Event::into_recordbatch(Utc::now(), &data, &schema, None, SchemaVersion::V0).unwrap();
863853

@@ -935,7 +925,7 @@ mod tests {
935925
.to_data(None, None, None, SchemaVersion::V1)
936926
.unwrap();
937927
let (schema, _) =
938-
Event::infer_schema(&data[0], &store_schema, None, false, SchemaVersion::V1).unwrap();
928+
Event::infer_schema(&data[3], &store_schema, None, false, SchemaVersion::V1).unwrap();
939929
let rb =
940930
Event::into_recordbatch(Utc::now(), &data, &schema, None, SchemaVersion::V1).unwrap();
941931

0 commit comments

Comments
 (0)