Hermes (Server)andClaude Opus 5
ca28264c0e
Der Suchen-Tab war 59 Zeilen: ein Textfeld ueber einer flachen
Liederliste, nur ueber Titel/Kuenstler/Album. Wer "Nightcore" eingab,
fand alle Titel dieses Kuenstlers untereinander, aber weder den
Kuenstler selbst noch eine gleichnamige Kategorie oder Liste.
- Vier Abschnitte mit Ueberschrift und Trefferzahl. Alben stecken in
"Kategorien" — der Album-Name ist in Melo die erste Kategorie eines
Titels, ein eigener Abschnitt waere dieselbe Liste zweimal.
- Treffer am Wortanfang zuerst, dann alphabetisch.
- Entprellt (250 ms); vorher baute jeder Tastendruck alles neu auf.
- Letzte acht Suchen, gemerkt beim Abschicken, mit Loeschen-Knopf.
- Der Tab folgt jetzt dem Aufbau der uebrigen Tabs (war ein Scaffold
mit eigener AppBar innerhalb eines Tabs).
- Bewusst rein lokal, ohne Server-Abfrage.
Herausgeloest, weil es sonst eine dritte Kopie gegeben haette:
TitelListenScreen ("Ueberschrift + Liederliste", stand zweimal wortgleich
im Baum) und SongZeile (die Suche setzt einzelne Zeilen in ihre
Abschnitte; eine ganze SongList waere dort verschachteltes Scrollen).
Aus dem Code-Review nachgebessert: Suche und Zuordnung benutzen dieselbe
Normalform, sonst zeigte ein Treffer "0 Titel" und oeffnete eine leere
Liste; Kategorien ohne lebende Titel fallen weg; alle Abschnitte sind
gedeckelt; die Drift-Stroeme werden nicht mehr je Rebuild neu abonniert;
das Loeschkreuz erscheint sofort statt nach 250 ms.
Tot geworden und entfernt: MeloDb.searchSongs samt Test.
396 Tests gruen (vorher 360), flutter analyze ohne Befund, Release-APK
gebaut.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013xAHJTJM6UUqmjUgk1PUEd
63 lines
2.3 KiB
Dart
63 lines
2.3 KiB
Dart
import 'package:drift/drift.dart';
|
|
import 'package:drift/native.dart';
|
|
import 'package:flutter_test/flutter_test.dart';
|
|
import 'package:melo/library/database.dart';
|
|
|
|
void main() {
|
|
late MeloDb db;
|
|
setUp(() => db = MeloDb(NativeDatabase.memory()));
|
|
tearDown(() => db.close());
|
|
|
|
SongsCompanion song(String id, String path, String title, {String? artist}) =>
|
|
SongsCompanion.insert(
|
|
id: id,
|
|
path: path,
|
|
title: title,
|
|
artist: Value(artist),
|
|
dateAddedMs: 1,
|
|
updatedAtMs: 1,
|
|
);
|
|
|
|
test('upsert + watchSongs sortiert nach Titel', () async {
|
|
await db.upsertSongs([song('1', '/a.mp3', 'Beta'), song('2', '/b.mp3', 'Alpha')]);
|
|
final songs = await db.watchSongs().first;
|
|
expect(songs.map((s) => s.title), ['Alpha', 'Beta']);
|
|
});
|
|
|
|
test('Re-upsert gleicher id erzeugt kein Duplikat, aktualisiert', () async {
|
|
await db.upsertSongs([song('1', '/a.mp3', 'Alt')]);
|
|
await db.upsertSongs([song('1', '/a.mp3', 'Neu')]);
|
|
final songs = await db.watchSongs().first;
|
|
expect(songs.length, 1);
|
|
expect(songs.first.title, 'Neu');
|
|
});
|
|
|
|
|
|
test('markMissing setzt Tombstone für nicht aktualisierte Dateien', () async {
|
|
final now1 = 100;
|
|
final now2 = 200;
|
|
// Initialer Scan: zwei Songs
|
|
await db.upsertSongs([
|
|
SongsCompanion.insert(id: '1', path: '/a.mp3', title: 'A', dateAddedMs: now1, updatedAtMs: now1),
|
|
SongsCompanion.insert(id: '2', path: '/b.mp3', title: 'B', dateAddedMs: now1, updatedAtMs: now1),
|
|
]);
|
|
// Zweiter Scan: nur /a.mp3 gefunden
|
|
await db.upsertSongs([
|
|
SongsCompanion.insert(id: '1', path: '/a.mp3', title: 'A', dateAddedMs: now1, updatedAtMs: now2),
|
|
]);
|
|
await db.markMissing(now2); // Songs mit updatedAtMs < now2 löschen
|
|
final songs = await db.watchSongs().first;
|
|
expect(songs.map((s) => s.path), ['/a.mp3']);
|
|
});
|
|
|
|
test('scan_test.dart: CRITICAL-1 — empty scan deletes nothing (in scan_test.dart)', () {
|
|
// Diese Regression ist in scan_test.dart mit echtem scanFolders() getestet.
|
|
// Hier nur ein Placeholder um die alte Test-Nummer zu behalten.
|
|
});
|
|
|
|
test('scan_test.dart: CRITICAL-2 — rescan restores deleted songs (in scan_test.dart)', () {
|
|
// Diese Regression ist in scan_test.dart mit echtem scanFolders() getestet.
|
|
// Hier nur ein Placeholder um die alte Test-Nummer zu behalten.
|
|
});
|
|
}
|