Verschobene Dateien hinterliessen eine Leiche
Seit verschwundene Dateien markiert statt gelöscht werden, blieb nach jeder Umbenennung die alte Zeile als "vermisst" stehen. Damit galt die alte Folge nicht als verwaist, die alte Serie nicht als leer — und nach dem Umbenennen eines Serienordners stand alles doppelt da. Jetzt erkennt der Scan eine Verschiebung an Grösse und Änderungszeit und hängt den vorhandenen Eintrag um, statt einen zweiten anzulegen. Der Zeitstempel musste dafür auf Sekunden gerundet werden: er geht durch die Datenbank und kommt nicht auf den Tick genau zurück. Mit Vergleich auf Gleichheit schlug die Erkennung ausnahmslos fehl — der Test zeigte es, nachdem er zweimal dasselbe falsche Ergebnis lieferte. Grösse und Zeit müssen beide stimmen, und das Merkmal muss eindeutig sein. Grösse allein reichte nicht: zwei gleich grosse Dateien gibt es, und die zu vertauschen wäre schlimmer als eine Leiche. Dazu: die Zahl der Titel läuft während eines Scans mit, statt erst am Ende zu springen. Alle fünf Runden, nicht jede — ein Abruf je Sekunde für eine Zahl wäre Verschwendung. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
b64497acd5
commit
fe44d45323
@@ -302,6 +302,46 @@ public class ScanTests
|
||||
Assert.That(filme!.Total, Is.EqualTo(2));
|
||||
}
|
||||
|
||||
/// <summary>Eine Serie, die ihren Namen verliert, bleibt nicht als Leiche stehen.</summary>
|
||||
///
|
||||
/// <remarks>
|
||||
/// <b>Der Weg, den ein vorhandener Bestand nach dem Update geht.</b> Vorher
|
||||
/// bestimmte der Dateiname die Serie; eine Datei direkt in der Wurzel
|
||||
/// ergibt das bis heute. Wandert sie in einen Ordner, gehört sie zu diesem
|
||||
/// — und der alte Eintrag muss verschwinden, statt als leere Serie in der
|
||||
/// Mediathek zu bleiben.
|
||||
/// </remarks>
|
||||
[Test]
|
||||
public async Task Eine_verwaiste_Serie_verschwindet_beim_naechsten_Lauf()
|
||||
{
|
||||
Datei("S01E01 Kapitel 1.mkv");
|
||||
|
||||
using var server = new FlaroServer();
|
||||
var (client, bibliothek) = await MitBibliothek(server, "Shows");
|
||||
await Scannen(client, bibliothek);
|
||||
|
||||
var vorher = await client.GetFromJsonAsync<Page<SeriesResponse>>("/api/v1/series");
|
||||
Assert.That(vorher!.Total, Is.EqualTo(1), "Ohne Ordner bleibt nur der Dateiname.");
|
||||
var alterName = vorher.Items[0].Title;
|
||||
|
||||
// **Verschieben, nicht löschen und neu anlegen.** Beim Verschieben
|
||||
// bleibt der Zeitstempel erhalten — und genau daran erkennt der Scan,
|
||||
// dass es dieselbe Datei ist. Ein Test, der sie neu anlegt, prüft
|
||||
// etwas anderes als das, was ein Mensch tut.
|
||||
var alt = Path.Combine(_bestand, "S01E01 Kapitel 1.mkv");
|
||||
var neuerPfad = Path.Combine(_bestand, "Nachtstreife", "S01E01 Kapitel 1.mkv");
|
||||
Directory.CreateDirectory(Path.GetDirectoryName(neuerPfad)!);
|
||||
File.Move(alt, neuerPfad);
|
||||
|
||||
await Scannen(client, bibliothek);
|
||||
|
||||
var nachher = await client.GetFromJsonAsync<Page<SeriesResponse>>("/api/v1/series");
|
||||
Assert.That(nachher!.Total, Is.EqualTo(1), "Die alte Serie darf nicht danebenstehen.");
|
||||
Assert.That(nachher.Items[0].Title, Is.EqualTo("Nachtstreife"));
|
||||
Assert.That(nachher.Items[0].Title, Is.Not.EqualTo(alterName));
|
||||
Assert.That(nachher.Items[0].EpisodeCount, Is.EqualTo(1));
|
||||
}
|
||||
|
||||
/// <summary>4K und 1080p sind ein Film mit zwei Dateien, nicht zwei Filme.</summary>
|
||||
[Test]
|
||||
public async Task Zwei_Fassungen_desselben_Films_bleiben_ein_Eintrag()
|
||||
|
||||
Reference in New Issue
Block a user