(avro-rs) branch main updated: chore: Simplify reader tests (#355)

[mgrigorov]

This is an automated email from the ASF dual-hosted git repository.

mgrigorov pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/avro-rs.git


The following commit(s) were added to refs/heads/main by this push:
     new c6cbe79  chore: Simplify reader tests (#355)
c6cbe79 is described below

commit c6cbe7913a5afd558980e4b3d9c337de5874405a
Author: Kriskras99 <git...@kriskras99.nl>
AuthorDate: Fri Dec 5 12:12:34 2025 +0100

    chore: Simplify reader tests (#355)
---
 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());
         }