actually FIX the version command test for real this time

Author: CodyKoInABox

tests/cli/commands/test_version.py

@@ -30,10 +30,13 @@ def test_version_command_success(mock_file_open, mock_exists, mock_get_translati
     mock_get_translation.return_value = DUMMY_MESSAGES
     mock_exists.return_value = True
 
-    # Setup mock file content with proper line structure
-    setup_content = 'version="1.2.3",\nname="spicecode"'
-    mock_file_open.return_value.read.return_value = setup_content
-    mock_file_open.return_value.__iter__.return_value = iter(setup_content.splitlines())
+    # Setup mock file content - each line should end with \n for proper line iteration
+    file_lines = [
+        'name="spicecode",\n',
+        'version="1.2.3",\n',
+        'author="test"\n'
+    ]
+    mock_file_open.return_value.__iter__.return_value = iter(file_lines)
 
     version_command(LANG_FILE="dummy_lang.txt", CURRENT_DIR=TEST_CURRENT_DIR)
 
@@ -51,10 +54,13 @@ def test_version_command_version_not_in_setup(mock_file_open, mock_exists, mock_
     mock_get_translation.return_value = DUMMY_MESSAGES
     mock_exists.return_value = True
 
-    # Setup mock file content without version
-    setup_content = 'name="spicecode"\nauthor="test"'
-    mock_file_open.return_value.read.return_value = setup_content
-    mock_file_open.return_value.__iter__.return_value = iter(setup_content.splitlines())
+    # Setup mock file content without version line
+    file_lines = [
+        'name="spicecode",\n',
+        'author="test",\n',
+        'description="A CLI tool"\n'
+    ]
+    mock_file_open.return_value.__iter__.return_value = iter(file_lines)
 
     version_command(LANG_FILE="dummy_lang.txt", CURRENT_DIR=TEST_CURRENT_DIR)