diff --git a/functions/verfahrensgebiet_alkis_komplett.py b/functions/verfahrensgebiet_alkis_komplett.py index 880dbfb..15fb092 100644 --- a/functions/verfahrensgebiet_alkis_komplett.py +++ b/functions/verfahrensgebiet_alkis_komplett.py @@ -318,12 +318,8 @@ def verfahrensgebiet_alkis_komplett( # statt direkt abzubrechen. landkreis_kennzahl = ask_landkreis_auswahl(tab_widget, LANDKREISE) if not landkreis_kennzahl: - QMessageBox.critical( - tab_widget, - "Fehler", - "Weder eine Verfahrensnummer noch ein Landkreis ist in den " - "Projekteigenschaften eingetragen!", - ) + # Nutzer hat den Landkreis-Dialog abgebrochen – stiller Abbruch, + # keine Fehlermeldung (bewusster Abbruch, kein Fehlerzustand). return None, None, None, LoadStatus.NONE landkreis_modus = not verfahrensnummer diff --git a/tests/test_verfahrensgebiet_alkis_komplett.py b/tests/test_verfahrensgebiet_alkis_komplett.py index c6b11ce..aea3f94 100644 --- a/tests/test_verfahrensgebiet_alkis_komplett.py +++ b/tests/test_verfahrensgebiet_alkis_komplett.py @@ -681,18 +681,19 @@ class TestVerfahrensgebietAlkisKomplett(unittest.TestCase): @patch(_PATCH_MSGBOX) @patch(_PATCH_ASK_LANDKREIS, return_value=None) @patch(_PATCH_GET_VAR, return_value="") - def test_kein_vkz_kein_landkreis_zeigt_fehler( + def test_kein_vkz_kein_landkreis_dialog_abgebrochen( self, mock_get_var, mock_ask_landkreis, mock_msgbox, mock_prog ): """Weder Verfahrensnummer noch Landkreis gesetzt, Nutzer bricht den - Landkreis-Auswahldialog ab → Fehlermeldung, kein Layer.""" + Landkreis-Auswahldialog ab → stiller Abbruch, kein Layer, KEINE + Fehlermeldung (bewusster Abbruch, kein Fehlerzustand).""" enriched, flst, bror, status = verfahrensgebiet_alkis_komplett(None) self.assertIsNone(enriched) self.assertIsNone(flst) self.assertIsNone(bror) self.assertEqual(status, LoadStatus.NONE) - mock_msgbox.critical.assert_called_once() + mock_msgbox.critical.assert_not_called() @patch(_PATCH_PROGRESS) @patch(_PATCH_PROJECT)