test: Add logout error test for mark_emails coverage

Adds test_mark_emails_logout_error to cover lines 1377-1378 (the logout
exception handler in mark_emails function).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: Jack Koch

tests/test_email_client.py

@@ -477,3 +477,25 @@ async def test_mark_emails_custom_mailbox(self, email_client):
 
             # Verify custom mailbox was selected (quoted)
             mock_imap.select.assert_called_once_with('"[Gmail]/All Mail"')
+
+    @pytest.mark.asyncio
+    async def test_mark_emails_logout_error(self, email_client):
+        """Test mark_emails handles logout errors gracefully (covers logout exception handler)."""
+        mock_imap = AsyncMock()
+        mock_imap._client_task = asyncio.Future()
+        mock_imap._client_task.set_result(None)
+        mock_imap.wait_hello_from_server = AsyncMock()
+        mock_imap.login = AsyncMock()
+        mock_imap.select = AsyncMock()
+        mock_imap.uid = AsyncMock(return_value=(None, None))
+        mock_imap.logout = AsyncMock(side_effect=OSError("Connection closed"))
+
+        with patch.object(email_client, "imap_class", return_value=mock_imap):
+            # Should complete successfully despite logout error
+            marked_ids, failed_ids = await email_client.mark_emails(
+                email_ids=["123"],
+                mark_as="read",
+                mailbox="INBOX",
+            )
+            assert marked_ids == ["123"]
+            assert failed_ids == []