Function test_uncommitted_submodule_changes_are_committed

src/git.rs:2590–2751

()

    /// to do it automatically.
    #[tokio::test]
    async fn test_uncommitted_submodule_changes_are_committed() {
        let ctx = AppContext::builder().build();

        // Create RepoB (will become a submodule)
        let repo_b = TestGitRepository::new().unwrap();
        repo_b.init_with_main_branch().unwrap();

        // Create RepoA (superproject) with RepoB as submodule
        let repo_a = TestGitRepository::new().unwrap();
        repo_a.init_with_main_branch().unwrap();

        // Add RepoB as submodule
        repo_a.add_submodule(&repo_b, "RepoB").unwrap();
        repo_a.stage_all().unwrap();
        repo_a.commit("Add RepoB submodule").unwrap();

        // Capture source_commit before copying
        let source_commit = repo_a.get_current_commit().unwrap();

        // Step 1: Copy the repo (simulates `tsk shell` starting)
        let manager = RepoManager::new(&ctx);
        let task_id = "uncommittedsub";
        let branch_name = "tsk/test/uncommitted-submodule/uncommittedsub";
        let copied_path = manager
            .copy_repo(
                task_id,
                repo_a.path(),
                None,
                branch_name,
                CopyMode::WorkingTree,
            )
            .await
            .expect("Copy should succeed")
            .repo_path;

        // Step 2: Modify both READMEs WITHOUT committing (simulates user editing files)
        // This is the key difference - no manual commits in either repo
        std::fs::write(copied_path.join("README.md"), "# RepoA\nmodified").unwrap();
        std::fs::write(
            copied_path.join("RepoB/README.md"),
            "# RepoB\nmodified by agent",
        )
        .unwrap();

        // Verify submodule has uncommitted changes before commit_changes()
        let copied_submodule_path = copied_path.join("RepoB");
        let status_before = std::process::Command::new("git")
            .current_dir(&copied_submodule_path)
            .args(["status", "--porcelain"])
            .output()
            .expect("git status should work");
        let status_before_str = String::from_utf8_lossy(&status_before.stdout);
        assert!(
            !status_before_str.is_empty(),
            "Submodule should have uncommitted changes before commit_changes()"
        );

        // Configure git user for both superproject and submodule (needed for auto-commit)