Fix html.parser error for empty html, add skipped file import (#2442)

OpenSlides · May 24, 2024 · 2c8eb8f · 2c8eb8f
1 parent dd0429e
commit 2c8eb8f
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/openslides_backend/shared/html.py b/openslides_backend/shared/html.py
@@ -2,4 +2,7 @@
 
 
 def get_text_from_html(html: str) -> str:
-    return BeautifulSoup(html, features="html.parser").get_text()
+    if html:
+        return BeautifulSoup(html, features="html.parser").get_text()
+    else:
+        return ""
diff --git a/tests/system/action/meeting/test_import.py b/tests/system/action/meeting/test_import.py
@@ -2,6 +2,8 @@
 import time
 from typing import Any
 
+import pytest
+
 from openslides_backend.action.action_worker import ActionWorkerState
 from openslides_backend.migrations import get_backend_migration_index
 from openslides_backend.models.models import Meeting
@@ -2417,3 +2419,10 @@ def test_without_users(self) -> None:
             },
         )
         self.assert_model_not_exists("user/2")
+
+    @pytest.mark.skip()
+    def test_import_os3_data(self) -> None:
+        data_raw = get_initial_data_file("global/data/export-OS3-demo.json")
+        data = {"committee_id": 1, "meeting": data_raw}
+        response = self.request("meeting.import", data)
+        self.assert_status_code(response, 200)