From 868ba81f9c79dbd6cbe56d86c5e9fce0a812c848 Mon Sep 17 00:00:00 2001 From: AIOSAI Date: Fri, 15 May 2026 14:32:25 -0700 Subject: [PATCH] fix(drone): update commit test mock for lint gate (6 subprocess calls) --- src/aipass/drone/tests/test_git_access.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/aipass/drone/tests/test_git_access.py b/src/aipass/drone/tests/test_git_access.py index 62741777..cf2ec407 100644 --- a/src/aipass/drone/tests/test_git_access.py +++ b/src/aipass/drone/tests/test_git_access.py @@ -343,8 +343,9 @@ class TestCommitChanges: assert "nothing to commit" in result["stderr"].lower() def test_commit_all_stages_first(self, repo_dir: Path) -> None: - mock_ruff_check = MagicMock(returncode=0, stdout="", stderr="") + mock_ruff_fix = MagicMock(returncode=0, stdout="", stderr="") mock_ruff_format = MagicMock(returncode=0, stdout="", stderr="") + mock_ruff_gate = MagicMock(returncode=0, stdout="", stderr="") mock_add = MagicMock(returncode=0, stderr="") mock_diff = MagicMock(returncode=1, stdout="", stderr="") mock_commit = MagicMock(returncode=0, stdout="[main def456] all commit", stderr="") @@ -355,7 +356,7 @@ class TestCommitChanges: patch("shutil.which", return_value="/usr/bin/ruff"), patch( "aipass.drone.apps.handlers.git.commit_handler.subprocess.run", - side_effect=[mock_ruff_check, mock_ruff_format, mock_add, mock_diff, mock_commit], + side_effect=[mock_ruff_fix, mock_ruff_format, mock_ruff_gate, mock_add, mock_diff, mock_commit], ), ): result = commit_changes("all commit", branch_dir=branch_dir, all_files=True)