matthias-bs / BresserWeatherSensorReceiver / 25905989208
100%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 8
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 100.0%. Remained the same
25905989208

push

github

web-flow 
Fix HA auto-discovery device merging and add user-defined device names (#308)

* Fix HA device identifier collision for same-type sensors

Each sensor type in haAutoDiscovery() was using a hardcoded string as
the HA device identifier (e.g. "thermo_hygrometer_sensor_1"), causing
multiple physical sensors of the same type to overwrite each other's
HA device instead of registering as separate devices.

Build the identifier dynamically from sensor_id for all 9 sensor types.

* Surface sensor_map name as HA device display name

The HA device name was always "Bresser <Model>", ignoring any
user-defined name from sensor_map. Add an optional display_name field
to sensor_info and pass sensor_str (already derived from sensor_map via
sensorName()) in every sensor type block. publishAutoDiscovery() uses
display_name when set, falling back to "manufacturer model" otherwise.

* Fix format-specifier cast and update publishAutoDiscovery doc

Add missing (unsigned) cast to sensor_id in the 9 new identifier
snprintf calls, matching the existing pattern used elsewhere in the
file. Update the \param info Doxygen comment to mention display_name.

* Sync MQTTCustom and MQTTWifiMgr examples with MQTT fixes

Port the HA auto-discovery fixes to the two sibling sketches:
- Unique device identifier per sensor_id (collision fix)
- display_name field for sensor_map-based HA device naming
- (unsigned) cast on sensor_id in snprintf calls

* Fix display_name regression for sensors without a sensor_map entry

sensorName() now returns bool to indicate whether a user-defined name
was found. display_name is only set when a mapping exists, so sensors
without a sensor_map entry keep the default "Bresser <Model>" HA device
name instead of falling back to a raw hex sensor ID.

372 of 372 relevant lines covered (100.0%)

145.88 hits per line

Jobs
ID Job ID Ran Files Coverage
1 25905989208.1 8
100.0
 GitHub Action Run
Source Files on build 25905989208