Kein Fehlerdialog mehr bei Abbruch des Landkreis-Auswahldialogs
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 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 5
parent
5816b2af22
commit
7b48d29d56
@@ -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
|
||||
|
||||
@@ -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)
|
||||
|
||||
Reference in New Issue
Block a user