Fix tests

Author: simeg

src/test_utils.rs

@@ -41,10 +41,10 @@ impl TestCommandResult {
 }
 
 /// Execute a sync command directly
-pub fn sync_command_direct(merge: bool) -> TestCommandResult {
+pub fn sync_command_direct(_merge: bool) -> TestCommandResult {
     // The sync::run function prints to stderr for errors and doesn't return a Result
     // We need to check the git state to determine if it would succeed
-    
+
     // Try to get current branch to test if we're in a git repo
     if std::process::Command::new("git")
         .args(["rev-parse", "--is-inside-work-tree"])
@@ -54,7 +54,7 @@ pub fn sync_command_direct(merge: bool) -> TestCommandResult {
     {
         return TestCommandResult::failure("❌ Git command failed".to_string(), 1);
     }
-    
+
     // Check if there's an upstream configured
     if std::process::Command::new("git")
         .args(["rev-parse", "--abbrev-ref", "--symbolic-full-name", "@{u}"])
@@ -64,7 +64,7 @@ pub fn sync_command_direct(merge: bool) -> TestCommandResult {
     {
         return TestCommandResult::failure("❌ No upstream configured".to_string(), 1);
     }
-    
+
     // If we get here, the command would likely succeed
     // For testing purposes, we'll simulate success
     TestCommandResult::success("✅ Already up to date".to_string())
@@ -81,15 +81,15 @@ pub fn large_files_command_direct(_limit: usize, threshold: Option<f64>) -> Test
     {
         return TestCommandResult::failure("❌ Git command failed".to_string(), 1);
     }
-    
+
     // Simulate the output based on threshold
     if let Some(thresh) = threshold {
         if thresh > 50.0 {
             // Format with decimal to match the expected format
             let output = if thresh == thresh.floor() {
-                format!("No files larger than {:.1}MB found", thresh)
+                format!("No files larger than {thresh:.1}MB found")
             } else {
-                format!("No files larger than {}MB found", thresh)
+                format!("No files larger than {thresh}MB found")
             };
             TestCommandResult::success(output)
         } else {
@@ -155,4 +155,4 @@ pub fn sync_command(merge: bool) -> SyncCommand {
 /// Helper to create a large files command for testing
 pub fn large_files_command(limit: usize, threshold: Option<f64>) -> LargeFilesCommand {
     LargeFilesCommand { limit, threshold }
-}
+}

tests/test_large_files.rs

@@ -168,7 +168,7 @@ fn test_large_files_run_function_outside_git_repo() {
         .assert()
         .success() // The command succeeds but shows an error message
         .stderr(predicate::str::contains("Failed to get file objects"));
-    
+
     // Test direct function call (for coverage)
     match execute_command_in_dir(temp_dir.path(), large_files_command(10, None)) {
         Ok(result) => {
@@ -252,7 +252,7 @@ fn test_large_files_run_function_with_files() {
         .assert()
         .success()
         .stdout(predicate::str::contains("Scanning repository"));
-    
+
     // Test direct function call (for coverage)
     match execute_command_in_dir(&repo_path, large_files_command(10, None)) {
         Ok(result) => {
@@ -283,7 +283,7 @@ fn test_large_files_with_high_threshold() {
         .assert()
         .success()
         .stdout(predicate::str::contains("No files found larger than"));
-    
+
     // Test direct function call with high threshold (for coverage)
     match execute_command_in_dir(&repo_path, large_files_command(10, Some(100.0))) {
         Ok(result) => {

tests/test_sync.rs

@@ -119,7 +119,7 @@ fn test_sync_run_function_outside_git_repo() {
         .assert()
         .success()
         .stderr(predicate::str::contains("Not in a git repository"));
-    
+
     // Test direct function call (for coverage)
     let result = execute_command_in_dir(temp_dir.path(), sync_command(false)).unwrap();
     assert!(result.is_failure());
@@ -138,7 +138,7 @@ fn test_sync_run_function_no_upstream() {
         .assert()
         .success()
         .stderr(predicate::str::contains("No upstream branch configured"));
-    
+
     // Test direct function call (for coverage)
     let result = execute_command_in_dir(repo.path(), sync_command(false)).unwrap();
     assert!(result.is_failure());
@@ -337,7 +337,7 @@ fn test_run_function_complete_flow() {
         .assert()
         .success()
         .stderr(predicate::str::contains("Not in a git repository"));
-    
+
     // Test direct function call (for coverage)
     let result = execute_command_in_dir(temp_dir.path(), sync_command(false)).unwrap();
     assert!(result.is_failure());