test(exporter): increase test coverage

Adds tests for error handling and edge cases in the exporter module,
increasing test coverage to 100%.

Author: bosd

tests/test_exporter.py

@@ -2,7 +2,14 @@
 
 from unittest.mock import MagicMock, patch
 
-from odoo_data_flow.exporter import run_export, run_export_for_migration
+import pytest
+
+from odoo_data_flow.exporter import (
+    _show_error_panel,
+    run_export,
+    run_export_for_migration,
+    run_export_from_file,
+)
 
 
 @patch(
@@ -139,14 +146,78 @@ def test_run_export_for_migration_bad_domain(
     assert mock_export_data.call_args.args[2] == []
 
 
+@patch("odoo_data_flow.exporter.Console")
+def test_show_error_panel(mock_console: MagicMock) -> None:
+    """Tests the `_show_error_panel` function."""
+    _show_error_panel("Test Title", "Test Message")
+    mock_console.assert_called_once_with(stderr=True, style="bold red")
+    mock_console.return_value.print.assert_called_once()
+
+
+@patch(
+    "odoo_data_flow.exporter.export_threaded.export_data_to_file",
+    return_value=(False, "Error"),
+)
+@patch("odoo_data_flow.exporter._show_error_panel")
+def test_run_export_fail(
+    mock_show_error_panel: MagicMock, mock_export_data: MagicMock
+) -> None:
+    """Tests that `run_export` calls `_show_error_panel` when the export fails."""
+    run_export(
+        config="dummy.conf",
+        filename="dummy.csv",
+        model="dummy.model",
+        fields="id",
+    )
+    mock_show_error_panel.assert_called_once_with("Export Aborted", "Error")
+
+
 @patch("odoo_data_flow.exporter.export_threaded.export_data_for_migration")
-def test_run_export_for_migration_no_data(mock_export_data: MagicMock) -> None:
-    """Tests that `run_export_for_migration`.
+def test_run_export_for_migration_bad_context(mock_export_data: MagicMock) -> None:
+    """Tests `run_export_for_migration` with a bad context string."""
+    mock_export_data.return_value = ([], [])
+    run_export_for_migration(
+        config="dummy.conf",
+        model="res.partner",
+        fields=["id"],
+        context="bad-context",
+    )
+    # Assert that the context passed is an empty dict
+    assert mock_export_data.call_args.kwargs["context"] == {}
 
-    These handles the case where no data is returned.
-    """
-    mock_export_data.return_value = (["id"], None)
-    _header, data = run_export_for_migration(
-        config="dummy.conf", model="res.partner", fields=["id"]
+
+def test_run_export_from_file_not_implemented() -> None:
+    """Tests that `run_export_from_file` raises NotImplementedError."""
+    with pytest.raises(NotImplementedError):
+        run_export_from_file(
+            config="dummy.conf",
+            filename="dummy.csv",
+        )
+
+
+@patch("odoo_data_flow.exporter._show_error_panel")
+def test_run_export_domain_not_list(mock_show_error_panel: MagicMock) -> None:
+    """Tests that `run_export` logs an error for a domain that is not a list."""
+    run_export(
+        config="dummy.conf",
+        filename="dummy.csv",
+        model="dummy.model",
+        fields="id",
+        domain="{'is_company': True}",  # Not a list
     )
-    assert data is None
+    mock_show_error_panel.assert_called_once()
+    assert "Domain must be a list" in mock_show_error_panel.call_args[0][1]
+
+
+@patch("odoo_data_flow.exporter._show_error_panel")
+def test_run_export_context_not_dict(mock_show_error_panel: MagicMock) -> None:
+    """Tests that `run_export` logs an error for a context that is not a dict."""
+    run_export(
+        config="dummy.conf",
+        filename="dummy.csv",
+        model="dummy.model",
+        fields="id",
+        context="['lang', 'fr_FR']",  # Not a dict
+    )
+    mock_show_error_panel.assert_called_once()
+    assert "Context must be a dictionary" in mock_show_error_panel.call_args[0][1]