etm-powersync-app/test/merged_config_screen_test.dart
Patrick Schurig 3d2aab2b42 fix(config): les écrans suivent enfin la maquette — sélecteurs, table d'états, budget, borne
Je n'avais pas ouvert docs/mockups/ : les écrans ont été construits depuis le texte du brief
seul, et l'écart est franc. Les maquettes du 2026-08-25 font autorité, elles sont désormais
suivies.

ÉCRANS DE MÉCANISME, refaits :
- sélecteur de Things PAR NOM, filtré sur l'interface nymea (power / etmvariableload).
  Taper un UUID au doigt est une erreur garantie ET muette : la box accepte un UUID valide
  qui ne désigne rien, et la charge ne commande jamais rien. Le sélecteur grise les Things
  déjà pris — la box refuse en bloc un même Thing sur deux lignes — et signale en rouge un
  ThingId configuré qui n'existe plus sur la box ;
- routeur de relais : ajout/retrait de contacteurs, puissance par ligne, et les paliers
  atteignables AFFICHÉS. Ils sont déduits (2^N combinaisons, doublons écartés), jamais
  enregistrés — la maquette est explicite là-dessus, ma note précédente disant l'inverse
  était une sur-interprétation du brief ;
- SG-Ready : SG1/SG2 se choisissent EN HAUT, une fois, et la table des quatre états devient
  éditable — ce qui s'y coche est fermé/ouvert, et c'est ça qui devient relays[]. Les
  étiquettes SG1/SG2 sont une convention de l'écran (SG1 = fermé en état 1, SG2 = en état 3)
  et elles TOMBENT sur un encodage non standard, où les contacts s'affichent tels quels ;
- modulable : plafond requis > 0, et l'avertissement que changer d'appareil crée une autre
  charge puisque l'identifiant EST le ThingId ;
- temporisations minOnS/minOffS (et minStateHoldS pour la PAC) en pas de 30/60 s ;
- « Compteur dédié » présent et GRISÉ, avec sa raison : aucun champ n'existe côté box, et
  inventer meterThingId ferait rejeter l'enregistrement de TOUTES les charges ;
- « Inclure dans l'arbitrage » avec rang et domaine, et ce que la désactivation fait
  vraiment selon le mécanisme ;
- validation AVANT envoi : relays[] vide, powerW ≤ 0, Thing en double, état 2 absent,
  maxPowerW ≤ 0. SetLoadConfig refuse en bloc et sans motif — laisser partir une config
  invalide condamne l'installateur à un refus muet qui perd tout l'appel.

LISTE :
- l'en-tête d'arbitrage et le tableau de budget, PERDUS à la fusion, sont rétablis. Sans
  eux « Alloué 3 000 W » ne se rattache à rien ;
- le rang passe en pastille près de la poignée, et le sous-titre devient lisible :
  « Routeur de relais · 3 relais · min ON 60 s » au lieu de « Routeur de relais · rang 1 » ;
- section « Recharge véhicule » : la borne n'est pas une LoadConfig, elle n'apparaissait donc
  NULLE PART — ce qu'un installateur qui vient de la régler lit comme une perte.

MENU : l'entrée « Ordre de service des charges » est retirée ; elle pointait sur une
redirection, donc sur le même écran sous deux noms.

Tests 80/80, analyze 0 erreur.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HajXLUczEyZd22JeewRfff
2026-08-25 23:27:24 +02:00

220 lines
8.8 KiB
Dart

