feat: add command timeout handling, add ensure_git_author_identity

Author: krerkkiat

grading_lib/common.py

@@ -94,8 +94,10 @@ def run_executable(
         return CommandResult(True, " ".join(args), make_cmd_output.decode())
     except subprocess.CalledProcessError as e:
         return CommandResult(False, " ".join(args), e.output.decode())
-    except subprocess.TimeoutExpired as e:
-        return CommandResult(False, " ".join(args), e.output.decode())
+    except subprocess.TimeoutExpired:
+        return CommandResult(
+            False, " ".join(args), f"Command timed out after {timeout} seconds."
+        )
 
 
 def ensure_lf_line_ending(path: Path | str) -> CommandResult:

grading_lib/repository.py

@@ -5,13 +5,32 @@
 import uuid
 from pathlib import Path
 
-from git import Head, Repo
-from git.refs.tag import Tag
+from git import Head, Repo, Tag
 from git.repo.fun import is_git_dir
 
 from .common import BaseTestCase, CommandResult, run_executable
 
 
+def ensure_git_author_identity(
+    name="ou-cs3560-grading-script", email="[email protected]"
+):
+    """
+    Set user.name and user.email is not set unless they are already set.
+    """
+    existing_user_name = subprocess.run(
+        "git config --get user.name", shell=True, capture_output=True
+    )
+    existing_user_email = subprocess.run(
+        "git config --get user.email", shell=True, capture_output=True
+    )
+
+    if existing_user_name is not None or len(existing_user_name.strip()) == 0:
+        subprocess.run(["git", "config", "--global", "user.name", name])
+
+    if existing_user_email is not None or len(existing_user_email.strip()) == 0:
+        subprocess.run(["git", "config", "--global", "user.email", email])
+
+
 class Repository:
     """
     A wrapper over git.Repo with our own utilities.