Add mission/drone profile management with autosave and persistence
Vervollstaendigt Doku 3.5/3.6/7.1/7.3/7.5: ein Vollbild-Verwaltungsmenue fuer Missionen und Drohnenprofile (per Tabs umschaltbar, wie im HTML- Demonstrator #missionsPanel gemeinsam mit der Profilliste), Autosave der aktuell bearbeiteten Mission, Drift-Persistenz statt der bisherigen In-Memory-only currentMissionProvider, sowie Anzeige/Zugang ueber neue Chips in der Fusszeile (HTML-Demonstrator: #missionNameBtn/#droneNameBtn). Datenbank (services/database/): Missions-Tabelle um updatedAt ergaenzt, neue DroneProfiles- und AppSettingsTable-Tabellen (schemaVersion 2 mit onUpgrade-Migration). Drei Repositories kapseln Drift-Zugriff + JSON-(De-)Serialisierung der Wegpunktliste: MissionRepository, DroneProfileRepository, AppSettingsRepository - je mit Unit-Tests gegen eine In-Memory-Datenbank (NativeDatabase.memory()). Provider: CurrentMissionMetaNotifier haelt Name/id der aktuellen Mission und autosaved sie 800ms-debounced (identisch zum HTML-Demonstrator: scheduleAutosave()/flushPendingAutosave()) - inklusive Wiederherstellung des zuletzt bearbeiteten Autosave-Drafts beim App-Start (Doku 7.5). ActiveDroneProfileNotifier haelt das aktive Profil, seedet beim ersten Start automatisch T1 Ranger und merkt sich die Auswahl ueber AppSettingsTable neustart-fest. UI: MissionsDronesPanel (Missions-Tab: Liste mit Umbenennen/Loeschen/ Laden; Drones-Tab: Liste mit Bearbeiten/Loeschen/Auswaehlen, faellt beim Loeschen des aktiven Profils auf das naechste zurueck). DroneProfileEditor als Formular (bewusste Vereinfachung gegenueber den Swipe-Gesten-Chips des HTML-Demonstrators - bei elf Feldern ist ein normales Formular auf einem Mobilgeraet zugaenglicher). Rename per AlertDialog+TextField statt Browser-prompt(). Aktives Drohnenprofil ist jetzt tatsaechlich wirksam statt nur eine feste Anzeige: Speed-/Alt-Drehrad-Grenzen, Fangradius neuer Wegpunkte und die Flugpfad-Machbarkeitspruefung (Kurvenradius, Steig-/Sinkrate) in PlanScreen und WaypointListPanel lesen jetzt activeDroneProfileProvider statt der bisherigen statischen DefaultDroneProfile-Konstanten (die als T1-Ranger-Seed-Werte weiterleben). Beim Verifizieren zwei echte Bugs in BottomStatsBar gefunden und gefixt: ein Stack+Align-ohne-Factor blaehte sich auf unbegrenzte Groesse auf und verschob die Details-Pille aus ihrer sichtbaren Position (durch zwei gleich grosse Expanded-Bereiche ersetzt); zwei ConstrainedBox-Chips nebeneinander verursachten auf schmaleren Breiten einen RenderFlex- Overflow (durch Flexible ersetzt). Widget-Tests: AppShell/PlanScreen initialisieren beim Start jetzt die echte Datenbank - alle Tests, die AppShell pumpen, ueberschreiben appDatabaseProvider testweise mit einer In-Memory-Instanz. Ausserdem mussten mehrere Tests den neuen 800ms-Autosave-Timer abwarten (wie zuvor schon beim WaypointChip-Flash-Timer etabliert), damit nach Testende kein Timer mehr aussteht. Verifiziert: flutter analyze (0 issues), flutter test (59/59, davon 14 neue Repository-Tests), manuell auf Pixel_10a-Emulator - Wegpunkte werden automatisch als "New mission" gespeichert und erscheinen in der Liste, Umbenennen/Laden/Loeschen funktionieren, neues Drohnenprofil mit abweichender Max-Speed wird angelegt+aktiviert+in der Fusszeile angezeigt, Loeschen des aktiven Profils faellt auf das verbleibende zurueck. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 5
parent
3dd9c38fb1
commit
aaec7f0f72
@@ -0,0 +1,52 @@
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
import 'package:dmc_app/services/database/app_database.dart';
|
||||
import 'package:dmc_app/services/database/app_settings_repository.dart';
|
||||
|
||||
void main() {
|
||||
late AppDatabase db;
|
||||
late AppSettingsRepository repository;
|
||||
|
||||
setUp(() {
|
||||
db = AppDatabase.forTesting(NativeDatabase.memory());
|
||||
repository = AppSettingsRepository(db);
|
||||
});
|
||||
|
||||
tearDown(() => db.close());
|
||||
|
||||
test('load ohne vorherige Werte liefert beides null', () async {
|
||||
final settings = await repository.load();
|
||||
expect(settings.activeDroneProfileId, isNull);
|
||||
expect(settings.currentMissionId, isNull);
|
||||
});
|
||||
|
||||
test('setActiveDroneProfileId und setCurrentMissionId sind unabhaengig voneinander',
|
||||
() async {
|
||||
await repository.setActiveDroneProfileId(3);
|
||||
await repository.setCurrentMissionId(7);
|
||||
|
||||
final settings = await repository.load();
|
||||
expect(settings.activeDroneProfileId, 3);
|
||||
expect(settings.currentMissionId, 7);
|
||||
});
|
||||
|
||||
test('erneutes Setzen eines Werts laesst den anderen unangetastet', () async {
|
||||
await repository.setActiveDroneProfileId(1);
|
||||
await repository.setCurrentMissionId(2);
|
||||
|
||||
await repository.setCurrentMissionId(9);
|
||||
|
||||
final settings = await repository.load();
|
||||
expect(settings.activeDroneProfileId, 1);
|
||||
expect(settings.currentMissionId, 9);
|
||||
});
|
||||
|
||||
test('Wert kann auf null zurueckgesetzt werden', () async {
|
||||
await repository.setCurrentMissionId(5);
|
||||
await repository.setCurrentMissionId(null);
|
||||
|
||||
final settings = await repository.load();
|
||||
expect(settings.currentMissionId, isNull);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,67 @@
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
import 'package:dmc_app/domain/mission/drone_profile.dart';
|
||||
import 'package:dmc_app/services/database/app_database.dart' hide DroneProfile;
|
||||
import 'package:dmc_app/services/database/drone_profile_repository.dart';
|
||||
|
||||
void main() {
|
||||
late AppDatabase db;
|
||||
late DroneProfileRepository repository;
|
||||
|
||||
setUp(() {
|
||||
db = AppDatabase.forTesting(NativeDatabase.memory());
|
||||
repository = DroneProfileRepository(db);
|
||||
});
|
||||
|
||||
tearDown(() => db.close());
|
||||
|
||||
test('ensureSeeded legt beim ersten Aufruf T1 Ranger an', () async {
|
||||
final id = await repository.ensureSeeded();
|
||||
|
||||
final all = await repository.loadAll();
|
||||
expect(all, hasLength(1));
|
||||
expect(all.single.id, id);
|
||||
expect(all.single.name, 'T1 Ranger');
|
||||
expect(all.single.cruiseSpeed, 15);
|
||||
});
|
||||
|
||||
test('ensureSeeded ist ein no-op, wenn bereits Profile existieren', () async {
|
||||
await repository.insert(DroneProfile.t1Ranger().copyWith(name: 'Custom'));
|
||||
|
||||
await repository.ensureSeeded();
|
||||
|
||||
final all = await repository.loadAll();
|
||||
expect(all, hasLength(1));
|
||||
expect(all.single.name, 'Custom');
|
||||
});
|
||||
|
||||
test('insert/update/delete rundtrip', () async {
|
||||
final id = await repository.insert(
|
||||
DroneProfile.t1Ranger().copyWith(name: 'Racer', maxSpeed: 40),
|
||||
);
|
||||
|
||||
var all = await repository.loadAll();
|
||||
expect(all.single.maxSpeed, 40);
|
||||
|
||||
await repository.update(all.single.copyWith(maxSpeed: 45));
|
||||
all = await repository.loadAll();
|
||||
expect(all.single.maxSpeed, 45);
|
||||
expect(all.single.id, id);
|
||||
|
||||
await repository.delete(id);
|
||||
expect(await repository.loadAll(), isEmpty);
|
||||
});
|
||||
|
||||
test('watchAll gibt Aenderungen reaktiv weiter', () async {
|
||||
final stream = repository.watchAll();
|
||||
final emissions = <int>[];
|
||||
final sub = stream.listen((profiles) => emissions.add(profiles.length));
|
||||
addTearDown(sub.cancel);
|
||||
|
||||
await repository.insert(DroneProfile.t1Ranger());
|
||||
await pumpEventQueue();
|
||||
|
||||
expect(emissions, contains(1));
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,104 @@
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
import 'package:dmc_app/domain/waypoint/flat_waypoint_list.dart';
|
||||
import 'package:dmc_app/services/database/app_database.dart';
|
||||
import 'package:dmc_app/services/database/mission_repository.dart';
|
||||
|
||||
void main() {
|
||||
late AppDatabase db;
|
||||
late MissionRepository repository;
|
||||
|
||||
setUp(() {
|
||||
db = AppDatabase.forTesting(NativeDatabase.memory());
|
||||
repository = MissionRepository(db);
|
||||
});
|
||||
|
||||
tearDown(() => db.close());
|
||||
|
||||
const waypoints = [
|
||||
Waypoint(lat: 52.0, lon: 4.0, altitudeM: 60, speedMs: 15, catchRadiusM: 60),
|
||||
Waypoint(
|
||||
lat: 52.01,
|
||||
lon: 4.01,
|
||||
altitudeM: 80,
|
||||
speedMs: 18,
|
||||
catchRadiusM: 60,
|
||||
action: WaypointAction.loiter,
|
||||
),
|
||||
];
|
||||
|
||||
test('upsert ohne id legt eine neue Mission an', () async {
|
||||
final id = await repository.upsert(
|
||||
name: 'Amsterdam',
|
||||
placeName: 'Amsterdam',
|
||||
waypoints: waypoints,
|
||||
);
|
||||
|
||||
final loaded = await repository.load(id);
|
||||
expect(loaded, isNotNull);
|
||||
expect(loaded!.name, 'Amsterdam');
|
||||
expect(loaded.placeName, 'Amsterdam');
|
||||
expect(loaded.waypoints, hasLength(2));
|
||||
expect(loaded.waypoints[1].action, WaypointAction.loiter);
|
||||
expect(loaded.waypoints[1].speedMs, 18);
|
||||
});
|
||||
|
||||
test('upsert mit bestehender id ueberschreibt statt zu duplizieren', () async {
|
||||
final id = await repository.upsert(name: 'Draft', waypoints: waypoints);
|
||||
|
||||
await repository.upsert(
|
||||
id: id,
|
||||
name: 'Draft',
|
||||
waypoints: [waypoints.first],
|
||||
);
|
||||
|
||||
final summaries = await repository.watchSummaries().first;
|
||||
expect(summaries, hasLength(1));
|
||||
expect(summaries.single.waypointCount, 1);
|
||||
});
|
||||
|
||||
test('rename aendert nur den Namen, nicht die Wegpunkte', () async {
|
||||
final id = await repository.upsert(name: 'Old name', waypoints: waypoints);
|
||||
|
||||
await repository.rename(id, 'New name');
|
||||
|
||||
final loaded = await repository.load(id);
|
||||
expect(loaded!.name, 'New name');
|
||||
expect(loaded.waypoints, hasLength(2));
|
||||
});
|
||||
|
||||
test('delete entfernt die Mission aus der Liste', () async {
|
||||
final id = await repository.upsert(name: 'To delete', waypoints: waypoints);
|
||||
|
||||
await repository.delete(id);
|
||||
|
||||
final loaded = await repository.load(id);
|
||||
expect(loaded, isNull);
|
||||
expect(await repository.watchSummaries().first, isEmpty);
|
||||
});
|
||||
|
||||
test('load fuer unbekannte id liefert null', () async {
|
||||
expect(await repository.load(999), isNull);
|
||||
});
|
||||
|
||||
test('Windfelder werden mitgespeichert und wiederhergestellt', () async {
|
||||
const withWind = Waypoint(
|
||||
lat: 52.0,
|
||||
lon: 4.0,
|
||||
altitudeM: 60,
|
||||
speedMs: 15,
|
||||
catchRadiusM: 60,
|
||||
windSpeedMs: 5.5,
|
||||
windDirFromDeg: 210,
|
||||
windElevationM: 2,
|
||||
);
|
||||
final id = await repository.upsert(name: 'Windy', waypoints: [withWind]);
|
||||
|
||||
final loaded = await repository.load(id);
|
||||
|
||||
expect(loaded!.waypoints.single.windSpeedMs, 5.5);
|
||||
expect(loaded.waypoints.single.windDirFromDeg, 210);
|
||||
expect(loaded.waypoints.single.windElevationM, 2);
|
||||
});
|
||||
}
|
||||
@@ -1,3 +1,4 @@
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
import 'package:flutter_map/flutter_map.dart';
|
||||
@@ -6,8 +7,10 @@ import 'package:flutter_test/flutter_test.dart';
|
||||
|
||||
import 'package:dmc_app/app_mode/app_mode_cubit.dart';
|
||||
import 'package:dmc_app/domain/waypoint/flat_waypoint_list.dart';
|
||||
import 'package:dmc_app/services/database/app_database.dart' hide DroneProfile, Mission;
|
||||
import 'package:dmc_app/ui/app_shell.dart';
|
||||
import 'package:dmc_app/ui/providers/current_mission_provider.dart';
|
||||
import 'package:dmc_app/ui/providers/database_provider.dart';
|
||||
import 'package:dmc_app/ui/widgets/full_value_chart.dart';
|
||||
import 'package:dmc_app/ui/widgets/halo_menu.dart';
|
||||
import 'package:dmc_app/ui/widgets/header_wind_pill.dart';
|
||||
@@ -19,7 +22,18 @@ import 'package:dmc_app/ui/widgets/waypoint_list_panel.dart';
|
||||
import 'package:dmc_app/ui/widgets/wind_marker_pill.dart';
|
||||
import 'package:latlong2/latlong.dart';
|
||||
|
||||
// AppShell/PlanScreen initialisieren beim Start die Drift-Datenbank (Doku
|
||||
// 4.15/7.5: Drohnenprofil-Bootstrap, Autosave-Restore) - in Widget-Tests
|
||||
// durch eine In-Memory-Instanz ersetzt, damit dafuer nicht das
|
||||
// path_provider-Plugin (echtes Dateisystem) benoetigt wird.
|
||||
AppDatabase _testDatabase() => AppDatabase.forTesting(NativeDatabase.memory());
|
||||
|
||||
ProviderContainer _testContainer() => ProviderContainer(
|
||||
overrides: [appDatabaseProvider.overrideWithValue(_testDatabase())],
|
||||
);
|
||||
|
||||
Widget _wrap(Widget child) => ProviderScope(
|
||||
overrides: [appDatabaseProvider.overrideWithValue(_testDatabase())],
|
||||
child: BlocProvider(
|
||||
create: (_) => AppModeCubit(),
|
||||
child: MaterialApp(home: child),
|
||||
@@ -27,7 +41,7 @@ Widget _wrap(Widget child) => ProviderScope(
|
||||
);
|
||||
|
||||
Future<ProviderContainer> _pumpWithContainer(WidgetTester tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
await tester.pumpWidget(
|
||||
UncontrolledProviderScope(
|
||||
container: container,
|
||||
@@ -73,7 +87,7 @@ void main() {
|
||||
|
||||
testWidgets('Reticle-Button fuegt einen Wegpunkt am Kartenzentrum hinzu',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
await tester.pumpWidget(
|
||||
@@ -96,12 +110,16 @@ void main() {
|
||||
expect(waypoints, hasLength(1));
|
||||
expect(waypoints.single.lat, closeTo(52.3676, 0.0001));
|
||||
expect(waypoints.single.lon, closeTo(4.9041, 0.0001));
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets(
|
||||
'Zweiter Tap auf den frisch gesetzten Wegpunkt oeffnet das Halo-Menue',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
await tester.pumpWidget(
|
||||
@@ -124,11 +142,15 @@ void main() {
|
||||
await tester.tap(find.byType(ReticleButton));
|
||||
await tester.pump();
|
||||
expect(find.byType(HaloMenu), findsOneWidget);
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets('Halo-Remove entfernt den Wegpunkt und schliesst das Menue',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
await tester.pumpWidget(
|
||||
@@ -155,11 +177,15 @@ void main() {
|
||||
|
||||
expect(container.read(currentMissionProvider), isEmpty);
|
||||
expect(find.byType(HaloMenu), findsNothing);
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets('Halo-Loiter setzt die Execute-Aktion und schliesst das Menue',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
await tester.pumpWidget(
|
||||
@@ -184,6 +210,10 @@ void main() {
|
||||
final waypoints = container.read(currentMissionProvider);
|
||||
expect(waypoints.single.action, WaypointAction.loiter);
|
||||
expect(find.byType(HaloMenu), findsNothing);
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets('Bottom-Stats-Bar oeffnet die leere Wegpunktliste',
|
||||
@@ -219,6 +249,10 @@ void main() {
|
||||
await tester.pump(const Duration(milliseconds: 200));
|
||||
|
||||
expect(find.byType(WaypointListPanel), findsNothing);
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets('Papierkorb-Icon in der Liste entfernt den Wegpunkt',
|
||||
@@ -238,6 +272,10 @@ void main() {
|
||||
|
||||
expect(container.read(currentMissionProvider), isEmpty);
|
||||
expect(find.text('No waypoints yet'), findsOneWidget);
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets(
|
||||
@@ -267,6 +305,10 @@ void main() {
|
||||
|
||||
final waypoints = container.read(currentMissionProvider);
|
||||
expect(waypoints.single.altitudeM, greaterThan(60));
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets(
|
||||
@@ -298,6 +340,10 @@ void main() {
|
||||
final speed = container.read(currentMissionProvider).single.speedMs;
|
||||
expect(speed, greaterThan(startSpeed));
|
||||
expect(speed, lessThan(startSpeed + 4));
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets(
|
||||
@@ -321,6 +367,10 @@ void main() {
|
||||
|
||||
final waypoints = container.read(currentMissionProvider);
|
||||
expect(waypoints.single.altitudeM, greaterThan(60));
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets(
|
||||
@@ -344,6 +394,10 @@ void main() {
|
||||
|
||||
final waypoints = container.read(currentMissionProvider);
|
||||
expect(waypoints.single.speedMs, greaterThan(15));
|
||||
|
||||
// Autosave debounced ueber 800ms - abwarten, damit nach Testende kein
|
||||
// Timer mehr aussteht (Doku 7.5).
|
||||
await tester.pump(const Duration(milliseconds: 900));
|
||||
});
|
||||
|
||||
testWidgets('MapSearchControls zeigt Suchfeld und Home-/Fit-Buttons',
|
||||
@@ -391,7 +445,7 @@ void main() {
|
||||
testWidgets(
|
||||
'Kameraposition bleibt beim Wechsel Plan -> Fly -> Plan erhalten',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
final appModeCubit = AppModeCubit();
|
||||
addTearDown(() {
|
||||
container.dispose();
|
||||
@@ -434,7 +488,7 @@ void main() {
|
||||
testWidgets(
|
||||
'Wegpunktliste scrollt beim Oeffnen automatisch zum aktiven Wegpunkt',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
final notifier = container.read(currentMissionProvider.notifier);
|
||||
@@ -476,7 +530,7 @@ void main() {
|
||||
testWidgets(
|
||||
'Wind-Marker an Wegpunkten erscheinen erst nach Antippen der Kopfleisten-Pille',
|
||||
(tester) async {
|
||||
final container = ProviderContainer();
|
||||
final container = _testContainer();
|
||||
addTearDown(container.dispose);
|
||||
|
||||
// Am Kartenzentrum platziert (siehe MissionMap._initialCenter), damit
|
||||
|
||||
Reference in New Issue
Block a user