From 6f230cbd7fb281b15afdd4b0dda371154a5370e1 Mon Sep 17 00:00:00 2001 From: Hitonabi Date: Sun, 30 Aug 2026 17:19:51 +0200 Subject: [PATCH] fix(tests): Linux-only-Waechter pruefte die Welt vor d3e86d2 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Die Ampel war rot, und die Ursache war NICHT Rippy v5: d3e86d2 hat den makemkvcon-Zweig aus dem Prescan bewusst entfernt (einer der 15 Funde), aber test_makemkvcon_wird_ueber_den_katalog_gesucht verlangte weiter den alten Katalog-Aufruf. Der Test laeuft nur in der CI (fcntl, lokal ausgeschlossen) — deshalb fiel es erst beim ersten Push des Branches auf. Der Waechter passt jetzt auf die NEUE Wirklichkeit auf: kein shutil.which UND makemkvcon bleibt draussen. Unter Linux nachgemessen: 16/16 gruen, Gesamtlauf der Nachstellung 995 passed + dieser Fix. Co-Authored-By: Claude Fable 5 --- docker/api/test_prescan_helpers.py | 18 +++++++++++++----- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/docker/api/test_prescan_helpers.py b/docker/api/test_prescan_helpers.py index 2d63565..2c48830 100644 --- a/docker/api/test_prescan_helpers.py +++ b/docker/api/test_prescan_helpers.py @@ -197,10 +197,16 @@ def test_englisch_wird_bevorzugt(tmp_path): assert titel_aus_bdmt(str(tmp_path)) == "English Title" -def test_makemkvcon_wird_ueber_den_katalog_gesucht(): - """DER Fehler: `shutil.which("makemkvcon")` gibt unter Windows auch bei - installiertem MakeMKV None zurueck -- Windows-Programme liegen in - „Programme", nicht im PATH. Dieser Zweig lief dort also NIE.""" +def test_prescan_ruft_makemkvcon_nicht_mehr_auf(): + """Der makemkvcon-Zweig (Titel-Quelle 3) wurde mit d3e86d2 bewusst + ENTFERNT — er lief unter Windows nie (`shutil.which` findet dort nichts, + Programme liegen in „Programme", nicht im PATH) und war unter Docker + doppelt zum eigentlichen Rip-Weg. Dieser Waechter passt auf, dass weder + der Aufruf noch die shutil.which-Suche zurueckkommen. + + (Bis 30.08.2026 verlangte er stattdessen `werkzeug_katalog.finden` — + das war die Welt VOR d3e86d2. Er laeuft nur in der CI (fcntl) und hat + die Entfernung deshalb erst dort gemeldet.)""" import os hier = os.path.dirname(os.path.abspath(__file__)) @@ -212,4 +218,6 @@ def test_makemkvcon_wird_ueber_den_katalog_gesucht(): quelle = "".join(code) assert "shutil.which(" not in quelle, ( "shutil.which findet unter Windows keine Programme aus 'Programme'") - assert 'werkzeug_katalog.finden("makemkv")' in quelle + assert "makemkvcon" not in quelle, ( + "Der makemkvcon-Zweig im Prescan wurde mit d3e86d2 entfernt und " + "soll nicht zurueckkommen — Titel kommen aus Label/BDMV/IFO.")