refactor: remove common test utilities and replace with direct file path handling in tests

Author: excoffierleonard

benches/function_parse.rs

@@ -1,6 +1,7 @@
 use std::time::{Duration, Instant};
 
-use criterion::{BenchmarkId, Criterion, Throughput, black_box, criterion_group, criterion_main};
+use criterion::{BenchmarkId, Criterion, Throughput, criterion_group, criterion_main};
+use std::hint::black_box;
 use rayon::prelude::*;
 
 use parser::{ParserError, parse};
@@ -156,7 +157,7 @@ fn benchmark_parallel_threshold(c: &mut Criterion) {
 
     // Read each test file only once
     for &filename in TEST_FILESNAMES_BASE {
-        let file_extension = filename.split('.').last().unwrap_or("unknown");
+        let file_extension = filename.split('.').next_back().unwrap_or("unknown");
         let group_name = format!("Parallel {} Processing", file_extension.to_uppercase());
         let mut group = c.benchmark_group(&group_name);
 

tests/common/mod.rs

@@ -1,6 +1,4 @@
-mod common;
-
-use common::test_file_path;
+use std::path::PathBuf;
 
 // Note: The endpoint tests require the web module to be exposed publicly.
 // For now, we'll keep this as a placeholder. The web functionality can be tested
@@ -21,7 +19,9 @@ fn test_file_paths_exist() {
     ];
 
     for name in file_names {
-        let path = test_file_path(name);
+        let path = PathBuf::from(env!("CARGO_MANIFEST_DIR"))
+            .join("tests/assets")
+            .join(name);
         assert!(path.exists(), "Test file should exist: {:?}", path);
     }
 }

tests/endpoints.rs

@@ -1,6 +1,5 @@
-mod common;
+use std::{fs, path::PathBuf};
 
-use common::read_test_file;
 use parser::parse;
 use rayon::prelude::*;
 
@@ -67,7 +66,17 @@ grey07;2070;Laura;Grey"
 #[test]
 fn parse_success() {
     let (file_names, expected_texts) = get_test_data();
-    let data: Vec<Vec<u8>> = file_names.iter().map(|name| read_test_file(name)).collect();
+    let data: Vec<Vec<u8>> = file_names
+        .iter()
+        .map(|name| {
+            fs::read(
+                PathBuf::from(env!("CARGO_MANIFEST_DIR"))
+                    .join("tests/assets")
+                    .join(name),
+            )
+            .unwrap()
+        })
+        .collect();
 
     let result: Vec<String> = data.par_iter().map(|d| parse(d).unwrap()).collect();