fix: incorrect use of config_writer

Author: krerkkiat

grading_lib/repository.py

@@ -53,10 +53,6 @@ def __init__(self, path: str | Path, *args, **kwargs):
                 )
 
             self.repo = Repo(temp_dir_path / "repo", *args, **kwargs)
-            # Some git commands when run on GH Actions need user indentity.
-            with self.repo.config_writer(config_level="repository") as conf_writer:
-                conf_writer.set_value("user.name", "ou-cs3560-grading-script")
-                conf_writer.set_value("user.email", "[email protected]")
         else:
             if is_git_dir(path):
                 self.repo = Repo(path, *args, **kwargs)
@@ -68,6 +64,11 @@ def __init__(self, path: str | Path, *args, **kwargs):
                 "A repository must have a working tree directory (Repo.working_tree_dir must not be None)."
             )
 
+        # Some git commands when run on GitHub's Actions need a user's indentity.
+        with self.repo.config_writer(config_level="repository") as conf_writer:
+            conf_writer.set_value("user", "name", "ou-cs3560-grading-script")
+            conf_writer.set_value("user", "email", "[email protected]")
+
         self.working_tree_dir = Path(self.repo.working_tree_dir)
 
     def __enter__(self):

tests/test_repository.py

@@ -17,7 +17,7 @@ def an_empty_folder():
     temp_dir.cleanup()
 
 
-def test_Repository_init(an_empty_folder):
+def test_Repository_init_with_an_empty_folder(an_empty_folder):
     repo = Repository(an_empty_folder.name)
     assert isinstance(repo.working_tree_dir, Path)