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
This commit is contained in:
Hermes (Server)
co-authored by
Claude Opus 5
Claude Opus 5
parent
34d6d8e52a
commit
e6d5f1c9ee
@@ -0,0 +1,37 @@
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/library/permissions.dart';
|
||||
|
||||
/// Ohne POST_NOTIFICATIONS zeigt Android 13+ die Wiedergabe-Benachrichtigung
|
||||
/// nicht an — und damit fehlt die wichtigste Bedienfläche einer Musik-App.
|
||||
/// Vor Android 13 gab es das Recht noch nicht; dort darf nicht gefragt werden.
|
||||
void main() {
|
||||
group('brauchtBenachrichtigungsRecht', () {
|
||||
test('Android 13 (API 33) braucht das Recht', () {
|
||||
expect(
|
||||
brauchtBenachrichtigungsRecht(istAndroid: true, sdkInt: 33),
|
||||
isTrue,
|
||||
);
|
||||
});
|
||||
|
||||
test('Android 14 (API 34) braucht das Recht', () {
|
||||
expect(
|
||||
brauchtBenachrichtigungsRecht(istAndroid: true, sdkInt: 34),
|
||||
isTrue,
|
||||
);
|
||||
});
|
||||
|
||||
test('Android 12 (API 32) fragt nicht — das Recht gibt es dort nicht', () {
|
||||
expect(
|
||||
brauchtBenachrichtigungsRecht(istAndroid: true, sdkInt: 32),
|
||||
isFalse,
|
||||
);
|
||||
});
|
||||
|
||||
test('außerhalb von Android wird nie gefragt', () {
|
||||
expect(
|
||||
brauchtBenachrichtigungsRecht(istAndroid: false, sdkInt: 33),
|
||||
isFalse,
|
||||
);
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,60 @@
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/library/song_media.dart';
|
||||
import 'package:melo/player/audio_handler.dart';
|
||||
import 'package:melo/services/navidrome_service.dart';
|
||||
|
||||
/// Server-Titel wurden bisher an zwei Stellen von Hand zu [MediaItem]s
|
||||
/// gebaut — beide ohne `artUri`. Auf Sperrbildschirm, in der Liste und im
|
||||
/// Player blieb deshalb der Noten-Platzhalter stehen, obwohl der Server das
|
||||
/// Cover unter `getCoverArt.view` bereithält.
|
||||
void main() {
|
||||
late NavidromeService nav;
|
||||
|
||||
setUp(() {
|
||||
nav = NavidromeService()
|
||||
..setCredentials('https://musik.example', 'dustin', 'geheim');
|
||||
});
|
||||
|
||||
const mitCover = SubsonicSong(
|
||||
id: 'song-42',
|
||||
titel: 'Roses',
|
||||
kuenstler: 'Neoni',
|
||||
album: 'Trailerpark',
|
||||
dauerSekunden: 221,
|
||||
coverId: 'al-7',
|
||||
);
|
||||
|
||||
test('das Cover kommt als Server-Adresse mit', () {
|
||||
final item = subsonicToMediaItem(mitCover, nav);
|
||||
|
||||
expect(item.artUri, isNotNull);
|
||||
expect(item.artUri!.scheme, 'https');
|
||||
expect(item.artUri!.path, contains('getCoverArt.view'));
|
||||
expect(item.artUri!.queryParameters['id'], 'al-7');
|
||||
});
|
||||
|
||||
test('ohne coverArt-Tag bleibt die Adresse leer statt zu raten', () {
|
||||
const ohneCover = SubsonicSong(id: 'song-43', titel: 'Ohne Bild');
|
||||
|
||||
expect(subsonicToMediaItem(ohneCover, nav).artUri, isNull);
|
||||
});
|
||||
|
||||
test('Titel, Künstler, Album und Dauer werden übernommen', () {
|
||||
final item = subsonicToMediaItem(mitCover, nav);
|
||||
|
||||
expect(item.title, 'Roses');
|
||||
expect(item.artist, 'Neoni');
|
||||
expect(item.album, 'Trailerpark');
|
||||
expect(item.duration, const Duration(seconds: 221));
|
||||
});
|
||||
|
||||
test('die Server-ID bleibt getrennt von der Abspiel-Adresse', () {
|
||||
// Sonst fragt die App den Server nach einem Titel namens "https://…"
|
||||
// — die Wurzel-Ursache von "(0) Source error".
|
||||
final item = subsonicToMediaItem(mitCover, nav);
|
||||
|
||||
expect(navidromeIdOf(item), 'song-42');
|
||||
expect(item.id, contains('stream.view'));
|
||||
expect(songIdOf(item), isNull);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,82 @@
|
||||
import 'package:audio_service/audio_service.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/library/database.dart';
|
||||
import 'package:melo/library/song_media.dart';
|
||||
import 'package:melo/player/audio_handler.dart';
|
||||
|
||||
Song _lokalerSong() => Song(
|
||||
id: 'uuid-1',
|
||||
path: '/storage/emulated/0/Music/Nightcore.mp3',
|
||||
title: 'No Friends',
|
||||
dateAddedMs: 0,
|
||||
updatedAtMs: 0,
|
||||
deleted: false,
|
||||
playCount: 0,
|
||||
categoriesEdited: false,
|
||||
metadataEdited: false,
|
||||
);
|
||||
|
||||
const _serverTitel = MediaItem(
|
||||
id: 'https://musik.example/rest/stream.view?id=abc123',
|
||||
title: 'Serverlied',
|
||||
extras: {'navidromeId': 'abc123'},
|
||||
);
|
||||
|
||||
/// Der Offline-Schalter in den Einstellungen war ein reines Anzeige-Element:
|
||||
/// gespeichert, aber von niemandem gelesen. Die App streamte weiter und
|
||||
/// verbrauchte Mobilfunkdaten, obwohl der Nutzer das Gegenteil eingestellt
|
||||
/// hatte.
|
||||
void main() {
|
||||
group('istOfflineVerfuegbar', () {
|
||||
test('ein Titel vom Gerät liegt immer als Datei vor', () {
|
||||
expect(
|
||||
istOfflineVerfuegbar(songToMediaItem(_lokalerSong()),
|
||||
cacheTreffer: false),
|
||||
isTrue,
|
||||
);
|
||||
});
|
||||
|
||||
test('ein Server-Titel mit Cache-Treffer läuft ohne Netz', () {
|
||||
expect(istOfflineVerfuegbar(_serverTitel, cacheTreffer: true), isTrue);
|
||||
});
|
||||
|
||||
test('ein Server-Titel ohne Cache-Treffer bräuchte das Netz', () {
|
||||
expect(istOfflineVerfuegbar(_serverTitel, cacheTreffer: false), isFalse);
|
||||
});
|
||||
});
|
||||
|
||||
group('startIndexNachFilter', () {
|
||||
test('bleibt stehen, wenn nichts vor dem Titel wegfällt', () {
|
||||
expect(startIndexNachFilter([true, true, true], 2), 2);
|
||||
});
|
||||
|
||||
test('rückt vor, wenn Titel davor wegfallen', () {
|
||||
// [x _ x X] → behalten sind 0,2,3; der gewählte Titel 3 ist jetzt 2.
|
||||
expect(startIndexNachFilter([true, false, true, true], 3), 2);
|
||||
});
|
||||
|
||||
test('fällt der gewählte Titel weg, rückt der nächste behaltene nach', () {
|
||||
// [x _ x] → der gewählte Titel 1 ist weg; an seine Stelle rückt Titel 2,
|
||||
// der in der gefilterten Liste den Index 1 hat.
|
||||
expect(startIndexNachFilter([true, false, true], 1), 1);
|
||||
});
|
||||
|
||||
test('fällt der letzte Titel weg, bleibt der davor', () {
|
||||
expect(startIndexNachFilter([true, true, false], 2), 1);
|
||||
});
|
||||
|
||||
test('bleibt bei leerer Auswahl bei 0', () {
|
||||
expect(startIndexNachFilter([false, false], 1), 0);
|
||||
});
|
||||
});
|
||||
|
||||
group('offlineHinweis', () {
|
||||
test('nennt einen einzelnen Titel', () {
|
||||
expect(offlineHinweis(1), contains('1 Titel übersprungen'));
|
||||
});
|
||||
|
||||
test('nennt die Anzahl bei mehreren Titeln', () {
|
||||
expect(offlineHinweis(7), contains('7 Titel übersprungen'));
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,97 @@
|
||||
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);
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -1,85 +0,0 @@
|
||||
import 'package:audio_service/audio_service.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/player/queue_screen.dart';
|
||||
|
||||
MediaItem _item(String id) => MediaItem(id: id, title: 'Titel $id');
|
||||
|
||||
void main() {
|
||||
group('removeFromQueue', () {
|
||||
test('entfernt den Titel am angegebenen Index und verschiebt den Rest', () {
|
||||
final queue = [_item('a'), _item('b'), _item('c')];
|
||||
|
||||
final result = removeFromQueue(queue, 1);
|
||||
|
||||
expect(result.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')];
|
||||
|
||||
final result = reorderQueue(queue, 0, 2);
|
||||
|
||||
expect(result.map((e) => e.id), ['b', 'c', 'a']);
|
||||
});
|
||||
|
||||
test('verschiebt einen Titel nach vorne', () {
|
||||
final queue = [_item('a'), _item('b'), _item('c')];
|
||||
|
||||
final result = reorderQueue(queue, 2, 0);
|
||||
|
||||
expect(result.map((e) => e.id), ['c', 'a', 'b']);
|
||||
});
|
||||
});
|
||||
|
||||
group('newPlayingIndex', () {
|
||||
test('findet den abgespielten Titel an seiner neuen Position', () {
|
||||
final newQueue = [_item('b'), _item('c'), _item('a')];
|
||||
|
||||
final index = newPlayingIndex(newQueue, 'a', 0);
|
||||
|
||||
expect(index, 2);
|
||||
});
|
||||
|
||||
test('fällt auf fallbackIndex zurück, wenn der Titel entfernt wurde', () {
|
||||
// 'b' wurde entfernt (war der abgespielte Titel); an Index 1 steht jetzt 'c'.
|
||||
final newQueue = [_item('a'), _item('c')];
|
||||
|
||||
final index = newPlayingIndex(newQueue, 'b', 1);
|
||||
|
||||
expect(index, 1);
|
||||
});
|
||||
|
||||
test('begrenzt fallbackIndex auf die neue Listenlänge', () {
|
||||
// Letzter Titel der Liste wurde entfernt, fallbackIndex zeigt hinter das Ende.
|
||||
final newQueue = [_item('a')];
|
||||
|
||||
final index = newPlayingIndex(newQueue, 'b', 1);
|
||||
|
||||
expect(index, 0);
|
||||
});
|
||||
|
||||
test('gibt 0 zurück, wenn die neue Queue leer ist', () {
|
||||
final index = newPlayingIndex(const [], 'a', 0);
|
||||
|
||||
expect(index, 0);
|
||||
});
|
||||
|
||||
test('nutzt fallbackIndex, wenn playingId null ist', () {
|
||||
final newQueue = [_item('a'), _item('b')];
|
||||
|
||||
final index = newPlayingIndex(newQueue, null, 1);
|
||||
|
||||
expect(index, 1);
|
||||
});
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,49 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/shared/cover.dart';
|
||||
|
||||
/// Server-Titel trugen nie ein Coverbild: [CoverImage] kannte nur `file://`
|
||||
/// und zeigte für alles andere den Noten-Platzhalter.
|
||||
void main() {
|
||||
Future<void> pumpe(WidgetTester tester, Uri? artUri) => tester.pumpWidget(
|
||||
MaterialApp(
|
||||
home: Scaffold(
|
||||
body: Center(child: CoverImage(artUri: artUri, size: 48)),
|
||||
),
|
||||
),
|
||||
);
|
||||
|
||||
testWidgets('ohne Adresse bleibt der Noten-Platzhalter', (tester) async {
|
||||
await pumpe(tester, null);
|
||||
|
||||
expect(find.byIcon(Icons.music_note), findsOneWidget);
|
||||
expect(find.byType(Image), findsNothing);
|
||||
});
|
||||
|
||||
testWidgets('eine Datei-Adresse wird als Bild geladen', (tester) async {
|
||||
await pumpe(tester, Uri.file('/tmp/gibtesnicht.jpg'));
|
||||
|
||||
expect(find.byType(Image), findsOneWidget);
|
||||
});
|
||||
|
||||
testWidgets('eine http-Adresse wird als Bild geladen', (tester) async {
|
||||
await pumpe(tester, Uri.parse('http://musik.example/getCoverArt.view?id=1'));
|
||||
|
||||
expect(find.byType(Image), findsOneWidget);
|
||||
});
|
||||
|
||||
testWidgets('eine https-Adresse wird als Bild geladen', (tester) async {
|
||||
await pumpe(
|
||||
tester, Uri.parse('https://musik.example/getCoverArt.view?id=1'));
|
||||
|
||||
expect(find.byType(Image), findsOneWidget);
|
||||
});
|
||||
|
||||
testWidgets('ein unbekanntes Schema fällt auf den Platzhalter zurück',
|
||||
(tester) async {
|
||||
await pumpe(tester, Uri.parse('content://media/external/audio/42'));
|
||||
|
||||
expect(find.byIcon(Icons.music_note), findsOneWidget);
|
||||
expect(find.byType(Image), findsNothing);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,72 @@
|
||||
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() {
|
||||
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));
|
||||
});
|
||||
}
|
||||
Reference in New Issue
Block a user