From 3904743e854b2553e85831f07c2878ccff80a4fc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9sar=20P=C3=A9rez?= Date: Sat, 7 Jun 2025 21:13:16 +0200 Subject: [PATCH] Fixed comment 4 --- pr_agent/tools/pr_update_changelog.py | 1 + tests/unittest/test_pr_update_changelog.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/pr_agent/tools/pr_update_changelog.py b/pr_agent/tools/pr_update_changelog.py index 963cbe9e..e6c1788b 100644 --- a/pr_agent/tools/pr_update_changelog.py +++ b/pr_agent/tools/pr_update_changelog.py @@ -198,6 +198,7 @@ Example: get_logger().warning(f"Error getting changelog file: {e}") self.changelog_file_str = "" self.changelog_file = "" + return if not self.changelog_file_str: self.changelog_file_str = self._get_default_changelog() diff --git a/tests/unittest/test_pr_update_changelog.py b/tests/unittest/test_pr_update_changelog.py index ce17e654..e3443896 100644 --- a/tests/unittest/test_pr_update_changelog.py +++ b/tests/unittest/test_pr_update_changelog.py @@ -91,7 +91,7 @@ class TestPRUpdateChangelog: # Assert assert changelog_tool.changelog_file == "" - assert changelog_tool.changelog_file_str != "" # Should have default content + assert changelog_tool.changelog_file_str == "" # Exception should result in empty string, no default template def test_prepare_changelog_update_with_existing_content(self, changelog_tool): """Test preparing changelog update when existing content exists."""