Fix Qt and Windows pytest symlink teardown crashes; add right-click test - #12
Conversation
…-click test - Guard against Windows pytest-current symlink PermissionError during sessionfinish cleanup - Cleanly teardown top-level Qt widgets in tests/conftest.py to avoid dangling QObjects at exit - Explicitly close and schedule deleteLater() in test_checksum_dialog_close_cancels_worker - Add regression test for right-click on unselected row selecting the row before context menu actions - Fixes T-20260928-295547351
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing. |
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: CI grün (13/13 Checks), keine Blocker.
Umfang: nur Tests/Teardown (tests/conftest.py, zwei Testdateien). Keine Credentials, keine Nutzerpfade.
Hinweise:
conftest.pyersetzt die privaten pytest-Funktionen_pytest.pathlib.cleanup_dead_symlinksundcleanup_numbered_dirper Monkeypatch. Ändert pytest die Signatur, schlägt das erst beim Teardown alsTypeErrorfehl (gefangen werden nurOSErrorbeim Aufruf undImportError/AttributeErrorbeim Import). Mitpytest>=9.1.1aktuell grün; ein Kommentar mit dem getesteten pytest-Bereich wäre sinnvoll.- Merge-Konflikt-Risiko:
tests/conftest.pywird auch von #14, #15 und #17 neu angelegt (jeweils mit anderem Inhalt, hier 80 Zeilen, dort 25 bis 29). Nach dem ersten Merge brauchen die übrigen ein Rebase. Empfehlung: diese Variante zuerst mergen und die anderen darauf rebasen.
Generated by Claude Code
|
Review (Claude Sonnet 5.5, Zweitmodell nach D-20260902-002) — Empfehlung: merge-ready, aber nach dem Stapel #15/#14/#17 mergen Reiner Test-Infrastruktur-PR (+129/-1, kein Produktivcode): Windows-Symlink-Teardown-Schutz und Qt-Widget-Cleanup in Hinweis: #15/#17 bringen ebenfalls eine |
conftest.py: master-Fixture (GUI-GC) beibehalten, Windows-Symlink-Teardown-Schutz aus diesem PR ergaenzt; test_checksum_dialog.py auf master-Stand (qtbot, waitUntil).
settings_dialog: master-Variante (Papierkorb-Tooltip); translations.json vereinigt, bei Doppelschluesseln master; CHANGELOG beide Abschnitte.
Ticket
T-20260928-295547351
Problem & Ursache
Loesung
Verifikation
uff check .: 0 Fehler.