kreuzberg 4.1.2 → 4.2.0

data/vendor/kreuzberg/tests/pptx_regression_tests.rs

@@ -25,9 +25,9 @@ use zip::write::{FileOptions, ZipWriter};
 async fn test_ppsx_slideshow_extraction() {
     let workspace_root = std::path::Path::new(env!("CARGO_MANIFEST_DIR"))
         .parent()
-        .unwrap()
+        .expect("Operation failed")
         .parent()
-        .unwrap();
+        .expect("Operation failed");
     let test_file = workspace_root.join("test_documents/presentations/sample.ppsx");
 
     if !test_file.exists() {
@@ -69,9 +69,9 @@ async fn test_ppsx_slideshow_extraction() {
 async fn test_ppsx_with_explicit_mime_type() {
     let workspace_root = std::path::Path::new(env!("CARGO_MANIFEST_DIR"))
         .parent()
-        .unwrap()
+        .expect("Operation failed")
         .parent()
-        .unwrap();
+        .expect("Operation failed");
     let test_file = workspace_root.join("test_documents/presentations/sample.ppsx");
 
     if !test_file.exists() {
@@ -120,24 +120,26 @@ async fn test_pptx_with_image_placeholder_no_txbody() {
         let options: FileOptions<()> = FileOptions::default().compression_method(CompressionMethod::Stored);
 
         // Add [Content_Types].xml
-        zip.start_file("[Content_Types].xml", options).unwrap();
+        zip.start_file("[Content_Types].xml", options)
+            .expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types">
   <Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/>
   <Default Extension="xml" ContentType="application/xml"/>
   <Override PartName="/ppt/presentation.xml" ContentType="application/vnd.openxmlformats-officedocument.presentationml.presentation.main+xml"/>
   <Override PartName="/ppt/slides/slide1.xml" ContentType="application/vnd.openxmlformats-officedocument.presentationml.slide+xml"/>
-</Types>"#).unwrap();
+</Types>"#).expect("Operation failed");
 
         // Add _rels/.rels
-        zip.start_file("_rels/.rels", options).unwrap();
+        zip.start_file("_rels/.rels", options).expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
   <Relationship Id="rId1" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument" Target="ppt/presentation.xml"/>
-</Relationships>"#).unwrap();
+</Relationships>"#).expect("Operation failed");
 
         // Add ppt/presentation.xml
-        zip.start_file("ppt/presentation.xml", options).unwrap();
+        zip.start_file("ppt/presentation.xml", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <p:presentation xmlns:p="http://schemas.openxmlformats.org/presentationml/2006/main"
@@ -148,18 +150,20 @@ async fn test_pptx_with_image_placeholder_no_txbody() {
   </p:sldIdLst>
 </p:presentation>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
         // Add ppt/_rels/presentation.xml.rels
-        zip.start_file("ppt/_rels/presentation.xml.rels", options).unwrap();
+        zip.start_file("ppt/_rels/presentation.xml.rels", options)
+            .expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
   <Relationship Id="rId2" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/slide" Target="slides/slide1.xml"/>
-</Relationships>"#).unwrap();
+</Relationships>"#).expect("Operation failed");
 
         // Add ppt/slides/slide1.xml with a shape WITHOUT txBody (image placeholder)
         // This is the critical test case - a <p:sp> element with no <p:txBody>
-        zip.start_file("ppt/slides/slide1.xml", options).unwrap();
+        zip.start_file("ppt/slides/slide1.xml", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <p:sld xmlns:p="http://schemas.openxmlformats.org/presentationml/2006/main"
@@ -259,18 +263,19 @@ async fn test_pptx_with_image_placeholder_no_txbody() {
   </p:cSld>
 </p:sld>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
         // Add ppt/slides/_rels/slide1.xml.rels (empty)
-        zip.start_file("ppt/slides/_rels/slide1.xml.rels", options).unwrap();
+        zip.start_file("ppt/slides/_rels/slide1.xml.rels", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
 </Relationships>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     // Extract the PPTX file
@@ -336,24 +341,26 @@ async fn test_pptx_mixed_shapes_extraction() {
         let options: FileOptions<()> = FileOptions::default().compression_method(CompressionMethod::Stored);
 
         // Add [Content_Types].xml
-        zip.start_file("[Content_Types].xml", options).unwrap();
+        zip.start_file("[Content_Types].xml", options)
+            .expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Types xmlns="http://schemas.openxmlformats.org/package/2006/content-types">
   <Default Extension="rels" ContentType="application/vnd.openxmlformats-package.relationships+xml"/>
   <Default Extension="xml" ContentType="application/xml"/>
   <Override PartName="/ppt/presentation.xml" ContentType="application/vnd.openxmlformats-officedocument.presentationml.presentation.main+xml"/>
   <Override PartName="/ppt/slides/slide1.xml" ContentType="application/vnd.openxmlformats-officedocument.presentationml.slide+xml"/>
-</Types>"#).unwrap();
+</Types>"#).expect("Operation failed");
 
         // Add _rels/.rels
-        zip.start_file("_rels/.rels", options).unwrap();
+        zip.start_file("_rels/.rels", options).expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
   <Relationship Id="rId1" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/officeDocument" Target="ppt/presentation.xml"/>
-</Relationships>"#).unwrap();
+</Relationships>"#).expect("Operation failed");
 
         // Add ppt/presentation.xml
-        zip.start_file("ppt/presentation.xml", options).unwrap();
+        zip.start_file("ppt/presentation.xml", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <p:presentation xmlns:p="http://schemas.openxmlformats.org/presentationml/2006/main"
@@ -364,17 +371,19 @@ async fn test_pptx_mixed_shapes_extraction() {
   </p:sldIdLst>
 </p:presentation>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
         // Add ppt/_rels/presentation.xml.rels
-        zip.start_file("ppt/_rels/presentation.xml.rels", options).unwrap();
+        zip.start_file("ppt/_rels/presentation.xml.rels", options)
+            .expect("Operation failed");
         zip.write_all(br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
   <Relationship Id="rId2" Type="http://schemas.openxmlformats.org/officeDocument/2006/relationships/slide" Target="slides/slide1.xml"/>
-</Relationships>"#).unwrap();
+</Relationships>"#).expect("Operation failed");
 
         // Add slide with various shapes - some with txBody, some without
-        zip.start_file("ppt/slides/slide1.xml", options).unwrap();
+        zip.start_file("ppt/slides/slide1.xml", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <p:sld xmlns:p="http://schemas.openxmlformats.org/presentationml/2006/main"
@@ -453,18 +462,19 @@ async fn test_pptx_mixed_shapes_extraction() {
   </p:cSld>
 </p:sld>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
         // Add empty rels
-        zip.start_file("ppt/slides/_rels/slide1.xml.rels", options).unwrap();
+        zip.start_file("ppt/slides/_rels/slide1.xml.rels", options)
+            .expect("Operation failed");
         zip.write_all(
             br#"<?xml version="1.0" encoding="UTF-8"?>
 <Relationships xmlns="http://schemas.openxmlformats.org/package/2006/relationships">
 </Relationships>"#,
         )
-        .unwrap();
+        .expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let result = extract_file(

data/vendor/kreuzberg/tests/registry_integration_tests.rs

@@ -184,9 +184,9 @@ fn test_register_multiple_validators_succeeds() {
         should_fail: true,
     });
 
-    registry.register(v1).unwrap();
-    registry.register(v2).unwrap();
-    registry.register(v3).unwrap();
+    registry.register(v1).expect("Operation failed");
+    registry.register(v2).expect("Operation failed");
+    registry.register(v3).expect("Operation failed");
 
     let list = registry.list();
     assert_eq!(list.len(), 3, "Should have three validators");
@@ -205,7 +205,7 @@ fn test_validator_unregistration_succeeds() {
         should_fail: false,
     });
 
-    registry.register(validator).unwrap();
+    registry.register(validator).expect("Operation failed");
     assert_eq!(registry.list().len(), 1);
 
     let result = registry.remove("temp-validator");
@@ -298,8 +298,8 @@ fn test_clear_validators_succeeds() {
         should_fail: false,
     });
 
-    registry.register(v1).unwrap();
-    registry.register(v2).unwrap();
+    registry.register(v1).expect("Operation failed");
+    registry.register(v2).expect("Operation failed");
     assert_eq!(registry.list().len(), 2);
 
     let result = registry.shutdown_all();
@@ -355,9 +355,9 @@ fn test_get_all_validators_respects_priority() {
         priority: 100,
     });
 
-    registry.register(medium).unwrap();
-    registry.register(low).unwrap();
-    registry.register(high).unwrap();
+    registry.register(medium).expect("Operation failed");
+    registry.register(low).expect("Operation failed");
+    registry.register(high).expect("Operation failed");
 
     let all = registry.get_all();
     assert_eq!(all.len(), 3, "Should have three validators");
@@ -397,11 +397,11 @@ fn test_get_extractor_by_mime_type_succeeds() {
         priority: 50,
     });
 
-    registry.register(extractor).unwrap();
+    registry.register(extractor).expect("Operation failed");
 
     let result = registry.get("application/pdf");
     assert!(result.is_ok(), "Should find extractor for PDF");
-    assert_eq!(result.unwrap().name(), "pdf-extractor");
+    assert_eq!(result.expect("Operation failed").name(), "pdf-extractor");
 }
 
 /// Test extractor not found for unsupported MIME type.
@@ -437,10 +437,10 @@ fn test_extractor_priority_selection() {
         priority: 100,
     });
 
-    registry.register(low_priority).unwrap();
-    registry.register(high_priority).unwrap();
+    registry.register(low_priority).expect("Operation failed");
+    registry.register(high_priority).expect("Operation failed");
 
-    let result = registry.get("text/plain").unwrap();
+    let result = registry.get("text/plain").expect("Value not found");
     assert_eq!(
         result.name(),
         "high-priority-extractor",
@@ -459,15 +459,15 @@ fn test_extractor_wildcard_mime_matching() {
         priority: 50,
     });
 
-    registry.register(extractor).unwrap();
+    registry.register(extractor).expect("Operation failed");
 
     let result = registry.get("text/plain");
     assert!(result.is_ok(), "Should match text/plain with text/*");
-    assert_eq!(result.unwrap().name(), "text-extractor");
+    assert_eq!(result.expect("Operation failed").name(), "text-extractor");
 
     let result = registry.get("text/html");
     assert!(result.is_ok(), "Should match text/html with text/*");
-    assert_eq!(result.unwrap().name(), "text-extractor");
+    assert_eq!(result.expect("Operation failed").name(), "text-extractor");
 
     let result = registry.get("application/pdf");
     assert!(result.is_err(), "Should not match application/pdf with text/*");
@@ -484,7 +484,7 @@ fn test_extractor_unregistration_succeeds() {
         priority: 50,
     });
 
-    registry.register(extractor).unwrap();
+    registry.register(extractor).expect("Operation failed");
     assert_eq!(registry.list().len(), 1);
 
     let result = registry.remove("temp-extractor");
@@ -506,17 +506,20 @@ fn test_extractor_multiple_mime_types() {
         priority: 50,
     });
 
-    registry.register(extractor).unwrap();
+    registry.register(extractor).expect("Operation failed");
 
     assert!(registry.get("application/pdf").is_ok());
     assert!(registry.get("application/vnd.ms-excel").is_ok());
     assert!(registry.get("text/csv").is_ok());
 
     assert_eq!(
-        registry.get("application/pdf").unwrap().name(),
+        registry.get("application/pdf").expect("Value not found").name(),
+        "multi-format-extractor"
+    );
+    assert_eq!(
+        registry.get("text/csv").expect("Value not found").name(),
         "multi-format-extractor"
     );
-    assert_eq!(registry.get("text/csv").unwrap().name(), "multi-format-extractor");
 }
 
 /// Test clearing all extractors.
@@ -535,8 +538,8 @@ fn test_clear_extractors_succeeds() {
         priority: 50,
     });
 
-    registry.register(e1).unwrap();
-    registry.register(e2).unwrap();
+    registry.register(e1).expect("Operation failed");
+    registry.register(e2).expect("Operation failed");
     assert_eq!(registry.list().len(), 2);
 
     let result = registry.shutdown_all();

data/vendor/kreuzberg/tests/rst_extractor_tests.rs

@@ -686,7 +686,7 @@ async fn test_rst_extraction_no_errors() {
         result.err()
     );
 
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Extracted content should not be empty");
 

data/vendor/kreuzberg/tests/rtf_extractor_tests.rs

@@ -74,7 +74,7 @@ async fn test_rtf_accent_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for accent.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert_eq!(extraction.mime_type, "application/rtf");
 
@@ -112,7 +112,7 @@ async fn test_rtf_bookmark_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for bookmark.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     let content = extraction.content.to_lowercase();
 
@@ -137,7 +137,7 @@ async fn test_rtf_footnote_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for footnote.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -176,7 +176,7 @@ async fn test_rtf_formatting_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for formatting.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -223,7 +223,7 @@ async fn test_rtf_heading_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for heading.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -269,7 +269,7 @@ async fn test_rtf_image_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for image.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -301,7 +301,7 @@ async fn test_rtf_link_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for link.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -328,7 +328,7 @@ async fn test_rtf_list_complex_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for list_complex.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -381,7 +381,7 @@ async fn test_rtf_list_simple_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for list_simple.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -422,7 +422,7 @@ async fn test_rtf_table_error_codes_extraction() {
         result.is_ok(),
         "RTF extraction should succeed for table_error_codes.rtf"
     );
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(
         extraction.mime_type == "application/rtf",
@@ -448,7 +448,7 @@ async fn test_rtf_table_simple_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for table_simple.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(
         extraction.mime_type == "application/rtf",
@@ -470,7 +470,7 @@ async fn test_rtf_unicode_extraction() {
     let result = extract_file(&path, Some("application/rtf"), &config).await;
 
     assert!(result.is_ok(), "RTF extraction should succeed for unicode.rtf");
-    let extraction = result.unwrap();
+    let extraction = result.expect("Operation failed");
 
     assert!(!extraction.content.is_empty(), "Content should not be empty");
 
@@ -493,8 +493,8 @@ async fn test_rtf_extraction_deterministic_unicode() {
 
     assert!(result1.is_ok() && result2.is_ok(), "Both extractions should succeed");
 
-    let extraction1 = result1.unwrap();
-    let extraction2 = result2.unwrap();
+    let extraction1 = result1.expect("Operation failed");
+    let extraction2 = result2.expect("Operation failed");
 
     assert_eq!(
         extraction1.content, extraction2.content,
@@ -514,8 +514,8 @@ async fn test_rtf_extraction_deterministic_list_complex() {
 
     assert!(result1.is_ok() && result2.is_ok(), "Both extractions should succeed");
 
-    let extraction1 = result1.unwrap();
-    let extraction2 = result2.unwrap();
+    let extraction1 = result1.expect("Operation failed");
+    let extraction2 = result2.expect("Operation failed");
 
     assert_eq!(
         extraction1.content, extraction2.content,
@@ -551,7 +551,7 @@ async fn test_rtf_no_critical_content_loss() {
             filename
         );
 
-        let extraction = result.unwrap();
+        let extraction = result.expect("Operation failed");
         assert!(
             !extraction.content.is_empty(),
             "FAIL: CRITICAL - Extracted 0 bytes from {}. RTF extractor lost all content.",
@@ -582,7 +582,7 @@ async fn test_rtf_mime_type_preservation() {
 
         assert!(result.is_ok(), "Extraction should succeed for {}", filename);
 
-        let extraction = result.unwrap();
+        let extraction = result.expect("Operation failed");
         assert_eq!(
             extraction.mime_type, "application/rtf",
             "FAIL: MIME type not preserved for {}",

data/vendor/kreuzberg/tests/security_validation.rs

@@ -31,11 +31,11 @@ fn test_archive_zip_bomb_detection() {
         let mut zip = ZipWriter::new(&mut cursor);
         let options = FileOptions::<'_, ()>::default();
 
-        zip.start_file("large.txt", options).unwrap();
+        zip.start_file("large.txt", options).expect("Operation failed");
         let zeros = vec![0u8; 10 * 1024 * 1024];
-        zip.write_all(&zeros).unwrap();
+        zip.write_all(&zeros).expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let bytes = cursor.into_inner();
@@ -57,10 +57,10 @@ fn test_archive_path_traversal_zip() {
         let mut zip = ZipWriter::new(&mut cursor);
         let options = FileOptions::<'_, ()>::default();
 
-        zip.start_file("../../etc/passwd", options).unwrap();
-        zip.write_all(b"malicious content").unwrap();
+        zip.start_file("../../etc/passwd", options).expect("Operation failed");
+        zip.write_all(b"malicious content").expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let bytes = cursor.into_inner();
@@ -97,10 +97,10 @@ fn test_archive_absolute_paths_rejected() {
         let mut zip = ZipWriter::new(&mut cursor);
         let options = FileOptions::<'_, ()>::default();
 
-        zip.start_file("/tmp/malicious.txt", options).unwrap();
-        zip.write_all(b"malicious content").unwrap();
+        zip.start_file("/tmp/malicious.txt", options).expect("Operation failed");
+        zip.write_all(b"malicious content").expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let bytes = cursor.into_inner();
@@ -125,10 +125,10 @@ fn test_archive_deeply_nested_directories() {
         let deep_path = (0..100).map(|i| format!("dir{}", i)).collect::<Vec<_>>().join("/");
         let file_path = format!("{}/file.txt", deep_path);
 
-        zip.start_file(&file_path, options).unwrap();
-        zip.write_all(b"deep content").unwrap();
+        zip.start_file(&file_path, options).expect("Operation failed");
+        zip.write_all(b"deep content").expect("Operation failed");
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let bytes = cursor.into_inner();
@@ -149,11 +149,12 @@ fn test_archive_many_small_files() {
         let options = FileOptions::<'_, ()>::default();
 
         for i in 0..1000 {
-            zip.start_file(format!("file{}.txt", i), options).unwrap();
-            zip.write_all(b"small content").unwrap();
+            zip.start_file(format!("file{}.txt", i), options)
+                .expect("Operation failed");
+            zip.write_all(b"small content").expect("Operation failed");
         }
 
-        zip.finish().unwrap();
+        zip.finish().expect("Operation failed");
     }
 
     let bytes = cursor.into_inner();
@@ -404,13 +405,13 @@ fn test_security_directory_instead_of_file() {
 
 #[test]
 fn test_security_special_file_handling() {
-    let mut tmpfile = NamedTempFile::new().unwrap();
-    tmpfile.write_all(b"test content").unwrap();
-    tmpfile.flush().unwrap();
+    let mut tmpfile = NamedTempFile::new().expect("Operation failed");
+    tmpfile.write_all(b"test content").expect("Operation failed");
+    tmpfile.flush().expect("Operation failed");
     let path = tmpfile.path();
 
     let config = ExtractionConfig::default();
-    let result = extract_file_sync(path.to_str().unwrap(), None, &config);
+    let result = extract_file_sync(path.to_str().expect("Operation failed"), None, &config);
 
     assert!(result.is_ok() || result.is_err());
 }