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
78 lines
2.6 KiB
Dart
78 lines
2.6 KiB
Dart
import 'package:drift/drift.dart' show 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:melo/library/database.dart';
|
|
import 'package:melo/library/playlist_service.dart';
|
|
import 'package:melo/shared/favorite_button.dart';
|
|
|
|
/// Ein Titel, der nur am Server liegt, hat keine UUID in der Bibliothek.
|
|
/// 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(
|
|
providers: [
|
|
Provider<MeloDb>.value(value: db),
|
|
ChangeNotifierProvider<PlaylistService>.value(
|
|
value: PlaylistService(db)),
|
|
],
|
|
child: MaterialApp(
|
|
home: Scaffold(body: FavoriteButton(songId: songId)),
|
|
),
|
|
),
|
|
);
|
|
}
|
|
|
|
testWidgets('ohne Song-ID ist das Herz nicht bedienbar', (tester) async {
|
|
final db = MeloDb(NativeDatabase.memory());
|
|
await pumpe(tester, db, '');
|
|
await tester.pumpAndSettle();
|
|
|
|
final knopf = tester.widget<IconButton>(find.byType(IconButton));
|
|
expect(knopf.onPressed, isNull);
|
|
|
|
await tester.pumpWidget(const SizedBox());
|
|
await tester.pump(const Duration(milliseconds: 1));
|
|
});
|
|
|
|
testWidgets('ohne Song-ID entsteht kein Favoriten-Eintrag', (tester) async {
|
|
final db = MeloDb(NativeDatabase.memory());
|
|
await pumpe(tester, db, '');
|
|
await tester.pumpAndSettle();
|
|
|
|
await tester.tap(find.byType(IconButton));
|
|
await tester.pumpAndSettle();
|
|
|
|
expect(await db.select(db.favorites).get(), isEmpty);
|
|
|
|
await tester.pumpWidget(const SizedBox());
|
|
await tester.pump(const Duration(milliseconds: 1));
|
|
});
|
|
|
|
testWidgets('mit Song-ID bleibt das Herz bedienbar', (tester) async {
|
|
final db = MeloDb(NativeDatabase.memory());
|
|
await db.into(db.songs).insert(SongsCompanion.insert(
|
|
id: 'song-1',
|
|
path: '/a.mp3',
|
|
title: 'A',
|
|
dateAddedMs: 0,
|
|
updatedAtMs: 0,
|
|
));
|
|
await pumpe(tester, db, 'song-1');
|
|
await tester.pumpAndSettle();
|
|
|
|
final knopf = tester.widget<IconButton>(find.byType(IconButton));
|
|
expect(knopf.onPressed, isNotNull);
|
|
|
|
await tester.pumpWidget(const SizedBox());
|
|
await tester.pump(const Duration(milliseconds: 1));
|
|
});
|
|
}
|