feat: add error handling for staging files

This commit introduces error handling in the GitOperations class to raise a ValueError when no files are provided to the stage_files method. Additionally, it adds corresponding tests to ensure that this behavior is correctly implemented and validated.

aicmt/git_operations.py

@@ -245,6 +245,9 @@ def stage_files(self, files: List[str]) -> None:
         Raises:
             git.GitCommandError: If there is an error staging the files
         """
+        if not files:
+            raise ValueError("No files to stage!")
+
         try:
             # Get current status of files
             status = self.repo.git.status("--porcelain").splitlines()

tests/test_cli.py

@@ -130,6 +130,11 @@ def test_run_no_approved_groups(mock_confirm, assistant):
             with patch.multiple(assistant.cli, display_commit_groups=lambda x: []):
                 assistant.run()
 
+def test_create_new_commits(assistant, capsys):
+    commit_groups = [{"files": [], "commit_message": "test commit", "description": "test description"}]
+    assistant._create_new_commits(commit_groups)
+    captured = capsys.readouterr()
+    assert "No files to stage!" in captured.out
 
 def test_commit_creation_failure(assistant, capsys):
     changes = [Change(file="test.py", status="modified", diff="test diff", insertions=1, deletions=0)]

tests/test_git_operations.py

@@ -90,6 +90,11 @@ def test_stage_files(temp_git_repo):
         assert "stage_test.txt" not in untracked
         # Directly check if the file name is in the index
         assert any("stage_test.txt" in str(entry) for entry in git_ops.repo.index.entries.keys())
+
+        with pytest.raises(ValueError) as excinfo:
+            git_ops.stage_files([])
+        assert str(excinfo.value) == "No files to stage!"
+
     finally:
         # Restore the original directory
         os.chdir(current_dir)