Files
Melo/test/player/queue_mutation_test.dart
T
Hermes (Server)andClaude Opus 5 e6d5f1c9ee Fuenf P0-Fehler behoben: Benachrichtigung, Offline, Cover, Warteschlange
Ergebnis einer Durchsicht der App gegen den Massstab "Apple Music".
Alle fuenf Punkte waren keine fehlenden Funktionen, sondern Versprechen,
die die Oberflaeche gab und der Code nicht hielt.

- POST_NOTIFICATIONS fehlte im Manifest. Auf Android 13+ blieb die
  Wiedergabe-Benachrichtigung dadurch unsichtbar. Das Recht wird erst
  erfragt, wenn wirklich ein Titel laeuft.
- Der Offline-Schalter wurde nirgends gelesen. Er greift jetzt an allen
  drei Wegen in die Warteschlange; uebersprungene Titel werden benannt.
- CoverImage kannte nur file://, und der Server-Browser gab die
  Cover-Adresse gar nicht erst mit. Server-Titel waren deshalb immer ein
  graues Notensymbol.
- Warteschlange entfernen/umsortieren baute die ganze Queue neu auf:
  hoerbarer Aussetzer und doppelte Wiedergabe-Zaehlung. Jetzt mutiert nur
  der betroffene Eintrag, die Buchfuehrung steht vor dem Player.
- Das Herz bei Server-Titeln legte einen Favoriten mit leerer Kennung an,
  der nie in der Liste auftauchte. Es ist jetzt ausgegraut.

Nachgebessert aus dem Code-Review: Offline-Meldung an die Stelle geholt,
wo sie greifen kann; Queue-Buchfuehrung vor den Player gezogen;
Fehlerpfad fuer die Berechtigungs-Abfrage ergaenzt.

360 Tests gruen (vorher 328), flutter analyze ohne Befund, Release-APK
gebaut und POST_NOTIFICATIONS darin nachgeprueft.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013xAHJTJM6UUqmjUgk1PUEd
2026-08-23 18:34:26 +02:00

98 lines
3.3 KiB
Dart

import 'package:audio_service/audio_service.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:melo/player/audio_handler.dart';
MediaItem _item(String id) => MediaItem(id: id, title: 'Titel $id');
/// Entfernen und Umsortieren luden früher die ganze Warteschlange neu
/// (`loadPlaylist`). Das riss die Wiedergabe hörbar auf und setzte dabei
/// `_lastCountedIndex` zurück — der laufende Titel wurde jedes Mal erneut als
/// Wiedergabe gezählt und verfälschte die Sortierung "Wie oft abgespielt".
void main() {
group('removeFromQueue', () {
test('entfernt den Titel am angegebenen Index und verschiebt den Rest', () {
final queue = [_item('a'), _item('b'), _item('c')];
expect(removeFromQueue(queue, 1).map((e) => e.id), ['a', 'c']);
});
test('lässt die Original-Liste unverändert', () {
final queue = [_item('a'), _item('b')];
removeFromQueue(queue, 0);
expect(queue.map((e) => e.id), ['a', 'b']);
});
});
group('reorderQueue', () {
test('verschiebt einen Titel nach hinten', () {
final queue = [_item('a'), _item('b'), _item('c')];
expect(reorderQueue(queue, 0, 2).map((e) => e.id), ['b', 'c', 'a']);
});
test('verschiebt einen Titel nach vorne', () {
final queue = [_item('a'), _item('b'), _item('c')];
expect(reorderQueue(queue, 2, 0).map((e) => e.id), ['c', 'a', 'b']);
});
test('lässt die Original-Liste unverändert', () {
final queue = [_item('a'), _item('b')];
reorderQueue(queue, 0, 1);
expect(queue.map((e) => e.id), ['a', 'b']);
});
});
group('gezaehlterIndexNachEntfernen', () {
test('ein Titel vor dem gezählten rückt ihn eine Stelle nach vorn', () {
// Sonst zeigt der gemerkte Index nach dem Entfernen auf den Nachbarn
// und der laufende Titel wird ein zweites Mal gezählt.
expect(gezaehlterIndexNachEntfernen(3, 1), 2);
});
test('ein Titel hinter dem gezählten ändert nichts', () {
expect(gezaehlterIndexNachEntfernen(1, 3), 1);
});
test('wird der gezählte Titel selbst entfernt, gilt keiner mehr', () {
expect(gezaehlterIndexNachEntfernen(2, 2), isNull);
});
test('ohne gezählten Titel bleibt es dabei', () {
expect(gezaehlterIndexNachEntfernen(null, 0), isNull);
});
});
group('gezaehlterIndexNachVerschieben', () {
test('der gezählte Titel selbst wandert mit', () {
expect(gezaehlterIndexNachVerschieben(1, 1, 3), 3);
});
test('ein Titel von vorn nach hinten rückt den gezählten nach vorn', () {
// [a b C d] → b nach 3 → [a C d b]: C war 2, ist jetzt 1.
expect(gezaehlterIndexNachVerschieben(2, 1, 3), 1);
});
test('ein Titel von hinten nach vorn rückt den gezählten nach hinten', () {
// [a B c d] → d nach 1 → [a d B c]: B war 1, ist jetzt 2.
expect(gezaehlterIndexNachVerschieben(1, 3, 1), 2);
});
test('eine Verschiebung ganz hinter dem gezählten ändert nichts', () {
expect(gezaehlterIndexNachVerschieben(0, 2, 3), 0);
});
test('eine Verschiebung ganz vor dem gezählten ändert nichts', () {
expect(gezaehlterIndexNachVerschieben(3, 0, 1), 3);
});
test('ohne gezählten Titel bleibt es dabei', () {
expect(gezaehlterIndexNachVerschieben(null, 0, 1), isNull);
});
});
}