Fix: Fortschritts-Regex toleriert Leerzeichen vor % (Testfund)
Ampel / ampel (push) Failing after 14m47s
Ampel / ampel (push) Failing after 14m47s
Echtes HandBrake schreibt "45.50 %" mit Leerzeichen - die alte Regex hat aus echter Ausgabe NIE einen Fortschritt geparst. Der neue Test mit realistischer Zeile hat es sofort gefangen.
This commit is contained in:
@@ -35,8 +35,13 @@ def check_cdparanoia_installed() -> bool:
|
|||||||
|
|
||||||
|
|
||||||
def get_progress_from_line(line: str) -> int:
|
def get_progress_from_line(line: str) -> int:
|
||||||
"""Extrahiert Fortschritt in Prozent aus HandBrake-Ausgabe."""
|
"""Extrahiert Fortschritt in Prozent aus HandBrake-Ausgabe.
|
||||||
match = re.search(r'(\d+\.\d+)%', line)
|
|
||||||
|
Testfund 22.07.: echtes HandBrake schreibt „45.50 %" MIT Leerzeichen vor
|
||||||
|
dem Prozentzeichen — die alte Regex ohne \\s* hat NIE einen Fortschritt
|
||||||
|
aus echter Ausgabe geparst.
|
||||||
|
"""
|
||||||
|
match = re.search(r'(\d+\.\d+)\s*%', line)
|
||||||
if match:
|
if match:
|
||||||
return int(float(match.group(1)))
|
return int(float(match.group(1)))
|
||||||
return 0
|
return 0
|
||||||
|
|||||||
Reference in New Issue
Block a user