From 8d6764138117226aa588e5bac405ef5f2c8610a9 Mon Sep 17 00:00:00 2001 From: Jon Michael Aanes Date: Tue, 15 Apr 2025 00:46:12 +0200 Subject: [PATCH] Ruff --- aider_gitea/__init__.py | 9 ++-- aider_gitea/gitea_client.py | 8 +++- test/test_solve_issue_in_repository.py | 64 +++++++++++++------------- 3 files changed, 46 insertions(+), 35 deletions(-) diff --git a/aider_gitea/__init__.py b/aider_gitea/__init__.py index 4799100..388e033 100644 --- a/aider_gitea/__init__.py +++ b/aider_gitea/__init__.py @@ -304,7 +304,7 @@ def solve_issue_in_repository( run_cmd(['bash', '-c', RUFF_FORMAT_AND_AUTO_FIX], tmpdirname, check=False) run_cmd(['git', 'add', '.'], tmpdirname) run_cmd(['git', 'commit', '-m', 'Initial ruff pass'], tmpdirname, check=False) - + # Save the commit hash after ruff but before aider result = subprocess.run( ['git', 'rev-parse', 'HEAD'], @@ -340,9 +340,12 @@ def solve_issue_in_repository( text=True, ) files_changed = result.stdout.strip() - + if not files_changed: - logger.info('Aider did not make any changes beyond the initial ruff pass for issue #%s', issue_number) + logger.info( + 'Aider did not make any changes beyond the initial ruff pass for issue #%s', + issue_number, + ) return False # Push changes diff --git a/aider_gitea/gitea_client.py b/aider_gitea/gitea_client.py index 76028f8..267c3c7 100644 --- a/aider_gitea/gitea_client.py +++ b/aider_gitea/gitea_client.py @@ -158,7 +158,13 @@ class GiteaClient: requests.HTTPError: If the API request fails. """ url = f'{self.gitea_url}/repos/{owner}/{repo}/pulls' - json_data = {'title': title, 'body': body, 'head': head, 'base': base, 'labels': labels} + json_data = { + 'title': title, + 'body': body, + 'head': head, + 'base': base, + 'labels': labels, + } response = self.session.post(url, json=json_data) response.raise_for_status() diff --git a/test/test_solve_issue_in_repository.py b/test/test_solve_issue_in_repository.py index 6d8bdf2..33c90c8 100644 --- a/test/test_solve_issue_in_repository.py +++ b/test/test_solve_issue_in_repository.py @@ -1,43 +1,43 @@ -import subprocess -import tempfile from pathlib import Path from unittest.mock import MagicMock, patch -import pytest - from aider_gitea import solve_issue_in_repository class TestSolveIssueInRepository: def setup_method(self): self.args = MagicMock() - self.args.gitea_url = "https://gitea.example.com" - self.args.owner = "test-owner" - self.args.repo = "test-repo" - self.args.base_branch = "main" - - self.gitea_client = MagicMock() - self.tmpdirname = Path("/tmp/test-repo") - self.branch_name = "issue-123-test-branch" - self.issue_title = "Test Issue" - self.issue_description = "This is a test issue" - self.issue_number = "123" + self.args.gitea_url = 'https://gitea.example.com' + self.args.owner = 'test-owner' + self.args.repo = 'test-repo' + self.args.base_branch = 'main' - @patch('aider_gitea.secrets.llm_api_key', return_value="fake-api-key") + self.gitea_client = MagicMock() + self.tmpdirname = Path('/tmp/test-repo') + self.branch_name = 'issue-123-test-branch' + self.issue_title = 'Test Issue' + self.issue_description = 'This is a test issue' + self.issue_number = '123' + + @patch('aider_gitea.secrets.llm_api_key', return_value='fake-api-key') @patch('aider_gitea.run_cmd') @patch('aider_gitea.push_changes') @patch('subprocess.run') - def test_solve_issue_with_aider_changes(self, mock_subprocess_run, mock_push_changes, mock_run_cmd, mock_llm_api_key): + def test_solve_issue_with_aider_changes( + self, mock_subprocess_run, mock_push_changes, mock_run_cmd, mock_llm_api_key, + ): # Setup mocks mock_run_cmd.return_value = True mock_push_changes.return_value = True - + # Mock subprocess.run to return different commit hashes and file changes mock_subprocess_run.side_effect = [ - MagicMock(stdout="abc123\n", returncode=0), # First git rev-parse - MagicMock(stdout="file1.py\nfile2.py\n", returncode=0), # git diff with changes + MagicMock(stdout='abc123\n', returncode=0), # First git rev-parse + MagicMock( + stdout='file1.py\nfile2.py\n', returncode=0, + ), # git diff with changes ] - + # Call the function result = solve_issue_in_repository( self.args, @@ -46,28 +46,30 @@ class TestSolveIssueInRepository: self.issue_title, self.issue_description, self.issue_number, - self.gitea_client + self.gitea_client, ) - + # Verify results assert result is True assert mock_run_cmd.call_count >= 8 # Verify all expected commands were run mock_push_changes.assert_called_once() - @patch('aider_gitea.secrets.llm_api_key', return_value="fake-api-key") + @patch('aider_gitea.secrets.llm_api_key', return_value='fake-api-key') @patch('aider_gitea.run_cmd') @patch('aider_gitea.push_changes') @patch('subprocess.run') - def test_solve_issue_without_aider_changes(self, mock_subprocess_run, mock_push_changes, mock_run_cmd, mock_llm_api_key): + def test_solve_issue_without_aider_changes( + self, mock_subprocess_run, mock_push_changes, mock_run_cmd, mock_llm_api_key, + ): # Setup mocks mock_run_cmd.return_value = True - + # Mock subprocess.run to return same commit hash and no file changes mock_subprocess_run.side_effect = [ - MagicMock(stdout="abc123\n", returncode=0), # First git rev-parse - MagicMock(stdout="", returncode=0), # git diff with no changes + MagicMock(stdout='abc123\n', returncode=0), # First git rev-parse + MagicMock(stdout='', returncode=0), # git diff with no changes ] - + # Call the function result = solve_issue_in_repository( self.args, @@ -76,9 +78,9 @@ class TestSolveIssueInRepository: self.issue_title, self.issue_description, self.issue_number, - self.gitea_client + self.gitea_client, ) - + # Verify results assert result is False assert mock_push_changes.call_count == 0 # push_changes should not be called