From 7b48d29d56d7fc0bc01ef844037f7e26c27fdbef Mon Sep 17 00:00:00 2001 From: Michael Otto Date: Fri, 28 Aug 2026 14:44:25 +0200 Subject: [PATCH] Kein Fehlerdialog mehr bei Abbruch des Landkreis-Auswahldialogs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Abbruch des Dialogs ist eine bewusste Nutzerentscheidung, kein Fehlerzustand – verfahrensgebiet_alkis_komplett() bricht jetzt still ab statt eine QMessageBox.critical zu zeigen. Test entsprechend angepasst/umbenannt. Co-Authored-By: Claude Sonnet 5 --- functions/verfahrensgebiet_alkis_komplett.py | 8 ++------ tests/test_verfahrensgebiet_alkis_komplett.py | 7 ++++--- 2 files changed, 6 insertions(+), 9 deletions(-) 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)