ef0a574a70
Vier Funde aus der Durchsicht, alle auf der VM gemessen.
1. DER PLATTEN-SCHUTZ AUS c065967 WAR WIRKUNGSLOS
_original_aufheben() entschied per os.stat().st_dev, ob umgehaengt oder
kopiert werden muss. Im Worker-Container gemessen - beides gleichzeitig wahr:
st_dev /app/temp = 2050
st_dev /app/media = 2050 → identisch
os.rename(...) → EXDEV, "Invalid cross-device link"
Der Kernel vergleicht bei rename() den MOUNT, nicht das Geraet. /app/temp
(Docker-Volume) und /app/media (Bind-Mount) sind zwei Mounts DERSELBEN
ext4-Partition. Die Pruefung sah "gleiches Dateisystem", uebersprang die
Platzpruefung, und shutil.move kopierte doch - 75 GB bei 37 GB frei. Der
Schutz haette genau den Schaden zugelassen, gegen den er gebaut wurde.
Jetzt wird os.rename VERSUCHT statt vorhergesagt: klappt es, ist es
umgehaengt und fertig; kommt EXDEV, steht die Kopie fest und ERST DANN wird
der Platz geprueft. Das ist keine Vermutung mehr, sondern die Antwort des
Kernels. Vier Tests in test_original_aufheben.py, darunter genau der Fall,
der die Platte fuellte. Die zwei alten Tests in test_medien.py sind dorthin
gewandert - sie taeuschten per gefaelschtem os.stat "verschiedene
Dateisysteme" vor, also genau die Annahme, an der der Schutz scheiterte.
2. DIE FORTSCHRITTSANZEIGE ZEIGTE DEN SCAN, NICHT DEN ENCODE
get_progress_from_line matchte jede Zahl vor einem Prozentzeichen. HandBrake
gibt Prozente aber in drei Phasen aus (Formatstrings aus dem Binary gelesen):
Scanning title %d of %d, preview %d, %.2f %% → laeuft VOR dem
Encode bis 100 %
Encoding: task %d of %d, %.2f %% (%.2f fps, avg → der echte Wert
Encoding: task %d of %d, Searching for start time, ... → Vorlauf
Dazu warf `if progress > 0` im Aufrufer jeden Wert unter 1,00 % weg. Live
beobachtet: Anzeige stand auf 99 %, der Encode bei 1,06 %; sie fiel erst auf
1, als der Encode die 1-%-Marke ueberschritt. Jetzt wird nur die
Encoding-Zeile gelesen, `task N of M` mitgerechnet (sonst springt die
Anzeige bei Zwei-Pass-Presets mitten in der Datei zurueck), und -1 heisst
"keine Angabe" - dasselbe Muster wie bei get_progress_from_prgv.
3. "AUTOMATISCHER AUSWURF" WURDE VON NIEMANDEM GELESEN
Die Einstellung (Standard: ein, "Disc nach erfolgreichem Ripping automatisch
auswerfen") kam in keiner Zeile Backend-Code vor. DVD/Blu-ray warfen deshalb
NIE aus, Audio-CDs IMMER, weil abcde `-x` fest verdrahtet bekam. Jetzt
entscheidet die Einstellung beides: wirf_disc_aus() per CDROMEJECT-ioctl
(fcntl-guarded, der native Windows-Worker laedt das Modul auch) und `-x` nur
noch, wenn gewuenscht.
4. PFAD-PRUEFUNG FIEL AUF PRAEFIX-NAMEN HEREIN
Elf Stellen prueften mit nacktem startswith(MEDIA_ROOT). "/app/media-boese/x"
beginnt mit "/app/media", liegt aber ausserhalb - betroffen waren auch
/browse und /browse/mkdir, wo der Pfad vom Nutzer kommt. Neuer
Zwillings-Helfer unter_wurzel() in api/main.py und worker/tasks.py, alle elf
Stellen umgestellt, Tests in beiden.
Nebenbefund: _zielbasis() benutzte os.path.normpath - unter Windows werden
daraus Backslashes, die MEDIA_ROOT-Pruefung greift nicht mehr, und das
gewaehlte Ziel faellt still auf den Standard zurueck. Genau die Falle, die
_arbeitsverzeichnis() drei Zeilen weiter dokumentiert und mit posixpath
vermeidet. Live war es nie (nur aus rip_disc, das auf Windows verriegelt
ist), jetzt konsistent.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
176 lines
7.0 KiB
Python
176 lines
7.0 KiB
Python
"""Tests für die Media-Server-Aufbereitung (Benennung, NFO, Serien-Matching)."""
|
|
|
|
import os
|
|
|
|
from medien import baue_nfo, matche_episoden, serien_ordner, sicherer_name, zielordner
|
|
|
|
|
|
def test_sicherer_name_entfernt_verbotene_zeichen():
|
|
assert sicherer_name('Alien: Die Wiedergeburt') == "Alien Die Wiedergeburt"
|
|
assert sicherer_name('Was/ist\\das?<>|*"') == "Wasistdas"
|
|
|
|
|
|
def test_sicherer_name_mit_jahr():
|
|
assert sicherer_name("Evangelion 2.22", 2009) == "Evangelion 2.22 (2009)"
|
|
|
|
|
|
def test_sicherer_name_kollabiert_leerraum_und_punkte():
|
|
assert sicherer_name(" Viel Raum ... ") == "Viel Raum"
|
|
|
|
|
|
def test_sicherer_name_leer_bleibt_leer():
|
|
assert sicherer_name("") == ""
|
|
assert sicherer_name("???") == ""
|
|
|
|
|
|
def test_zielordner_nutzt_titel_und_jahr(tmp_path):
|
|
pfad = zielordner(str(tmp_path), "Inception", 2010, "abc-123")
|
|
assert pfad == os.path.join(str(tmp_path), "Inception (2010)")
|
|
|
|
|
|
def test_zielordner_faellt_auf_job_id_zurueck(tmp_path):
|
|
pfad = zielordner(str(tmp_path), "", None, "abc-123")
|
|
assert pfad == os.path.join(str(tmp_path), "abc-123")
|
|
|
|
|
|
def test_zielordner_weicht_bei_kollision_aus(tmp_path):
|
|
os.makedirs(tmp_path / "Inception (2010)")
|
|
pfad = zielordner(str(tmp_path), "Inception", 2010, "abcdef12-3456")
|
|
assert pfad == os.path.join(str(tmp_path), "Inception (2010) [abcdef12]")
|
|
|
|
|
|
def test_baue_nfo_film_mit_plot_und_genres():
|
|
nfo = baue_nfo(
|
|
{"overview": "Ein Traum <im> Traum & so", "genres": ["Sci-Fi", "Action"]},
|
|
"Inception", 2010,
|
|
)
|
|
assert "<movie>" in nfo and "</movie>" in nfo
|
|
assert "<title>Inception</title>" in nfo
|
|
assert "<year>2010</year>" in nfo
|
|
# XML-Escaping: <, >, & dürfen den Parser nicht sprengen
|
|
assert "Ein Traum <im> Traum & so" in nfo
|
|
assert "<genre>Sci-Fi</genre>" in nfo
|
|
|
|
|
|
def test_baue_nfo_serie_bekommt_tvshow_wurzel():
|
|
nfo = baue_nfo({"type": "tv"}, "Neon Genesis Evangelion", 1995)
|
|
assert "<tvshow>" in nfo and "</tvshow>" in nfo
|
|
|
|
|
|
def test_sicherer_name_haengt_jahr_nicht_doppelt_an():
|
|
"""OMDb-Titel tragen das Jahr teils schon — kein 'X (2009) (2009)' mehr."""
|
|
assert sicherer_name("Summer Wars (2009)", 2009) == "Summer Wars (2009)"
|
|
assert sicherer_name("Summer Wars", 2009) == "Summer Wars (2009)"
|
|
|
|
|
|
def test_serien_ordner_jellyfin_schema():
|
|
assert serien_ordner("/app/media/series", "Neon Genesis: Evangelion", 2) == os.path.join(
|
|
"/app/media/series", "Neon Genesis Evangelion", "Season 02"
|
|
)
|
|
|
|
|
|
def test_matche_episoden_komplette_staffel_auch_bei_uniformen_laufzeiten():
|
|
# Anime-Standardfall: ganze Staffel auf der Disc, alle Episoden ~24 min
|
|
dateien = [1440, 1452, 1431]
|
|
episoden = [(1, 1440), (2, 1440), (3, 1440)]
|
|
assert matche_episoden(dateien, episoden) == [1, 2, 3]
|
|
|
|
|
|
def test_matche_episoden_teil_disc_mit_distinkten_laufzeiten():
|
|
# Disc 2 einer Staffel: Laufzeiten unterscheiden sich klar → zuordenbar
|
|
dateien = [1440, 2650]
|
|
episoden = [(1, 700), (2, 1440), (3, 2650)]
|
|
assert matche_episoden(dateien, episoden) == [2, 3]
|
|
|
|
|
|
def test_matche_episoden_haelt_reihenfolge():
|
|
# Datei 2 ist ein Doppellänger → muss auf die SPÄTERE lange Episode fallen
|
|
dateien = [1440, 2880]
|
|
episoden = [(1, 1440), (2, 2880)]
|
|
assert matche_episoden(dateien, episoden) == [1, 2]
|
|
|
|
|
|
def test_matche_episoden_mehrdeutig_gibt_none():
|
|
# Alle Episoden gleich lang und mehr Kandidaten als Dateien in Toleranz →
|
|
# Datei 1 passt auf Ep 1 UND Ep 2 gleich gut → keine eindeutige Wahl
|
|
dateien = [1440]
|
|
episoden = [(1, 1440), (2, 1440)]
|
|
assert matche_episoden(dateien, episoden) is None
|
|
|
|
|
|
def test_matche_episoden_ohne_treffer_gibt_none():
|
|
dateien = [600] # 10 min passt auf keine 24-min-Episode
|
|
episoden = [(1, 1440), (2, 1440)]
|
|
assert matche_episoden(dateien, episoden) is None
|
|
|
|
|
|
def test_pfad_lokal_uebersetzt_fuer_windows_worker():
|
|
from tasks import pfad_lokal
|
|
|
|
mapping = "/app/media=Z:\\media;/app/temp=Y:\\temp"
|
|
assert pfad_lokal("/app/temp/raw/abc", mapping) == "Y:\\temp\\raw\\abc"
|
|
assert pfad_lokal("/app/media/movies/Film (2020)", mapping) == "Z:\\media\\movies\\Film (2020)"
|
|
assert pfad_lokal("/woanders/x", mapping) == "/woanders/x"
|
|
assert pfad_lokal("/app/temp/raw/abc", "") == "/app/temp/raw/abc"
|
|
|
|
|
|
# Die beiden frueheren Tests zu _original_aufheben sind nach
|
|
# test_original_aufheben.py gewandert und dort erweitert. Sie pruefen jetzt das
|
|
# richtige Kriterium: Vorher taeuschten sie per gefaelschtem os.stat
|
|
# „verschiedene Dateisysteme" vor — genau die Annahme, an der der Schutz in der
|
|
# Praxis scheiterte (st_dev war identisch, os.rename trotzdem EXDEV).
|
|
|
|
|
|
def test_arbeitsverzeichnis_wahl_des_rips_schlaegt_die_einstellung():
|
|
"""Pro Rip waehlbar (Commander 25.07.2026), Einstellung bleibt Standard.
|
|
|
|
Reihenfolge: Wahl dieses Rips -> Setting -> Container-Default. Der
|
|
Setting-Wert ist genau der, der bei Vollautomatik-Rips greift, weil dort
|
|
niemand gefragt wird.
|
|
"""
|
|
import tasks
|
|
|
|
einst = {"workDir": "/app/media/movies"}
|
|
assert tasks._arbeitsverzeichnis(einst, "/app/media/rippy") == "/app/media/rippy"
|
|
assert tasks._arbeitsverzeichnis(einst) == "/app/media/movies"
|
|
assert tasks._arbeitsverzeichnis({}) == tasks.RAW_DIR
|
|
# Ausbruchsversuche und Pfade ausserhalb /app/media fallen durch
|
|
assert tasks._arbeitsverzeichnis({}, "/etc") == tasks.RAW_DIR
|
|
assert tasks._arbeitsverzeichnis({}, "/app/media/../etc") == tasks.RAW_DIR
|
|
# Leere Wahl faellt sauber auf die Einstellung zurueck
|
|
assert tasks._arbeitsverzeichnis(einst, " ") == "/app/media/movies"
|
|
|
|
|
|
def test_unter_wurzel_faellt_nicht_auf_praefix_namen_herein():
|
|
"""Befund 25.07.2026: Elf Stellen prueften mit nacktem startswith().
|
|
„/app/media-boese/x" beginnt mit „/app/media", liegt aber ausserhalb."""
|
|
import tasks
|
|
|
|
assert tasks.unter_wurzel("/app/media", "/app/media") is True
|
|
assert tasks.unter_wurzel("/app/media/movies", "/app/media") is True
|
|
assert tasks.unter_wurzel("/app/media/movies/Film (2020)", "/app/media") is True
|
|
# Der Fall, den startswith() durchgelassen haette
|
|
assert tasks.unter_wurzel("/app/media-boese/x", "/app/media") is False
|
|
assert tasks.unter_wurzel("/app/mediaX", "/app/media") is False
|
|
assert tasks.unter_wurzel("/etc/passwd", "/app/media") is False
|
|
assert tasks.unter_wurzel("", "/app/media") is False
|
|
assert tasks.unter_wurzel("/app/media", "") is False
|
|
# Wurzel mit Schrägstrich am Ende darf nichts aendern
|
|
assert tasks.unter_wurzel("/app/media/movies", "/app/media/") is True
|
|
|
|
|
|
def test_zielbasis_lehnt_praefix_ausbruch_ab():
|
|
import tasks
|
|
|
|
assert tasks._zielbasis("/app/media/movies", "bluray") == "/app/media/movies"
|
|
# Ausbruch per Praefix-Namen faellt auf den Standard zurueck
|
|
assert tasks._zielbasis("/app/media-boese", "bluray") != "/app/media-boese"
|
|
assert tasks._zielbasis("/etc", "bluray") != "/etc"
|
|
|
|
|
|
def test_arbeitsverzeichnis_lehnt_praefix_ausbruch_ab():
|
|
import tasks
|
|
|
|
assert tasks._arbeitsverzeichnis({}, "/app/media-boese") == tasks.RAW_DIR
|
|
assert tasks._arbeitsverzeichnis({"workDir": "/app/mediaX"}) == tasks.RAW_DIR
|