This is an automated email from the ASF dual-hosted git repository. kriskras99 pushed a commit to branch chore/simplify_test_reader_invalid_block in repository https://gitbox.apache.org/repos/asf/avro-rs.git
commit 8c2445db2a5e03934e0bee6e8bc9c994920ba0c6 Author: Kriskras99 <[email protected]> AuthorDate: Fri Dec 5 11:51:45 2025 +0100 chore: Simplify reader tests --- avro/src/reader.rs | 20 ++++++-------------- 1 file changed, 6 insertions(+), 14 deletions(-) diff --git a/avro/src/reader.rs b/avro/src/reader.rs index 0bacfd8..23af5db 100644 --- a/avro/src/reader.rs +++ b/avro/src/reader.rs @@ -766,8 +766,8 @@ mod tests { #[test] fn test_reader_invalid_header() -> TestResult { let schema = Schema::parse_str(SCHEMA)?; - let invalid = ENCODED.iter().copied().skip(1).collect::<Vec<u8>>(); - assert!(Reader::with_schema(&schema, &invalid[..]).is_err()); + let mut invalid = &ENCODED[1..]; + assert!(Reader::with_schema(&schema, &mut invalid).is_err()); Ok(()) } @@ -775,16 +775,8 @@ mod tests { #[test] fn test_reader_invalid_block() -> TestResult { let schema = Schema::parse_str(SCHEMA)?; - let invalid = ENCODED - .iter() - .copied() - .rev() - .skip(19) - .collect::<Vec<u8>>() - .into_iter() - .rev() - .collect::<Vec<u8>>(); - let reader = Reader::with_schema(&schema, &invalid[..])?; + let mut invalid = &ENCODED[0..ENCODED.len() - 19]; + let reader = Reader::with_schema(&schema, &mut invalid)?; for value in reader { assert!(value.is_err()); } @@ -802,8 +794,8 @@ mod tests { #[test] fn test_reader_only_header() -> TestResult { - let invalid = ENCODED.iter().copied().take(165).collect::<Vec<u8>>(); - let reader = Reader::new(&invalid[..])?; + let mut invalid = &ENCODED[..165]; + let reader = Reader::new(&mut invalid)?; for value in reader { assert!(value.is_err()); }
