From 5b13105d58117ff1d73bdeea1c31199cd037de78 Mon Sep 17 00:00:00 2001 From: "Paul Gauthier (aider)" Date: Mon, 24 Feb 2025 10:20:46 -0800 Subject: [PATCH] fix: Update tests to match string return type of discover_editor() --- tests/basic/test_editor.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/basic/test_editor.py b/tests/basic/test_editor.py index 578ff927c..415798da6 100644 --- a/tests/basic/test_editor.py +++ b/tests/basic/test_editor.py @@ -34,17 +34,17 @@ def test_discover_editor_defaults(): # Test Windows default mock_system.return_value = "Windows" with patch.dict(os.environ, {}, clear=True): - assert discover_editor() == [DEFAULT_EDITOR_WINDOWS] + assert discover_editor() == DEFAULT_EDITOR_WINDOWS # Test macOS default mock_system.return_value = "Darwin" with patch.dict(os.environ, {}, clear=True): - assert discover_editor() == [DEFAULT_EDITOR_OS_X] + assert discover_editor() == DEFAULT_EDITOR_OS_X # Test Linux default mock_system.return_value = "Linux" with patch.dict(os.environ, {}, clear=True): - assert discover_editor() == [DEFAULT_EDITOR_NIX] + assert discover_editor() == DEFAULT_EDITOR_NIX def test_write_temp_file(): @@ -81,8 +81,8 @@ def test_print_status_message(capsys): def test_discover_editor_override(): # Test editor override - assert discover_editor("code") == ["code"] - assert discover_editor('vim -c "set noswapfile"') == ["vim", "-c", "set noswapfile"] + assert discover_editor("code") == "code" + assert discover_editor('vim -c "set noswapfile"') == 'vim -c "set noswapfile"' # Test invalid editor command with pytest.raises(RuntimeError):