System messages fuer GPS-Fix und Drone connected/disconnected, Batterie-Schwellwerte auf Pro-Zelle umgestellt
- detectSystemMessages() erkennt jetzt auch den Uebergang zu einem GPS-Fix (flankengetriggert wie Battery/Failsafe), Meldung "GPS fix acquired (N satellites)". - Die automatisch erkannten Verbindungsmeldungen heissen jetzt "Drone connected"/"Drone disconnected" statt "Connected"/"Connection lost" - praeziser an die tatsaechliche Bedeutung angelehnt (Eintreffen/Ausbleiben echter Telemetrie-Frames, nicht nur des rohen Socket-Zustands). - DroneProfile: feste Pack-Alarmspannung (batteryVoltageGreenMinV/RedMinV) ersetzt durch batteryCellCount + Pro-Zelle-Schwellwerte (batteryVoltageGreenMinPerCellV/RedMinPerCellV, LiPo-Standardwerte 3.4/3.2 V als Default). Die alten Feldnamen bleiben als berechnete Getter (Zellenzahl * Pro-Zelle-Wert) erhalten, damit telemetry_field_status.dart unveraendert bleibt. T1-Ranger-Standardprofil auf 4S gesetzt. - DB-Schema v8 -> v9 (additiv, alte Pack-Spannungs-Spalten bleiben als tote Spalten stehen), Repository und Share-Codec-Im-/Export entsprechend angepasst; alte Exportdateien ohne die neuen Felder fallen auf die DroneProfile-Defaults zurueck statt eine unbekannte Zellenzahl zu raten. - DroneProfileEditor: "Battery voltage alarm"-Gruppe um ein Zellenzahl-Feld erweitert und auf V/Zelle umbenannt. - Tests ergaenzt/angepasst: GPS-Fix-Erkennung (Unit + Provider-Integration), neuer v8->v9-Migrationstest analog zum bestehenden v7->v8-Test, Connected/Disconnected-Umbenennung in Provider- und Widget-Tests, Cell-Count-Roundtrip in Repository- und Share-Codec-Tests. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Constantin Leue
co-authored by
Claude Sonnet 5
Claude Sonnet 5
parent
b68c16d8e6
commit
e902197462
@@ -76,7 +76,13 @@ void main() {
|
||||
expect(all, hasLength(1));
|
||||
expect(all.first.name, 'T1 Ranger');
|
||||
expect(all.first.gpsSatGreenMin, 8.0);
|
||||
expect(all.first.batteryVoltageRedMinV, 19.2);
|
||||
// batteryVoltageRedMinV ist seit Schema v9 ein berechneter Getter
|
||||
// (batteryCellCount * batteryVoltageRedMinPerCellV) statt eines
|
||||
// gespeicherten Absolutwerts - die alte v8-Spalte wird von diesem Pfad
|
||||
// (v7 -> aktuell) durchlaufen, aber nicht mehr gelesen. Erwartet wird
|
||||
// deshalb der neue v9-Default: 4 Zellen * 3.2 V.
|
||||
expect(all.first.batteryCellCount, 4);
|
||||
expect(all.first.batteryVoltageRedMinV, 12.8);
|
||||
|
||||
await db.close();
|
||||
await dir.delete(recursive: true);
|
||||
|
||||
@@ -0,0 +1,106 @@
|
||||
import 'dart:io';
|
||||
|
||||
import 'package:drift/native.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:sqlite3/sqlite3.dart' as sqlite3;
|
||||
import 'package:dmc_app/services/database/app_database.dart';
|
||||
import 'package:dmc_app/services/database/drone_profile_repository.dart';
|
||||
|
||||
/// Reproduziert das Upgrade eines Geraets auf Schema v8 (DroneProfiles mit
|
||||
/// den Ampel-Schwellwert-Spalten, aber noch der alten festen Pack-Spannung
|
||||
/// batteryVoltageGreenMinV/batteryVoltageRedMinV statt Zellenzahl +
|
||||
/// Pro-Zelle-Schwellwerten) auf das aktuelle Schema (v9, Doku: "zellen
|
||||
/// anzahl der batterie im drone profile hinzu fuegen") - analog
|
||||
/// migration_v7_to_v8_repro_test.dart: baut die v8-Tabellenstruktur samt
|
||||
/// Bestandszeile roh per sqlite3 auf, oeffnet die Datei dann ganz normal
|
||||
/// ueber AppDatabase und prueft, dass die neuen Spalten mit den erwarteten
|
||||
/// Default-Werten befuellt werden.
|
||||
void main() {
|
||||
test('Upgrade von Schema v8 auf v9 befuellt neue Spalten mit Defaults', () async {
|
||||
final dir = await Directory.systemTemp.createTemp('dmc_migration_test');
|
||||
final file = File('${dir.path}/test.sqlite');
|
||||
|
||||
final raw = sqlite3.sqlite3.open(file.path);
|
||||
raw.execute('''
|
||||
CREATE TABLE missions (
|
||||
id INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT,
|
||||
name TEXT NOT NULL,
|
||||
location TEXT,
|
||||
created_at INTEGER NOT NULL,
|
||||
updated_at INTEGER NOT NULL,
|
||||
waypoints_json TEXT NOT NULL,
|
||||
is_dirty INTEGER NOT NULL DEFAULT 1,
|
||||
server_id TEXT,
|
||||
terrain_profile_json TEXT
|
||||
);
|
||||
CREATE TABLE drone_profiles (
|
||||
id INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT,
|
||||
name TEXT NOT NULL,
|
||||
min_speed REAL NOT NULL,
|
||||
cruise_speed REAL NOT NULL,
|
||||
max_speed REAL NOT NULL,
|
||||
min_turn_radius REAL NOT NULL,
|
||||
loiter_radius REAL NOT NULL,
|
||||
catch_radius REAL NOT NULL,
|
||||
max_climb_rate REAL NOT NULL,
|
||||
max_descent_rate REAL NOT NULL,
|
||||
max_range_m REAL NOT NULL,
|
||||
max_endurance_min REAL NOT NULL,
|
||||
max_altitude_m REAL NOT NULL,
|
||||
gps_sat_green_min REAL NOT NULL DEFAULT 8.0,
|
||||
gps_sat_red_min REAL NOT NULL DEFAULT 6.0,
|
||||
hdop_green_max REAL NOT NULL DEFAULT 2.0,
|
||||
hdop_red_max REAL NOT NULL DEFAULT 5.0,
|
||||
current_green_max_a REAL NOT NULL DEFAULT 20.0,
|
||||
current_red_max_a REAL NOT NULL DEFAULT 30.0,
|
||||
battery_voltage_green_min_v REAL NOT NULL DEFAULT 20.4,
|
||||
battery_voltage_red_min_v REAL NOT NULL DEFAULT 19.2,
|
||||
rx_quality_green_min REAL NOT NULL DEFAULT 70.0,
|
||||
rx_quality_red_min REAL NOT NULL DEFAULT 50.0,
|
||||
rx_snr_green_min_db REAL NOT NULL DEFAULT 0.0,
|
||||
rx_snr_red_min_db REAL NOT NULL DEFAULT -10.0
|
||||
);
|
||||
CREATE TABLE app_settings_table (
|
||||
id INTEGER NOT NULL PRIMARY KEY,
|
||||
active_drone_profile_id INTEGER,
|
||||
current_mission_id INTEGER,
|
||||
connection_device_address TEXT,
|
||||
connection_auto_connect INTEGER NOT NULL DEFAULT 1,
|
||||
connection_type TEXT NOT NULL DEFAULT 'wifi',
|
||||
connection_wifi_host TEXT,
|
||||
connection_wifi_port INTEGER NOT NULL DEFAULT 14550,
|
||||
connection_wifi_ssid_prefix TEXT NOT NULL DEFAULT 'mLRS-',
|
||||
connection_wifi_last_ssid TEXT
|
||||
);
|
||||
INSERT INTO drone_profiles (
|
||||
id, name, min_speed, cruise_speed, max_speed, min_turn_radius,
|
||||
loiter_radius, catch_radius, max_climb_rate, max_descent_rate,
|
||||
max_range_m, max_endurance_min, max_altitude_m,
|
||||
battery_voltage_green_min_v, battery_voltage_red_min_v
|
||||
) VALUES (
|
||||
1, 'T1 Ranger', 13, 15, 25, 35, 100, 60, 5, 6, 15000, 45, 2000,
|
||||
20.4, 19.2
|
||||
);
|
||||
PRAGMA user_version = 8;
|
||||
''');
|
||||
raw.close();
|
||||
|
||||
final db = AppDatabase.forTesting(NativeDatabase(file));
|
||||
final repo = DroneProfileRepository(db);
|
||||
|
||||
final all = await repo.loadAll();
|
||||
expect(all, hasLength(1));
|
||||
expect(all.first.name, 'T1 Ranger');
|
||||
// Neue v9-Spalten sollen mit den DroneProfile.default*-Werten befuellt
|
||||
// werden, unabhaengig von der alten (jetzt toten) Pack-Spannung der
|
||||
// Bestandszeile.
|
||||
expect(all.first.batteryCellCount, 4);
|
||||
expect(all.first.batteryVoltageGreenMinPerCellV, 3.4);
|
||||
expect(all.first.batteryVoltageRedMinPerCellV, 3.2);
|
||||
expect(all.first.batteryVoltageGreenMinV, closeTo(13.6, 0.0001));
|
||||
expect(all.first.batteryVoltageRedMinV, closeTo(12.8, 0.0001));
|
||||
|
||||
await db.close();
|
||||
await dir.delete(recursive: true);
|
||||
});
|
||||
}
|
||||
@@ -24,6 +24,7 @@ void main() {
|
||||
expect(all.single.id, id);
|
||||
expect(all.single.name, 'T1 Ranger');
|
||||
expect(all.single.cruiseSpeed, 15);
|
||||
expect(all.single.batteryCellCount, 4);
|
||||
});
|
||||
|
||||
test('ensureSeeded ist ein no-op, wenn bereits Profile existieren', () async {
|
||||
|
||||
@@ -16,9 +16,41 @@ void main() {
|
||||
expect(parsed.profile.minSpeed, profile.minSpeed);
|
||||
expect(parsed.profile.maxSpeed, profile.maxSpeed);
|
||||
expect(parsed.profile.maxAltitudeM, profile.maxAltitudeM);
|
||||
expect(parsed.profile.batteryCellCount, profile.batteryCellCount);
|
||||
expect(
|
||||
parsed.profile.batteryVoltageGreenMinPerCellV,
|
||||
profile.batteryVoltageGreenMinPerCellV,
|
||||
);
|
||||
expect(
|
||||
parsed.profile.batteryVoltageRedMinPerCellV,
|
||||
profile.batteryVoltageRedMinPerCellV,
|
||||
);
|
||||
expect(parsed.versionMismatch, isNull);
|
||||
});
|
||||
|
||||
test('alte Exportdatei ohne batteryCellCount faellt auf die neuen '
|
||||
'Defaults zurueck, statt die alte Pack-Spannung zu uebernehmen', () {
|
||||
final json = buildDroneExportJson(profile).replaceFirst(
|
||||
RegExp(
|
||||
r',\s*"batteryCellCount": [\d.]+,\s*'
|
||||
r'"batteryVoltageGreenMinPerCellV": [\d.]+,\s*'
|
||||
r'"batteryVoltageRedMinPerCellV": [\d.]+',
|
||||
),
|
||||
'',
|
||||
);
|
||||
final parsed = parseDroneImportJson(json);
|
||||
|
||||
expect(parsed.profile.batteryCellCount, DroneProfile.defaultBatteryCellCount);
|
||||
expect(
|
||||
parsed.profile.batteryVoltageGreenMinPerCellV,
|
||||
DroneProfile.defaultBatteryVoltageGreenMinPerCellV,
|
||||
);
|
||||
expect(
|
||||
parsed.profile.batteryVoltageRedMinPerCellV,
|
||||
DroneProfile.defaultBatteryVoltageRedMinPerCellV,
|
||||
);
|
||||
});
|
||||
|
||||
test('erkennt eine abweichende Hauptversion als Mismatch', () {
|
||||
final json = buildDroneExportJson(profile).replaceFirst('"1.0.0"', '"2.0.0"');
|
||||
final parsed = parseDroneImportJson(json);
|
||||
|
||||
Reference in New Issue
Block a user