Restliche "ungeklärt"-Funktionen in sn_basis abschließend bewertet (#70) #71

Merged
Daniel merged 0 commits from feature/70-sn-basis-ungeklaerte-tests into unstable 2026-09-21 06:57:02 +00:00
Member

Was

Die 9 laut Prüfstand „ungeklärten" sn_basis-Funktionen (Dateipruefer, Datenschreiber, Linkpruefer,
Pruefmanager) sind einzeln entschieden: 4 mit Mock-Modus-Test, 5 mit Docstring-Vermerk. Issue: #70

Wie geprüft

  • python scripts/run_tests.py sn_basis --baseline: GRÜN 328/328 bestanden, unverändert gegenüber Baseline
  • QGIS-Test: nicht nötig, reine Mock-Modus-Logik ohne Qt-Widgets (SQLite-Datei-I/O und Verzweigungslogik)

Änderungen an Verträgen

keine

Übergabenotiz

Aufgabe: #70 – restliche „ungeklärt"-Funktionen in sn_basis abschließend bewerten
Stand: fertig
Geändert: tests/test_dateipruefer.py, tests/test_datenschreiber.py, tests/test_pruefmanager.py
(neue Tests); modules/linkpruefer.py, modules/Pruefmanager.py (Docstring-Vermerke); changelog.txt
Nachweis: run_tests.py sn_basis --baseline: GRÜN 328/328; QGIS-Test nicht nötig
Entscheidungen/Annahmen: Pruefmanager._ask_use_or_replace_pufferlayer bewusst nicht getestet, da
toter Code (self.pruefmanager existiert nie) – Fix gehört zu Bug #60, nicht zu diesem Issue.
Offen / nächste Schritte: keine für #70; #60 (toter Code) bleibt offen.
Wissensbasis: docs/wissen/fallstricke.md um sqlite3/Windows-TemporaryDirectory-Fallstrick ergänzt
(im AG_QGIS-Repo, Commit 049a5ff)
Kontext für die nächste Sitzung: modules/Pruefmanager.py (_ask_use_or_replace_pufferlayer, Bug #60)

Checkliste

  • Nur Änderungen zum Issue
  • Keine direkten Qt/QGIS-Importe, keine Debug-print
  • changelog.txt ergänzt
  • Wissensbasis nachgezogen
## Was Die 9 laut Prüfstand „ungeklärten" sn_basis-Funktionen (Dateipruefer, Datenschreiber, Linkpruefer, Pruefmanager) sind einzeln entschieden: 4 mit Mock-Modus-Test, 5 mit Docstring-Vermerk. Issue: #70 ## Wie geprüft - `python scripts/run_tests.py sn_basis --baseline`: GRÜN 328/328 bestanden, unverändert gegenüber Baseline - QGIS-Test: nicht nötig, reine Mock-Modus-Logik ohne Qt-Widgets (SQLite-Datei-I/O und Verzweigungslogik) ## Änderungen an Verträgen keine ## Übergabenotiz **Aufgabe:** #70 – restliche „ungeklärt"-Funktionen in sn_basis abschließend bewerten **Stand:** fertig **Geändert:** `tests/test_dateipruefer.py`, `tests/test_datenschreiber.py`, `tests/test_pruefmanager.py` (neue Tests); `modules/linkpruefer.py`, `modules/Pruefmanager.py` (Docstring-Vermerke); `changelog.txt` **Nachweis:** `run_tests.py sn_basis --baseline`: GRÜN 328/328; QGIS-Test nicht nötig **Entscheidungen/Annahmen:** `Pruefmanager._ask_use_or_replace_pufferlayer` bewusst nicht getestet, da toter Code (`self.pruefmanager` existiert nie) – Fix gehört zu Bug #60, nicht zu diesem Issue. **Offen / nächste Schritte:** keine für #70; #60 (toter Code) bleibt offen. **Wissensbasis:** `docs/wissen/fallstricke.md` um sqlite3/Windows-TemporaryDirectory-Fallstrick ergänzt (im AG_QGIS-Repo, Commit 049a5ff) **Kontext für die nächste Sitzung:** `modules/Pruefmanager.py` (`_ask_use_or_replace_pufferlayer`, Bug #60) ## Checkliste - [x] Nur Änderungen zum Issue - [x] Keine direkten Qt/QGIS-Importe, keine Debug-`print` - [x] `changelog.txt` ergänzt - [x] Wissensbasis nachgezogen
Daniel added 1 commit 2026-09-21 06:56:51 +00:00
Tests für Dateipruefer._ist_leer (None/Whitespace), Dateipruefer._handle_leere_eingabe
(Priorität bei mehreren Flags), Datenschreiber._store_style_in_gpkg (nachgezogen aus #63,
siehe #67) und Pruefmanager._handle_verfahrensgebiet_layer_existiert (entfernen/ersetzen)
ergänzt. Linkpruefer._ist_url/_pruefe_url/_pruefe_dateipfad und
Pruefmanager._handle_datei_existiert sind bereits über pruefe()/verarbeite() vollständig
abgedeckt und bekommen den Vermerk "von Test abgesehen". Pruefmanager._ask_use_or_replace_pufferlayer
ist toter Code (Bug #60) und bleibt bis zur Behebung unmarkiert getestet, nur vermerkt.

sn_basis steht damit laut Prüfstand bei 0 ungeklärten Funktionen.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Daniel merged commit 0a876c097d into unstable 2026-09-21 06:57:02 +00:00
Daniel deleted branch feature/70-sn-basis-ungeklaerte-tests 2026-09-21 06:57:02 +00:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: AG_QGIS/Plugin_SN_Basis#71