forked from fritob/Camper-Monitor
Kühlbox: Stellbefehl in der Länge, die die Box erwartet
Diesmal gegen die Referenzimplementierung geprüft statt geraten. Rahmen, Längenbyte, Prüfsumme, Anmelde- und Abfragepaket stimmen dort byteweise mit unseren überein. Ein Unterschied blieb: die Länge des Einstellungs- blocks. Ich hatte sie an die Zonenerkennung gehängt. Sendet eine Box den langen Datensatz, füllt den zweiten Block aber mit Nullen, galt sie als einzonig und bekam den kurzen Befehl - während sie den langen erwartet. Die Box verwirft ihn dann wortlos, und Ein- und Ausschalten blieb wirkungslos. Die Referenz hängt die Länge an die Antwort der Box, nicht an eine Deutung davon. Jetzt auch hier. Für die Anzeige bleibt die Erkennung unverändert: ein leerer zweiter Block wird weiterhin nicht als zweite Zone gezeigt. Zweitens die Anmeldung. Abfragen beantwortet die Box teils auch unangemeldet, Stellbefehle nicht. Ob die Anmeldung je bestätigt wurde, wurde bisher nicht verfolgt. Jetzt schon: fehlt die Bestätigung, wird vor dem Stellbefehl noch einmal angemeldet. In der Diagnose steht der Stand als eigene Zeile. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
+6
-5
@@ -530,12 +530,13 @@ checkEqual("leerer zweiter Block gilt nicht als zweite Zone", padded.isDualZone,
|
||||
// 20 Byte noch in einen Schreibvorgang, aus dem langen nicht mehr. Ohne
|
||||
// Aufteilen verwirft die Box ihn wortlos – genau das war der Fehler.
|
||||
checkEqual("Stellbefehl aus dem kurzen Datensatz", offBytes.count, 20)
|
||||
// Eine als einzonig erkannte Box bekommt auch nur den kurzen Befehl,
|
||||
// obwohl sie den langen Datensatz sendet.
|
||||
checkEqual("kurzer Stellbefehl bei einzoniger Box",
|
||||
padded.settingsCommand(poweredOn: false)?.count, 20)
|
||||
// Der Befehl folgt der Länge, in der die Box selbst antwortet – nicht dem,
|
||||
// was die Anzeige aus dem leeren zweiten Block macht. Eine Box, die den
|
||||
// langen Datensatz sendet, nimmt nur den langen Befehl an.
|
||||
checkEqual("langer Datensatz heisst langer Stellbefehl",
|
||||
padded.settingsCommand(poweredOn: false)?.count, 31)
|
||||
checkEqual("die Anzeige bleibt trotzdem einzonig", padded.isDualZone, false)
|
||||
|
||||
padded.zoneMode = .dual
|
||||
guard let longPacket = padded.settingsCommand(poweredOn: true) else {
|
||||
check("Stellbefehl aus dem langen Datensatz wird gebaut", false); exit(1)
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user