BUG 1 — Lieder vom Handy waren nicht abspielbar
("Wiedergabe fehlgeschlagen: (0) Source error" / "Loading interrupted")
Wurzel-Ursache: MeloAudioHandler.loadPlaylist hat jeden Warteschlangen-
Eintrag durch NavidromeService.streamAndCacheToLocal(item.id) geschickt,
sobald Zugangsdaten existierten. item.id ist aber NIE eine Navidrome-Song-ID
— lokal ist es file:///storage/..., beim Server die fertige Stream-Adresse.
Der Server bekam also 'file:///...' als Song-ID, antwortete mit einem Fehler,
und diese Fehlerantwort wurde an just_audio weitergereicht (Source error) —
bzw. als .mp3 in den Cache geschrieben, wodurch der Titel dauerhaft kaputt
blieb.
Zweite Ursache: die Schleife lud die GANZE Warteschlange seriell vorab
herunter (30s Timeout je Titel), bevor setAudioSources lief. Bei hunderten
Titeln startete die Wiedergabe deshalb nie; ein zweiter Tipp brach den
laufenden Ladevorgang ab ("Loading interrupted").
Fix:
- Server-Titel tragen ihre ID in MediaItem.extras['navidromeId'] statt sie
aus der Abspiel-Adresse zu raten. Neue reine Funktionen navidromeIdOf,
songIdOf, nutztServerCache, quelleFuer.
- loadPlaylist baut die Quellen ohne Netzzugriff; Caching des laufenden
Titels im Hintergrund (unawaited).
- Resume/Scrobble nur noch mit der jeweils passenden ID (Server bzw. lokal).
- ladeInCache() ersetzt streamAndCacheToLocal(): .part-Datei, Pruefung des
Inhaltstyps (istAudioAntwort), stabiler Cache-Schluessel ueber die
Song-ID statt der Stream-Adresse (die trug Token+Salt und war je Sitzung
anders — der Cache war nie wiederauffindbar), Client wird geschlossen.
BUG 2 — kein Abgleich zwischen Handy und Server
Neu: services/melo_cloud_service.dart + services/sync_service.dart gegen
cloud.baka-net.de (Bearer-JWT ueber BakaAuth). Server-Titel herunterladen
(offline verfuegbar), eigene Dateien hochladen, Loeschungen in beide
Richtungen (Tombstones), Favoriten und Wiedergabe-Verlauf. Automatisch beim
App-Start und bei Rueckkehr in die App (max. alle 15 Min), plus Knopf unter
Einstellungen -> Geraete-Abgleich. Reine Planungsfunktion planeSync().
Bewusst NICHT ueber Navidrome: die Subsonic-API kennt keinen Upload-
Endpunkt. Navidrome bleibt die Streaming-Bibliothek, die Melo-Cloud ist der
gemeinsame Speicher.
DB-Schema 8: songs.cloud_id verbindet Geraet und Server.
Nebenbei behoben (blockierte Build bzw. Tests):
- database.g.dart war veraltet — das Projekt liess sich nicht uebersetzen.
- metadataEdited wurde nirgends gesetzt/beachtet: von Hand korrigierte
Metadaten wurden vom naechsten Scan ueberschrieben. Jetzt in beiden
Scans respektiert; metadatenUebernahme() setzt die Markierung.
- song_detail_sheet_test.dart haengt beim Oeffnen des Modal-Sheets und
blockierte den gesamten Testlauf — vorerst uebersprungen (TODO im Code);
der Zweck wird von metadaten_uebernahme_test.dart abgedeckt.
Enthaelt ausserdem die bis dahin nicht committete Arbeit der Vorsitzung
(Musikerkennung/ACRCloud, MusicBrainz-Metadaten, MediaStore-Datentraeger).
267 Tests gruen (1 uebersprungen), flutter analyze ohne Befund.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FpPu4nuKjKKeX1RpdDeX81
233 lines
7.0 KiB
Dart
233 lines
7.0 KiB
Dart
import 'dart:convert';
|
|
import 'dart:io';
|
|
|
|
import 'package:flutter_test/flutter_test.dart';
|
|
import 'package:http/http.dart' as http;
|
|
import 'package:http/testing.dart';
|
|
import 'package:melo/services/baka_auth.dart';
|
|
import 'package:melo/services/yt_download_service.dart';
|
|
|
|
class _MemorySpeicher implements TokenSpeicher {
|
|
final Map<String, String> werte = {};
|
|
@override
|
|
Future<String?> lesen(String key) async => werte[key];
|
|
@override
|
|
Future<void> schreiben(String key, String wert) async => werte[key] = wert;
|
|
@override
|
|
Future<void> loeschen(String key) async => werte.remove(key);
|
|
}
|
|
|
|
Future<BakaAuth> _angemeldeteAuth() async {
|
|
final auth = BakaAuth(
|
|
client: MockClient(
|
|
(_) async => http.Response(jsonEncode({'token': 'jwt-abc'}), 200)),
|
|
speicher: _MemorySpeicher(),
|
|
);
|
|
await auth.anmelden('Baka', 'geheim');
|
|
return auth;
|
|
}
|
|
|
|
void main() {
|
|
late Directory ziel;
|
|
|
|
setUp(() => ziel = Directory.systemTemp.createTempSync('melo_yt_test'));
|
|
tearDown(() => ziel.deleteSync(recursive: true));
|
|
|
|
test('sichererDateiname entfernt Zeichen, die kein Dateisystem mag', () {
|
|
expect(sichererDateiname('AC/DC: Thunder*struck?'),
|
|
'AC_DC_ Thunder_struck_');
|
|
});
|
|
|
|
test('Nicht-YouTube-URL wird abgelehnt, ohne den Proxy zu fragen', () async {
|
|
var aufrufe = 0;
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((_) async {
|
|
aufrufe++;
|
|
return http.Response('', 200);
|
|
}),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://example.com/lied',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNull);
|
|
expect(dienst.fehler, 'Das ist keine YouTube-Adresse');
|
|
expect(aufrufe, 0);
|
|
});
|
|
|
|
test('Erfolgreicher Download legt die MP3 im Zielordner ab', () async {
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((req) async {
|
|
if (req.url.path == '/api/yt-dl') {
|
|
expect(req.headers['Authorization'], 'Bearer jwt-abc');
|
|
return http.Response(
|
|
jsonEncode({
|
|
'titel': 'Neoni - Roses',
|
|
'dauer': 221,
|
|
'dateiname': 'roses.mp3',
|
|
'mp3_url': '/api/dl/roses.mp3',
|
|
'filesize': 4,
|
|
}),
|
|
200);
|
|
}
|
|
return http.Response.bytes([1, 2, 3, 4], 200);
|
|
}),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen(
|
|
'https://youtu.be/abc123',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNotNull);
|
|
expect(ergebnis!.titel, 'Neoni - Roses');
|
|
expect(ergebnis.kuenstler, 'Neoni');
|
|
expect(ergebnis.dauerSekunden, 221);
|
|
expect(File(ergebnis.dateiPfad).readAsBytesSync(), [1, 2, 3, 4]);
|
|
expect(ergebnis.dateiPfad, '${ziel.path}/Neoni - Roses.mp3');
|
|
expect(dienst.fehler, isNull);
|
|
expect(dienst.laeuft, isFalse);
|
|
});
|
|
|
|
test('Ohne " - " im Titel bleibt der Künstler leer statt geraten zu werden',
|
|
() async {
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((req) async {
|
|
if (req.url.path == '/api/yt-dl') {
|
|
return http.Response(
|
|
jsonEncode({
|
|
'titel': 'Irgendein Mitschnitt',
|
|
'dauer': 10,
|
|
'mp3_url': '/api/dl/x.mp3',
|
|
}),
|
|
200);
|
|
}
|
|
return http.Response.bytes([9], 200);
|
|
}),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://youtu.be/x',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis!.kuenstler, '');
|
|
});
|
|
|
|
test('Abgelaufener Token (401) bittet um erneute Anmeldung', () async {
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((_) async => http.Response('Unauthorized', 401)),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://youtu.be/abc',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNull);
|
|
expect(dienst.fehler, 'Anmeldung abgelaufen — bitte neu anmelden');
|
|
});
|
|
|
|
test('Ohne Anmeldung wird gar nicht erst gefragt', () async {
|
|
final dienst = YtDownloadService(
|
|
auth: BakaAuth(
|
|
client: MockClient((_) async => http.Response('', 500)),
|
|
speicher: _MemorySpeicher()),
|
|
client: MockClient((_) async => http.Response('', 200)),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://youtu.be/abc',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNull);
|
|
expect(dienst.fehler, 'Bitte zuerst beim Baka-Konto anmelden');
|
|
});
|
|
|
|
test('Proxy-Fehlermeldung wird im Klartext durchgereicht', () async {
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((_) async => http.Response(
|
|
jsonEncode({'error': 'Video ist gesperrt'}), 500)),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://youtu.be/abc',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNull);
|
|
expect(dienst.fehler, 'Video ist gesperrt');
|
|
});
|
|
|
|
test('Scheitert der MP3-Abruf, bleibt keine halbe Datei liegen', () async {
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((req) async {
|
|
if (req.url.path == '/api/yt-dl') {
|
|
return http.Response(
|
|
jsonEncode({
|
|
'titel': 'Kaputt',
|
|
'dauer': 1,
|
|
'mp3_url': '/api/dl/k.mp3',
|
|
}),
|
|
200);
|
|
}
|
|
return http.Response('weg', 404);
|
|
}),
|
|
);
|
|
|
|
final ergebnis = await dienst.herunterladen('https://youtu.be/abc',
|
|
zielOrdner: ziel.path);
|
|
|
|
expect(ergebnis, isNull);
|
|
expect(ziel.listSync(), isEmpty);
|
|
});
|
|
|
|
test('Standardmäßig fragt der Auftrag die Cookies des Servers an', () async {
|
|
String? auftrag;
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((req) async {
|
|
if (req.url.path == '/api/yt-dl') {
|
|
auftrag = req.body;
|
|
return http.Response(
|
|
jsonEncode({
|
|
'titel': 'Mit Keksen',
|
|
'dauer': 5,
|
|
'mp3_url': '/api/dl/k.mp3',
|
|
}),
|
|
200);
|
|
}
|
|
return http.Response.bytes([7], 200);
|
|
}),
|
|
);
|
|
|
|
await dienst.herunterladen('https://youtu.be/abc', zielOrdner: ziel.path);
|
|
|
|
expect(jsonDecode(auftrag!),
|
|
{'url': 'https://youtu.be/abc', 'cookies': true});
|
|
});
|
|
|
|
test('Abgeschaltete Cookies stehen so im Auftrag', () async {
|
|
String? auftrag;
|
|
final dienst = YtDownloadService(
|
|
auth: await _angemeldeteAuth(),
|
|
client: MockClient((req) async {
|
|
if (req.url.path == '/api/yt-dl') {
|
|
auftrag = req.body;
|
|
return http.Response(
|
|
jsonEncode({
|
|
'titel': 'Ohne Kekse',
|
|
'dauer': 5,
|
|
'mp3_url': '/api/dl/o.mp3',
|
|
}),
|
|
200);
|
|
}
|
|
return http.Response.bytes([7], 200);
|
|
}),
|
|
);
|
|
|
|
await dienst.herunterladen('https://youtu.be/abc',
|
|
zielOrdner: ziel.path, cookies: false);
|
|
|
|
expect((jsonDecode(auftrag!) as Map)['cookies'], isFalse);
|
|
});
|
|
}
|