/// L'écran de configuration FUSIONNÉ, éprouvé sur les dumps **réels** de `.75`.
///
/// > ⚠️ **Ce sont des tests de widgets, pas une vérification machine.** Ils montent les
/// > vrais widgets sur les vraies charges utiles de la box (`test/fixtures/*_v3.json`,
/// > plugin `1.15.2+etm15`), ce qui couvre le rendu et les écritures préparées — mais
/// > **aucune** ne part vers `.75` ici. Ce que ces tests ne prouvent pas est écrit dans
/// > le journal de session : l'IU n'a pas pu être pilotée sur l'appareil, l'injection
/// > d'événements ADB étant refusée par MIUI.
library;
import 'dart:convert';
import 'dart:io';
import 'package:etm_powersync_app/l10n/app_localizations.dart';
import 'package:etm_powersync_app/models/load_config_entry.dart';
import 'package:etm_powersync_app/providers/load_config_provider.dart';
import 'package:etm_powersync_app/providers/load_telemetry_provider.dart';
import 'package:etm_powersync_app/services/load_hardware.dart';
import 'package:etm_powersync_app/services/nymea_service.dart';
import 'package:etm_powersync_app/services/rpc_schema.dart';
import 'package:etm_powersync_app/widgets/controllable_loads_section.dart';
import 'package:flutter/material.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:provider/provider.dart';
Map<String, dynamic> _fixture(String name) =>
jsonDecode(File('test/fixtures/$name').readAsStringSync())
as Map<String, dynamic>;
/// Config réelle de `.75`, avec les domaines tels qu'ils y sont posés aujourd'hui
/// (`chauffe-eau → ecs`, `pac-terrain → hvac`) — le multi-domaines du banc.
List<Map<String, dynamic>> _benchConfig() {
final raw = (_fixture('loadconfig_hems75_v3.json')['loadConfigs'] as List)
.cast<Map<String, dynamic>>();
return [
for (final c in raw)
{...c, 'domain': c['id'] == 'chauffe-eau' ? 'ecs' : 'hvac'},
];
}
Widget _harness({
required LoadConfigProvider config,
required LoadTelemetryProvider telemetry,
}) =>
MultiProvider(
providers: [
ChangeNotifierProvider.value(value: config),
ChangeNotifierProvider.value(value: telemetry),
// La section « Recharge véhicule » lit les Things de la box pour savoir s'il y a
// une borne. Hors connexion, la liste est vide et la section s'efface — c'est
// exactement le comportement attendu sur une installation sans borne.
ChangeNotifierProvider.value(value: NymeaService()),
],
child: MaterialApp(
locale: const Locale('fr'),
localizationsDelegates: L10n.localizationsDelegates,
supportedLocales: L10n.supportedLocales,
home: Scaffold(
body: SingleChildScrollView(
child: ControllableLoadsSection(onConfigure: (_, __) {}),
),
),
),
);
void main() {
group('fusion — la liste vient de GetLoadConfig, pas des rôles', () {
testWidgets('les deux charges du banc apparaissent, chacune dans son domaine',
(tester) async {
final config = LoadConfigProvider()..adoptForTest(_benchConfig());
final telemetry = LoadTelemetryProvider()
..adoptForTest(_fixture('loadtelemetry_hems75_v3.json')['telemetry']
as Map<String, dynamic>);
await tester.pumpWidget(_harness(config: config, telemetry: telemetry));
await tester.pump();
// Les identifiants viennent de la box — pas d'une énumération figée de rôles.
expect(find.text('chauffe-eau'), findsWidgets);
expect(find.text('PAC banc'), findsWidgets);
// Deux mécanismes distincts, deux écrans de détail : un bouton par charge.
expect(find.text('Configurer'), findsNWidgets(2));
});
testWidgets('une charge désactivée se lit « désactivée », jamais « perdue »',
(tester) async {
final cfg = _benchConfig();
final desactivee = [
for (final c in cfg)
if (c['id'] == 'pac-terrain') {...c, 'enabled': false} else c,
];
final config = LoadConfigProvider()..adoptForTest(desactivee);
// Télémétrie VIDE : c'est exactement le cas d'une charge hors arbitrage.
final telemetry = LoadTelemetryProvider()
..adoptForTest({'degradedMode': false, 'loads': const []});
await tester.pumpWidget(_harness(config: config, telemetry: telemetry));
await tester.pump();
expect(find.textContaining('Désactivée'), findsWidgets);
expect(find.textContaining('perdue'), findsNothing);
});
});
group('annonce d\'impact avant écriture', () {
test('réordonner les contacteurs déclenche un avertissement, pas un silence', () {
final config = LoadConfigProvider()..adoptForTest(_benchConfig());
final avant = config.entries.firstWhere((e) => e.id == 'chauffe-eau');
final inverse = avant.patched(relays: [
for (final r in avant.relays.reversed)
{'thingId': r.thingId, 'powerW': r.powerW}
]);
config.replaceEntry('chauffe-eau', inverse);
expect(config.hasPendingChanges, isTrue);
expect(config.pendingNeedsWarning, isTrue,
reason: 'la comparaison du moteur est indexée — vérifié sur .75');
final impact = config
.pendingImpact()
.firstWhere((i) => i.loadId == 'chauffe-eau');
expect(impact.impact, LoadChangeImpact.rebuild);
expect(impact.changedFields, ['relays']);
});
test('renommer ou reclasser n\'avertit de rien', () {
final config = LoadConfigProvider()..adoptForTest(_benchConfig());
config.setLabel('chauffe-eau', 'Ballon 200 L');
expect(config.hasPendingChanges, isTrue);
expect(config.pendingNeedsWarning, isFalse);
});
test('désactiver la PAC avertit — elle passe par son état sûr', () {
final config = LoadConfigProvider()..adoptForTest(_benchConfig());
config.setEnabled('pac-terrain', false);
final impact =
config.pendingImpact().firstWhere((i) => i.loadId == 'pac-terrain');
expect(impact.impact, LoadChangeImpact.removed);
expect(impact.needsWarning, isTrue);
});
});
group('grisage dérivé du schéma RÉEL de .75', () {
late RpcSchema schema;
setUp(() {
final f = _fixture('schema_hems75_etm15.json');
schema = RpcSchema.fromIntrospect(f, hello: {
'version': f['_hello_version'],
'experiences': [
for (final e in (f['_hello_experiences'] as Map).entries)
{'name': e.key, 'version': e.value},
],
});
});
test('le schéma se lit, et les versions d\'API avec lui', () {
expect(schema.loaded, isTrue);
expect(schema.experienceVersions['NymeaEnergy'], '0.8');
// La version du PAQUET (+etm15) n'est PAS sur le fil : ce qu'on annonce est
// la version d'API, qui décrit le contrat.
expect(schema.versionLineContains('etm15'), isFalse);
});
test('ChargingInfo : les champs d\'échéance existent, minCurrent NON', () {
expect(schema.typeSupport('ChargingInfo', ['targetPercentage']),
FieldSupport.writable);
expect(schema.typeSupport('ChargingInfo', ['endDateTime']),
FieldSupport.writable);
// Le mode existe, son paramètre non — c'est ce que l'écran doit dire.
expect(schema.typeSupport('ChargingInfo', ['minCurrent']),
FieldSupport.absent);
expect(schema.enumValues('ChargingMode'),
contains('ChargingModeEcoWithMinCurrent'));
});
test('ChargingInfo : chargingState est une MESURE, pas un champ manquant', () {
expect(schema.typeSupport('ChargingInfo', ['chargingState']),
FieldSupport.readOnly);
});
test('SetLoadConfig : les charges utiles de mécanisme sont au schéma', () {
for (final champ in ['relays', 'sgReady', 'maxPowerW', 'domain']) {
expect(
schema.paramSupport('NymeaEnergy.SetLoadConfig',
['loadConfigs', '[]', champ]),
FieldSupport.writable,
reason: '$champ doit être écrivable sur .75',
);
}
});
test('AirConditioning est pleinement pilotable — il n\'y a rien à griser', () {
for (final m in [
'AirConditioning.AddZone',
'AirConditioning.SetZoneThings',
'AirConditioning.SetZoneWeekSchedule',
'AirConditioning.SetZoneSetpointOverride',
]) {
expect(schema.hasMethod(m), isTrue, reason: '$m est publiée par .75');
}
});
test('un schéma NON LU ne dit pas « absent » — il dit « je ne sais pas »', () {
expect(RpcSchema.unknown.typeSupport('ChargingInfo', ['targetPercentage']),
FieldSupport.unknown);
expect(RpcSchema.unknown.paramSupport('NymeaEnergy.SetLoadConfig', ['x']),
FieldSupport.unknown);
});
});
}
extension on RpcSchema {
bool versionLineContains(String s) =>
(nymeaVersion ?? '').contains(s) ||
experienceVersions.values.any((v) => v.contains(s));
}