Remove duplicated tests

Author: simeg

tests/test_bisect.rs

@@ -599,21 +599,6 @@ fn test_bisect_start_relative_commits() {
         .success();
 }
 
-#[test]
-fn test_bisect_command_traits() {
-    use git_x::commands::commit::{BisectAction, BisectCommand};
-    use git_x::core::traits::Command;
-
-    let cmd = BisectCommand::new(BisectAction::Status);
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "bisect");
-    assert_eq!(
-        cmd.description(),
-        "Simplified Git bisect workflow for finding bugs"
-    );
-}
-
 #[test]
 fn test_bisect_command_direct() {
     use git_x::commands::commit::{BisectAction, BisectCommand};

tests/test_clean_branches.rs

@@ -70,58 +70,6 @@ fn test_clean_branches_run_function_actual_delete() {
     let _ = std::env::set_current_dir(&original_dir);
 }
 
-#[test]
-fn test_clean_branches_run_function_with_branches_to_delete() {
-    let repo = repo_with_merged_branch("test-branch", "master");
-    let original_dir = std::env::current_dir().unwrap();
-
-    // Switch back to master to ensure the test branch is merged
-    repo.checkout_branch("master");
-
-    // Change to repo directory and run the function directly
-    std::env::set_current_dir(repo.path()).unwrap();
-
-    let cmd = CleanBranchesCommand::new(true);
-    let result = cmd.execute();
-    assert!(result.is_ok());
-
-    // Restore original directory
-    let _ = std::env::set_current_dir(&original_dir);
-}
-
-#[test]
-fn test_clean_branches_run_function_non_dry_run_with_branches() {
-    if !should_run_destructive_tests() {
-        return;
-    }
-
-    let repo = repo_with_merged_branch("test-non-dry", "master");
-    let original_dir = std::env::current_dir().unwrap();
-
-    // Switch back to master to ensure the test branch is merged
-    repo.checkout_branch("master");
-
-    // Change to repo directory and run the function directly
-    std::env::set_current_dir(repo.path()).unwrap();
-
-    // Set non-interactive mode for this test
-    unsafe {
-        std::env::set_var("GIT_X_NON_INTERACTIVE", "1");
-    }
-
-    let cmd = CleanBranchesCommand::new(false);
-    let result = cmd.execute();
-    assert!(result.is_ok());
-
-    // Clean up environment variable
-    unsafe {
-        std::env::remove_var("GIT_X_NON_INTERACTIVE");
-    }
-
-    // Restore original directory
-    let _ = std::env::set_current_dir(&original_dir);
-}
-
 #[test]
 fn test_clean_branches_run_function_no_branches() {
     let repo = common::basic_repo();
@@ -169,12 +117,3 @@ fn test_clean_branches_actually_deletes_branch() {
     let stdout_after = String::from_utf8_lossy(&output_after.stdout);
     assert!(!stdout_after.contains("feature/cleanup"));
 }
-
-#[test]
-fn test_clean_branches_command_traits() {
-    let cmd = CleanBranchesCommand::new(true);
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "clean-branches");
-    assert_eq!(cmd.description(), "Delete merged branches");
-}

tests/test_color_graph.rs

@@ -37,12 +37,3 @@ fn test_color_graph_run_function_in_non_git_directory() {
     // Restore original directory
     let _ = std::env::set_current_dir(&original_dir);
 }
-
-#[test]
-fn test_color_graph_command_traits() {
-    let cmd = ColorGraphCommand::new();
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "color-graph");
-    assert_eq!(cmd.description(), "Show a colored commit graph");
-}

tests/test_contributors.rs

@@ -412,15 +412,3 @@ fn test_contributors_command_direct() {
     // Restore original directory
     let _ = std::env::set_current_dir(&original_dir);
 }
-
-#[test]
-fn test_contributors_command_traits() {
-    let cmd = ContributorsCommand::new(None);
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "contributors");
-    assert_eq!(
-        cmd.description(),
-        "Show repository contributors and their commit statistics"
-    );
-}

tests/test_fixup.rs

@@ -18,21 +18,6 @@ fn test_fixup_run_function_outside_git_repo() {
         .stderr(predicate::str::contains("Commit hash does not exist"));
 }
 
-#[test]
-fn test_fixup_command_traits() {
-    use git_x::commands::commit::FixupCommand;
-    use git_x::core::traits::Command;
-
-    let cmd = FixupCommand::new("abc123".to_string(), false);
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "fixup");
-    assert_eq!(
-        cmd.description(),
-        "Create fixup commits for easier interactive rebasing"
-    );
-}
-
 #[test]
 fn test_fixup_command_direct() {
     use git_x::commands::commit::FixupCommand;
@@ -128,14 +113,3 @@ fn test_fixup_command_help() {
             "Create fixup commits for easier interactive rebasing",
         ));
 }
-
-#[test]
-fn test_fixup_rebase_flag() {
-    let mut cmd = Command::cargo_bin("git-x").expect("Failed to find binary");
-    cmd.args(["fixup", "abc123", "--rebase", "--help"])
-        .assert()
-        .success()
-        .stdout(predicate::str::contains(
-            "Automatically rebase with autosquash after creating fixup",
-        ));
-}

tests/test_graph.rs

@@ -74,15 +74,3 @@ fn test_graph_run_function_in_non_git_directory() {
     // Restore original directory
     let _ = std::env::set_current_dir(&original_dir);
 }
-
-#[test]
-fn test_graph_command_traits() {
-    use git_x::commands::analysis::GraphCommand;
-    use git_x::core::traits::Command;
-
-    let cmd = GraphCommand::new();
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "graph");
-    assert_eq!(cmd.description(), "Show a simple commit graph");
-}

tests/test_health.rs

@@ -230,18 +230,6 @@ fn test_health_command_direct() {
     let _ = std::env::set_current_dir(&original_dir);
 }
 
-#[test]
-fn test_health_command_traits() {
-    let cmd = HealthCommand::new();
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "health");
-    assert_eq!(
-        cmd.description(),
-        "Check repository health and configuration"
-    );
-}
-
 #[test]
 fn test_health_command_in_non_git_directory() {
     let temp_dir = tempfile::tempdir().unwrap();

tests/test_info.rs

@@ -140,15 +140,6 @@ fn test_info_command_direct() {
     assert!(output.contains("Current branch:"));
 }
 
-#[test]
-fn test_info_command_traits() {
-    let cmd = InfoCommand::new();
-
-    // Test Command trait implementation
-    assert_eq!(cmd.name(), "info");
-    assert_eq!(cmd.description(), "Show repository information and status");
-}
-
 #[test]
 fn test_info_command_with_details() {
     let (repo, _remote) = repo_with_remote_ahead("main");