From e6623ae0a8d9f17bdbfaaffe615ec36c9baa2503 Mon Sep 17 00:00:00 2001 From: "Paul Gauthier (aider)" Date: Sat, 8 Mar 2025 13:45:44 -0800 Subject: [PATCH] test: mock URL launches in deprecated args tests --- tests/basic/test_deprecated.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/basic/test_deprecated.py b/tests/basic/test_deprecated.py index 6da5ff121..b1ba3e22d 100644 --- a/tests/basic/test_deprecated.py +++ b/tests/basic/test_deprecated.py @@ -21,7 +21,10 @@ class TestDeprecated(TestCase): os.environ.update(self.original_env) @patch("aider.io.InputOutput.tool_warning") - def test_deprecated_args_show_warnings(self, mock_tool_warning): + @patch("aider.io.InputOutput.offer_url") + def test_deprecated_args_show_warnings(self, mock_offer_url, mock_tool_warning): + # Prevent URL launches during tests + mock_offer_url.return_value = False # Test all deprecated flags to ensure they show warnings deprecated_flags = [ "--opus", @@ -65,7 +68,10 @@ class TestDeprecated(TestCase): self.assertIn("use --model", warning_msg.lower()) @patch("aider.io.InputOutput.tool_warning") - def test_model_alias_in_warning(self, mock_tool_warning): + @patch("aider.io.InputOutput.offer_url") + def test_model_alias_in_warning(self, mock_offer_url, mock_tool_warning): + # Prevent URL launches during tests + mock_offer_url.return_value = False # Test that the warning uses the model alias if available with patch("aider.models.MODEL_ALIASES", {"gpt4": "gpt-4-0613"}): with patch("aider.models.Model"):