Skip to content

test: add Svakom Fatima Pro temperature regression coverage #923

Description

@qdot

Summary

Add regression coverage for the Svakom Fatima Pro temperature output introduced in PR #908.

The protocol exposes a Temperature feature and implements distinct heat on/off packets, but test_svakom_fatima.yaml currently omits them. Its comment says the v4 device-test client cannot send Temperature commands; however, tests/util/device_test/client/client_v4/mod.rs already maps OutputType::Temperature to ClientDeviceOutputCommand::Temperature.

Expected coverage

  • Temperature 1.0 emits heat-on: 55 05 01 37 02 00 00
  • Temperature 0.0 emits heat-off: 55 05 00 00 02 00 00
  • Coverage runs through the YAML-driven device protocol tests where possible; extend the test message representation if Temperature is not currently expressible there.

Context

Follow-up from review of PR #908. This isolates the missing regression coverage from the protocol implementation itself.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions