Suche trennt nach Titel, Kuenstler, Kategorien und Wiedergabelisten
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
This commit is contained in:
Hermes (Server)
co-authored by
Claude Opus 5
Claude Opus 5
parent
d15867d962
commit
ca28264c0e
@@ -0,0 +1,170 @@
|
||||
import 'package:drift/drift.dart' show Value, driftRuntimeOptions;
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:provider/provider.dart';
|
||||
import 'package:shared_preferences/shared_preferences.dart';
|
||||
import 'package:melo/library/category_service.dart';
|
||||
import 'package:melo/library/database.dart';
|
||||
import 'package:melo/library/playlist_service.dart';
|
||||
import 'package:melo/library/search_screen.dart';
|
||||
import 'package:melo/settings/app_settings.dart';
|
||||
|
||||
/// Die Suche zeigte früher eine flache Liederliste. Jetzt trennt sie die
|
||||
/// Treffer nach Art — dieser Test prüft, dass die Abschnitte erscheinen und
|
||||
/// nur die passenden Treffer enthalten.
|
||||
void main() {
|
||||
late MeloDb db;
|
||||
late CategoryService kategorien;
|
||||
late AppSettings einstellungen;
|
||||
late PlaylistService playlists;
|
||||
|
||||
setUp(() async {
|
||||
// Jeder Test legt eine eigene In-Memory-Datenbank an; die Warnung über
|
||||
// mehrere Instanzen ist hier erwartet und kein Befund.
|
||||
driftRuntimeOptions.dontWarnAboutMultipleDatabases = true;
|
||||
SharedPreferences.setMockInitialValues({});
|
||||
db = MeloDb(NativeDatabase.memory());
|
||||
kategorien = CategoryService(db);
|
||||
playlists = PlaylistService(db);
|
||||
einstellungen = AppSettings();
|
||||
await einstellungen.init();
|
||||
|
||||
Future<void> song(String id, String titel, String kuenstler) =>
|
||||
db.into(db.songs).insert(SongsCompanion.insert(
|
||||
id: id,
|
||||
path: '/music/$id.mp3',
|
||||
title: titel,
|
||||
artist: Value(kuenstler),
|
||||
dateAddedMs: 0,
|
||||
updatedAtMs: 0,
|
||||
));
|
||||
|
||||
await song('s1', 'Roses', 'Neoni');
|
||||
await song('s2', 'Demons', 'Nightcore Gaming');
|
||||
await song('s3', 'Wild Roses', 'Lori');
|
||||
await db.setCategories('s1', ['Trap cool']);
|
||||
await db.createPlaylist('Rosengarten');
|
||||
});
|
||||
|
||||
// Kein db.close(): in Widget-Tests plant drift dabei eine Aufräum-Aufgabe
|
||||
// ein, die der Testrahmen nicht mehr abarbeitet (siehe BACKLOG.md). Die
|
||||
// Dienste liegen als `.value` im Baum, werden also nicht von ihm entsorgt.
|
||||
tearDown(() {
|
||||
kategorien.dispose();
|
||||
playlists.dispose();
|
||||
});
|
||||
|
||||
Future<void> pumpe(WidgetTester tester) async {
|
||||
await tester.pumpWidget(
|
||||
MultiProvider(
|
||||
providers: [
|
||||
Provider<MeloDb>.value(value: db),
|
||||
ChangeNotifierProvider<CategoryService>.value(value: kategorien),
|
||||
ChangeNotifierProvider<AppSettings>.value(value: einstellungen),
|
||||
ChangeNotifierProvider<PlaylistService>.value(value: playlists),
|
||||
],
|
||||
child: const MaterialApp(home: Scaffold(body: SearchScreen())),
|
||||
),
|
||||
);
|
||||
await tester.pumpAndSettle();
|
||||
}
|
||||
|
||||
Future<void> tippe(WidgetTester tester, String text) async {
|
||||
await tester.enterText(find.byType(TextField), text);
|
||||
// Die Eingabe ist entprellt — ohne dieses Warten passiert nichts.
|
||||
await tester.pump(const Duration(milliseconds: 300));
|
||||
await tester.pumpAndSettle();
|
||||
}
|
||||
|
||||
Future<void> aufraeumen(WidgetTester tester) async {
|
||||
await tester.pumpWidget(const SizedBox());
|
||||
await tester.pump(const Duration(milliseconds: 1));
|
||||
}
|
||||
|
||||
testWidgets('leeres Feld zeigt den Hinweis statt Treffer', (tester) async {
|
||||
await pumpe(tester);
|
||||
|
||||
expect(find.text('Suchbegriff eingeben'), findsOneWidget);
|
||||
expect(find.text('TITEL 2'), findsNothing);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('trennt Treffer nach Titel, Künstler und Listen',
|
||||
(tester) async {
|
||||
await pumpe(tester);
|
||||
await tippe(tester, 'rose');
|
||||
|
||||
expect(find.text('TITEL 2'), findsOneWidget);
|
||||
expect(find.text('WIEDERGABELISTEN 1'), findsOneWidget);
|
||||
expect(find.text('Roses'), findsOneWidget);
|
||||
expect(find.text('Wild Roses'), findsOneWidget);
|
||||
expect(find.text('Rosengarten'), findsOneWidget);
|
||||
// "Demons" passt nicht und darf nicht auftauchen.
|
||||
expect(find.text('Demons'), findsNothing);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('findet Künstler als eigenen Abschnitt', (tester) async {
|
||||
await pumpe(tester);
|
||||
await tippe(tester, 'nightcore');
|
||||
|
||||
expect(find.text('KÜNSTLER 1'), findsOneWidget);
|
||||
expect(find.text('Nightcore Gaming'), findsWidgets);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('findet Kategorien als eigenen Abschnitt', (tester) async {
|
||||
await pumpe(tester);
|
||||
await tippe(tester, 'trap');
|
||||
|
||||
expect(find.text('KATEGORIEN 1'), findsOneWidget);
|
||||
expect(find.text('Trap cool'), findsWidgets);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('ohne Treffer erscheint "Nichts gefunden"', (tester) async {
|
||||
await pumpe(tester);
|
||||
await tippe(tester, 'jazzfrühstück');
|
||||
|
||||
expect(find.text('Nichts gefunden'), findsOneWidget);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('entprellt: direkt nach dem Tippen steht noch nichts da',
|
||||
(tester) async {
|
||||
await pumpe(tester);
|
||||
await tester.enterText(find.byType(TextField), 'rose');
|
||||
// Weniger als die Entprellzeit — die Treffer dürfen noch nicht stehen.
|
||||
await tester.pump(const Duration(milliseconds: 100));
|
||||
|
||||
expect(find.text('TITEL 2'), findsNothing);
|
||||
|
||||
await tester.pump(const Duration(milliseconds: 300));
|
||||
await tester.pumpAndSettle();
|
||||
expect(find.text('TITEL 2'), findsOneWidget);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
|
||||
testWidgets('abgeschickte Suche steht danach im Verlauf', (tester) async {
|
||||
await pumpe(tester);
|
||||
await tippe(tester, 'rose');
|
||||
await tester.testTextInput.receiveAction(TextInputAction.search);
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
// Feld leeren — dann muss der Verlauf sichtbar werden.
|
||||
await tester.tap(find.byTooltip('Eingabe löschen'));
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
expect(find.text('ZULETZT GESUCHT'), findsOneWidget);
|
||||
expect(find.text('rose'), findsOneWidget);
|
||||
|
||||
await aufraeumen(tester);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,47 @@
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/library/such_verlauf.dart';
|
||||
|
||||
/// Die Reihenfolge-Regeln des Verlaufs — ohne SharedPreferences prüfbar.
|
||||
void main() {
|
||||
group('verlaufMit', () {
|
||||
test('legt den neuen Begriff nach vorn', () {
|
||||
expect(verlaufMit(['neoni', 'rock'], 'trap'), ['trap', 'neoni', 'rock']);
|
||||
});
|
||||
|
||||
test('ein erneut gesuchter Begriff wandert nach vorn statt sich zu doppeln',
|
||||
() {
|
||||
expect(verlaufMit(['neoni', 'rock'], 'rock'), ['rock', 'neoni']);
|
||||
});
|
||||
|
||||
test('Groß- und Kleinschreibung gilt als derselbe Begriff', () {
|
||||
expect(verlaufMit(['Rock'], 'rock'), ['rock']);
|
||||
});
|
||||
|
||||
test('Leerraum am Rand zählt nicht mit', () {
|
||||
expect(verlaufMit(['rock'], ' rock '), ['rock']);
|
||||
});
|
||||
|
||||
test('deckelt bei acht Einträgen', () {
|
||||
final voll = ['a', 'b', 'c', 'd', 'e', 'f', 'g', 'h'];
|
||||
|
||||
final neu = verlaufMit(voll, 'i');
|
||||
|
||||
expect(neu.length, 8);
|
||||
expect(neu.first, 'i');
|
||||
// Der älteste fällt hinten heraus.
|
||||
expect(neu, isNot(contains('h')));
|
||||
});
|
||||
|
||||
test('ein leerer Begriff ändert nichts', () {
|
||||
expect(verlaufMit(['rock'], ' '), ['rock']);
|
||||
});
|
||||
|
||||
test('lässt die Ausgangsliste unverändert', () {
|
||||
final vorher = ['rock'];
|
||||
|
||||
verlaufMit(vorher, 'trap');
|
||||
|
||||
expect(vorher, ['rock']);
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,227 @@
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/library/database.dart';
|
||||
import 'package:melo/library/suche.dart';
|
||||
|
||||
Song _song(String titel, {String? kuenstler, String? album, String? id}) => Song(
|
||||
id: id ?? titel.toLowerCase().replaceAll(' ', '-'),
|
||||
path: '/music/$titel.mp3',
|
||||
title: titel,
|
||||
artist: kuenstler,
|
||||
album: album,
|
||||
dateAddedMs: 0,
|
||||
updatedAtMs: 0,
|
||||
deleted: false,
|
||||
playCount: 0,
|
||||
categoriesEdited: false,
|
||||
metadataEdited: false,
|
||||
);
|
||||
|
||||
Playlist _liste(String name) => Playlist(
|
||||
id: name.toLowerCase(),
|
||||
name: name,
|
||||
createdAtMs: 0,
|
||||
updatedAtMs: 0,
|
||||
deleted: false,
|
||||
);
|
||||
|
||||
SuchTreffer _suche(
|
||||
String query, {
|
||||
List<Song>? songs,
|
||||
List<String>? kategorien,
|
||||
List<Playlist>? playlisten,
|
||||
}) =>
|
||||
sucheLokal(
|
||||
songs: songs ?? const [],
|
||||
kategorien: kategorien ?? const [],
|
||||
playlisten: playlisten ?? const [],
|
||||
query: query,
|
||||
);
|
||||
|
||||
void main() {
|
||||
group('leere Eingabe', () {
|
||||
test('findet nichts', () {
|
||||
final treffer = _suche('', songs: [_song('Roses')]);
|
||||
|
||||
expect(treffer.istLeer, isTrue);
|
||||
});
|
||||
|
||||
test('nur Leerzeichen zählen als leer', () {
|
||||
final treffer = _suche(' ', songs: [_song('Roses')]);
|
||||
|
||||
expect(treffer.istLeer, isTrue);
|
||||
});
|
||||
});
|
||||
|
||||
group('Titel', () {
|
||||
final songs = [
|
||||
_song('Roses', kuenstler: 'Neoni', album: 'Trailerpark'),
|
||||
_song('Demons', kuenstler: 'Nightcore'),
|
||||
_song('Wild Roses', kuenstler: 'Lori'),
|
||||
];
|
||||
|
||||
test('findet über den Titel', () {
|
||||
expect(_suche('demons', songs: songs).titel.map((s) => s.title),
|
||||
['Demons']);
|
||||
});
|
||||
|
||||
test('findet über den Künstler', () {
|
||||
expect(_suche('neoni', songs: songs).titel.map((s) => s.title),
|
||||
['Roses']);
|
||||
});
|
||||
|
||||
test('findet über das Album', () {
|
||||
expect(_suche('trailerpark', songs: songs).titel.map((s) => s.title),
|
||||
['Roses']);
|
||||
});
|
||||
|
||||
test('ignoriert Groß- und Kleinschreibung', () {
|
||||
expect(_suche('ROSES', songs: songs).titel.length, 2);
|
||||
});
|
||||
|
||||
test('Treffer am Wortanfang stehen vorn', () {
|
||||
// "Roses" beginnt mit der Eingabe, "Wild Roses" enthält sie nur.
|
||||
expect(_suche('roses', songs: songs).titel.map((s) => s.title),
|
||||
['Roses', 'Wild Roses']);
|
||||
});
|
||||
|
||||
test('ein Titel erscheint nur einmal, auch wenn mehrere Felder passen', () {
|
||||
final doppelt = [_song('Neoni', kuenstler: 'Neoni', album: 'Neoni')];
|
||||
|
||||
expect(_suche('neoni', songs: doppelt).titel.length, 1);
|
||||
});
|
||||
});
|
||||
|
||||
group('Künstler', () {
|
||||
final songs = [
|
||||
_song('A', kuenstler: 'Nightcore Gaming'),
|
||||
_song('B', kuenstler: 'Nightcore Gaming'),
|
||||
_song('C', kuenstler: 'Anime Nightcore'),
|
||||
_song('D', kuenstler: 'Neoni'),
|
||||
_song('E'),
|
||||
];
|
||||
|
||||
test('nennt jeden Künstler nur einmal', () {
|
||||
expect(_suche('nightcore', songs: songs).kuenstler,
|
||||
['Nightcore Gaming', 'Anime Nightcore']);
|
||||
});
|
||||
|
||||
test('Treffer am Wortanfang stehen vorn', () {
|
||||
expect(_suche('nightcore', songs: songs).kuenstler.first,
|
||||
'Nightcore Gaming');
|
||||
});
|
||||
|
||||
test('Titel ohne Künstler erzeugen keinen leeren Eintrag', () {
|
||||
expect(_suche('e', songs: songs).kuenstler, isNot(contains('')));
|
||||
});
|
||||
});
|
||||
|
||||
group('Kategorien', () {
|
||||
const kategorien = ['Trap cool', 'Trap neu', 'Nightcore', 'Rock'];
|
||||
|
||||
test('findet passende Kategorien', () {
|
||||
expect(_suche('trap', kategorien: kategorien).kategorien,
|
||||
['Trap cool', 'Trap neu']);
|
||||
});
|
||||
|
||||
test('findet nichts Unpassendes', () {
|
||||
expect(_suche('jazz', kategorien: kategorien).kategorien, isEmpty);
|
||||
});
|
||||
});
|
||||
|
||||
group('Wiedergabelisten', () {
|
||||
final listen = [_liste('Sport'), _liste('Sonntagmorgen')];
|
||||
|
||||
test('findet über den Namen', () {
|
||||
expect(_suche('sonn', playlisten: listen).playlisten.map((p) => p.name),
|
||||
['Sonntagmorgen']);
|
||||
});
|
||||
});
|
||||
|
||||
group('Zusammenspiel', () {
|
||||
test('zählt alle Abschnitte zusammen', () {
|
||||
final treffer = _suche(
|
||||
'rock',
|
||||
songs: [_song('Rock Anthem', kuenstler: 'Rocker')],
|
||||
kategorien: ['Rock'],
|
||||
playlisten: [_liste('Rock-Liste')],
|
||||
);
|
||||
|
||||
expect(treffer.titel.map((s) => s.title), ['Rock Anthem']);
|
||||
expect(treffer.kuenstler, ['Rocker']);
|
||||
expect(treffer.kategorien, ['Rock']);
|
||||
expect(treffer.playlisten.map((p) => p.name), ['Rock-Liste']);
|
||||
expect(treffer.istLeer, isFalse);
|
||||
});
|
||||
|
||||
test('ohne Treffer ist das Ergebnis leer, nicht null', () {
|
||||
final treffer = _suche('gibtesnicht', songs: [_song('Roses')]);
|
||||
|
||||
expect(treffer.istLeer, isTrue);
|
||||
expect(treffer.titel, isEmpty);
|
||||
expect(treffer.kuenstler, isEmpty);
|
||||
});
|
||||
});
|
||||
|
||||
group('kuenstlerNachSchluessel', () {
|
||||
test('fasst Schreibweisen zusammen, wie es die Suche tut', () {
|
||||
// Genau der Bruch, der "0 Titel" und einen leeren Bildschirm ergab:
|
||||
// die Suche nannte einen Künstler, die Zuordnung fand ihn nicht.
|
||||
final gebuendelt = kuenstlerNachSchluessel([
|
||||
_song('A', kuenstler: 'Neoni'),
|
||||
_song('B', kuenstler: 'NEONI'),
|
||||
_song('C', kuenstler: 'Neoni '),
|
||||
]);
|
||||
|
||||
expect(gebuendelt.keys, ['neoni']);
|
||||
expect(gebuendelt['neoni']!.length, 3);
|
||||
});
|
||||
|
||||
test('der Treffer der Suche findet seine Titel wieder', () {
|
||||
final songs = [_song('A', kuenstler: 'Neoni '), _song('B', kuenstler: 'NEONI')];
|
||||
final treffer = sucheLokal(
|
||||
songs: songs, kategorien: const [], playlisten: const [], query: 'neoni');
|
||||
final gebuendelt = kuenstlerNachSchluessel(songs);
|
||||
|
||||
final name = treffer.kuenstler.single;
|
||||
expect(gebuendelt[suchSchluessel(name)]!.length, 2);
|
||||
});
|
||||
|
||||
test('Titel ohne Künstler kommen nicht vor', () {
|
||||
expect(kuenstlerNachSchluessel([_song('A'), _song('B', kuenstler: ' ')]),
|
||||
isEmpty);
|
||||
});
|
||||
});
|
||||
|
||||
group('kategorienNachSchluessel', () {
|
||||
test('fasst Schreibweisen zusammen', () {
|
||||
final a = _song('A', id: 'a');
|
||||
final b = _song('B', id: 'b');
|
||||
final gebuendelt = kategorienNachSchluessel(
|
||||
[a, b],
|
||||
(s) => s.id == 'a' ? ['Rock'] : ['rock'],
|
||||
);
|
||||
|
||||
expect(gebuendelt.keys, ['rock']);
|
||||
expect(gebuendelt['rock']!.length, 2);
|
||||
});
|
||||
|
||||
test('ein Titel steht unter jeder seiner Kategorien', () {
|
||||
final gebuendelt = kategorienNachSchluessel(
|
||||
[_song('A')],
|
||||
(_) => ['Rock', 'Chill'],
|
||||
);
|
||||
|
||||
expect(gebuendelt.keys.toSet(), {'rock', 'chill'});
|
||||
});
|
||||
|
||||
test('leere Kategorienamen werden übergangen', () {
|
||||
expect(kategorienNachSchluessel([_song('A')], (_) => [' ']), isEmpty);
|
||||
});
|
||||
});
|
||||
|
||||
group('suchSchluessel', () {
|
||||
test('trimmt und senkt die Schreibweise', () {
|
||||
expect(suchSchluessel(' NEONI '), 'neoni');
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -32,15 +32,6 @@ void main() {
|
||||
expect(songs.first.title, 'Neu');
|
||||
});
|
||||
|
||||
test('searchSongs findet über Titel und Künstler', () async {
|
||||
await db.upsertSongs([
|
||||
song('1', '/a.mp3', 'Nachtpuls', artist: 'Rotklang'),
|
||||
song('2', '/b.mp3', 'Taglicht', artist: 'Beatzwei'),
|
||||
]);
|
||||
expect((await db.searchSongs('nacht').first).length, 1);
|
||||
expect((await db.searchSongs('rotklang').first).length, 1);
|
||||
expect((await db.searchSongs('xyz').first).length, 0);
|
||||
});
|
||||
|
||||
test('markMissing setzt Tombstone für nicht aktualisierte Dateien', () async {
|
||||
final now1 = 100;
|
||||
|
||||
@@ -1,3 +1,4 @@
|
||||
import 'package:drift/drift.dart' show driftRuntimeOptions;
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
@@ -10,6 +11,10 @@ import 'package:melo/shared/favorite_button.dart';
|
||||
/// Früher kam an dieser Stelle der leere String an: das Herz füllte sich,
|
||||
/// in den Favoriten (INNER JOIN auf songs) tauchte aber nie etwas auf.
|
||||
void main() {
|
||||
// Jeder Test legt eine eigene In-Memory-Datenbank an; die Warnung über
|
||||
// mehrere Instanzen ist hier erwartet und kein Befund.
|
||||
setUp(() => driftRuntimeOptions.dontWarnAboutMultipleDatabases = true);
|
||||
|
||||
Future<void> pumpe(WidgetTester tester, MeloDb db, String songId) {
|
||||
return tester.pumpWidget(
|
||||
MultiProvider(
|
||||
|
||||
Reference in New Issue
Block a user