Hermes (Server)andClaude Opus 5
e6d5f1c9ee
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
61 lines
2.0 KiB
Dart
61 lines
2.0 KiB
Dart
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);
|
|
});
|
|
}
|