windows working. fix test helper

Author: acunniffe

tests/ai_tab.rs

@@ -350,24 +350,17 @@ fn test_ai_tab_e2e_handles_dirty_files_map() {
         }),
     );
 
-    // // Debug: Check working logs before commit
-    // let working_logs = repo.current_working_logs();
-    // if let Ok(checkpoints) = working_logs.read_all_checkpoints() {
-    //     println!("Checkpoints before commit: {}", checkpoints.len());
-    //     for (i, cp) in checkpoints.iter().enumerate() {
-    //         println!("Checkpoint {}: kind={:?}, entries={}", i, cp.kind, cp.entries.len());
-    //         for entry in &cp.entries {
-    //             println!("  File: {}, attributions={}", entry.file, entry.attributions.len());
-    //         }
-    //     }
-    // }
 
     let working_logs = repo.current_working_logs();
     println!("dirty_files: {:?}", working_logs.dirty_files);
 
-    repo.stage_all_and_commit("Record AI tab completion while other files dirty").unwrap();
+    let commit_result = repo.stage_all_and_commit("Record AI tab completion while other files dirty").unwrap();
+    println!("COMMIT OUTPUT: {}", commit_result.stdout);
 
-    let mut file = repo.filename(&std::path::Path::new("src").join("lib.rs").to_string_lossy());
+
+    commit_result.print_authorship();
+
+    let mut file = repo.filename("src/lib.rs");
     file.assert_lines_and_blame(lines![
         "fn greet() {".human(),
         "    println!(\"hello\");".human(),

tests/repos/test_file.rs

@@ -120,8 +120,12 @@ impl<'a> TestFile<'a> {
             };
         }
 
-        // Run blame to get authorship
-        let filename = file_path.to_str().expect("valid path");
+        // Run blame to get authorship - use relative path from repo root
+        let filename = file_path
+            .strip_prefix(&repo.path)
+            .expect("file path should be within repo")
+            .to_str()
+            .expect("valid path");
         let blame_result = repo.git_ai(&["blame", filename]);
 
         let lines = if let Ok(blame_output) = blame_result {
@@ -213,7 +217,11 @@ impl<'a> TestFile<'a> {
     }
 
     pub fn assert_blame_snapshot(&self) {
-        let filename = self.file_path.to_str().expect("valid path");
+        let filename = self.file_path
+            .strip_prefix(&self.repo.path)
+            .expect("file path should be within repo")
+            .to_str()
+            .expect("valid path");
         let blame_output = self
             .repo
             .git_ai(&["blame", filename])
@@ -226,8 +234,12 @@ impl<'a> TestFile<'a> {
     pub fn assert_lines_and_blame<T: Into<ExpectedLine>>(&mut self, lines: Vec<T>) {
         let expected_lines: Vec<ExpectedLine> = lines.into_iter().map(|l| l.into()).collect();
 
-        // Get blame output
-        let filename = self.file_path.to_str().expect("valid path");
+        // Get blame output - use relative path from repo root
+        let filename = self.file_path
+            .strip_prefix(&self.repo.path)
+            .expect("file path should be within repo")
+            .to_str()
+            .expect("valid path");
         let blame_output = self
             .repo
             .git_ai(&["blame", filename])
@@ -300,8 +312,12 @@ impl<'a> TestFile<'a> {
     pub fn assert_committed_lines<T: Into<ExpectedLine>>(&mut self, lines: Vec<T>) {
         let expected_lines: Vec<ExpectedLine> = lines.into_iter().map(|l| l.into()).collect();
 
-        // Get blame output
-        let filename = self.file_path.to_str().expect("valid path");
+        // Get blame output - use relative path from repo root
+        let filename = self.file_path
+            .strip_prefix(&self.repo.path)
+            .expect("file path should be within repo")
+            .to_str()
+            .expect("valid path");
         let blame_output = self
             .repo
             .git_ai(&["blame", filename])
@@ -430,8 +446,12 @@ impl<'a> TestFile<'a> {
 
     /// Assert that the file at the given path matches the expected contents and authorship
     pub fn assert_blame_contents_expected(&self) {
-        // Get blame output
-        let filename = self.file_path.to_str().expect("valid path");
+        // Get blame output - use relative path from repo root
+        let filename = self.file_path
+            .strip_prefix(&self.repo.path)
+            .expect("file path should be within repo")
+            .to_str()
+            .expect("valid path");
         let blame_output = self
             .repo
             .git_ai(&["blame", filename])

tests/repos/test_repo.rs

@@ -14,7 +14,7 @@ use super::test_file::TestFile;
 
 #[derive(Clone, Debug)]
 pub struct TestRepo {
-    path: PathBuf,
+    pub path: PathBuf,
 }
 
 impl TestRepo {
@@ -216,7 +216,7 @@ impl TestRepo {
     }
 
     pub fn stage_all_and_commit(&self, message: &str) -> Result<NewCommit, String> {
-        self.git(&["add", "-A"]).expect("add --all should succeed");
+        println!("stage_all_and_commit: {:?}", self.git(&["add", "-A"]).expect("add --all should succeed"));
         self.commit(message)
     }