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
38 lines
1.1 KiB
Dart
38 lines
1.1 KiB
Dart
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,
|
|
);
|
|
});
|
|
});
|
|
}
|