From d991cb67219e0728679bf5d00773f12619c97279 Mon Sep 17 00:00:00 2001 From: Andrew Grigorev Date: Sat, 12 Apr 2025 19:54:56 +0300 Subject: [PATCH] test: cover user commit with no committer attribution Co-authored-by: aider (vertex_ai/gemini-2.5-pro-exp-03-25) --- tests/basic/test_repo.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/tests/basic/test_repo.py b/tests/basic/test_repo.py index 88c4b4edf..9633ec208 100644 --- a/tests/basic/test_repo.py +++ b/tests/basic/test_repo.py @@ -223,6 +223,15 @@ class TestRepo(unittest.TestCase): original_author_name = os.environ.get("GIT_AUTHOR_NAME") self.assertIsNone(original_author_name) + # Test user commit with explicit no-committer attribution + git_repo_user_no_committer = GitRepo(io, None, None, attribute_committer=False) + fname.write_text("user no committer content") + commit_result = git_repo_user_no_committer.commit(fnames=[str(fname)], aider_edits=False) + self.assertIsNotNone(commit_result) + commit = raw_repo.head.commit + self.assertEqual(commit.author.name, "Test User") # Author never modified for user commits + self.assertEqual(commit.committer.name, "Test User") # Explicit False prevents modification + def test_commit_with_co_authored_by(self): # Cleanup of the git temp dir explodes on windows if platform.system() == "Windows":