More

apache · Aug 4, 2022 · badbabb · badbabb
1 parent dae94f2
commit badbabb
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/arrow/src/ipc/writer.rs b/arrow/src/ipc/writer.rs
@@ -1375,7 +1375,7 @@ mod tests {
     }
 
     #[test]
-    fn read_and_rewrite_generated_files_100() {
+    fn read_and_rewrite_generated_files_100() -> Result<()> {
         let testdata = crate::util::test_util::arrow_test_data();
         let version = "1.0.0-littleendian";
         // the test is repetitive, thus we can read all supported files at once
@@ -1434,12 +1434,13 @@ mod tests {
 
             // read expected JSON output
             let arrow_json = read_gzip_json(version, path);
-            assert!(arrow_json.equals_reader(&mut reader).unwrap());
+            assert!(arrow_json.equals_reader(&mut reader)?);
         });
+        Ok(())
     }
 
     #[test]
-    fn read_and_rewrite_generated_streams_100() {
+    fn read_and_rewrite_generated_streams_100() -> Result<()> {
         let testdata = crate::util::test_util::arrow_test_data();
         let version = "1.0.0-littleendian";
         // the test is repetitive, thus we can read all supported files at once
@@ -1495,8 +1496,10 @@ mod tests {
 
             // read expected JSON output
             let arrow_json = read_gzip_json(version, path);
-            assert!(arrow_json.equals_reader(&mut reader).unwrap());
+            assert!(arrow_json.equals_reader(&mut reader)?);
         });
+
+        OK(())
     }
 
     /// Read gzipped JSON file