Support binary_sensor and device_tracker in HomeKit.

[Yonsm]

Description:

Map binary_sensor and device_tracker to HomeKit Binary Sensor, based on 'device_class' attributes:

'co2': 'CarbonDioxideSensor'
'gas': 'CarbonMonoxideSensor'
'moisture': 'LeakSensor'
'motion': 'MotionSensor'
'occupancy': 'OccupancySensor'
'opening': 'ContactSensor'
'smoke': 'SmokeSensor'

If there is no any matched device_class, e.g. device_tracker, then default sensor type is OccupancySensor.

Checklist:

• The code change is tested and works locally.
• Local tests pass with tox.

---

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#5143

[houndci-bot]

line too long (143 > 79 characters)

[cdce8p]

It's looking good so far. Unfortunately I can't test it until tomorrow evening (CET).
Since you dynamically assign the service and char, can you add tests to verify that the right ones where selected. You can get the service / char name with service.display_name or char.display_name. Those must be equal to the HomeKit char / service constants.

[cdce8p]

You don't need that comment.

[Yonsm]

done

[houndci-bot]

continuation line under-indented for visual indent

[cdce8p]

Please add this to the end. That way you improve the coverage.

        self.hass.states.remove(entity_id)
        self.hass.block_till_done()

[cdce8p]

Instead of L85 - L88, why not do something like that:

from homeassistant.components.homekit.type_sensors import BINARY_SENSOR_SERVICE_MAP

class TestHomekitSensors(unittest.TestCase):

#####
#####

    def test_binary(self):

#####
#####

    def test_binary_device_classes(self):

        entity_id = 'binary_sensor.demo'

        for device_class, (service, char) in BINARY_SENSOR_SERVICE_MAP.items():
            self.hass.states.set(entity_id, STATE_OFF,
                                 {ATTR_DEVICE_CLASS: device_class})
            self.hass.block_till_done()

            acc = BinarySensor(self.hass, entity_id, 'Binary Sensor', aid=2)
            self.assertEqual(acc.get_service(service).display_name, service)
            self.assertEqual(acc.char_detected.display_name, char)

That way you check if every service and char can be added successfully.

[Yonsm]

done

[cdce8p]

Thanks 👍
Looking forward to more PR's

[Yonsm]

Thank you for your effective advice and guidance.

[shuaiger]

cong~ merged!

[nicx]

@Yonsm great, thanks a lot! @cdce8p maybe its possible to add a default mapping to binary_sensors with device_class "opening" for all sensors which returns states like "open/tilted/closed"? something similar to the existing temperature sensors. with that change I don't need to create extra template binary_sensors anymore :)

[cdce8p]

@nicx There is a plan to add device_classes to normal sensors which would allow better types for HomeKit. I just don't know if I get to it this week.

[nicx]

@cdce8p great to hear, I can wait for this 👍 keep up this great work! 🥇

[leblancexplores]

Great work @Yonsm and @cdce8p! One more thing (cough I feel I say this too often), it would be nice to support "door" as a device_class for ContactSensor as well?

[leblancexplores]

Or maybe it's best to default to ContactSensor since this allows the most flexibility in Homekit and is probably the likely use of a binary_sensor?

[MartinHjelmare]

Please don't discuss feature requests in closed PRs.

[leblancexplores]

@MartinHjelmare seemed like an omission, what SHOULD I do instead?

[MartinHjelmare]

If you want to suggest an enhancement please open a feature request in the Feature Requests section of our community forum.

Merged PRs should not be used for enhancement discussion or bug reports. If you've found a bug it's ok to make a review with inline comments and link to an issue that reports the bug.

Thanks!