Add switch for motion detection

[SamMousa]

This depends on my PR here: cgarwood/python-fullykiosk#10

I've tested it to be working on my installation without modifying the underlying python library (harder to reach from vs-code plugin). To do that I just adapted the code from this PR to call existing function in python-fullykiosk directly.

So instead of calling enableMotionDetection etc I did this:

class FullyKioskMotionDetectionSwitch(FullySwitch):
    """Representation of a Fully Kiosk Browser kiosk lock switch."""

    def __init__(self, hass, coordinator):
        """Intialize the kiosk lock switch."""
        super().__init__(hass, coordinator)
        self._name = f"{coordinator.data['deviceName']} Motion Detection"
        self._unique_id = f"{coordinator.data['deviceID']}-motion-detection"

    @property
    def is_on(self):
        """Return if motion detection is on."""
        return self.coordinator.data["settings"]["motionDetection"]

    async def async_turn_on(self, **kwargs):
        """Turn on kiosk lock."""
        await self.coordinator.fully.setConfigurationBool("motionDetection", True)
        await self.coordinator.async_refresh()

    async def async_turn_off(self, **kwargs):
        """Turn off kiosk lock."""
        await self.coordinator.fully.setConfigurationBool("motionDetection", False)
        await self.coordinator.async_refresh()

And instead of calling getSettings in coordinator.py I did this:

async def _async_update_data(self):
        """Update data via library."""
        try:
            with timeout(15):
                """Get device info and settings in parallel"""
                result = await asyncio.gather(self.fully.getDeviceInfo(), self.fully.sendCommand("listSettings"))
                """Store settings under settings key in data"""
                result[0]["settings"] = result[1]
                return result[0]
        except (FullyKioskError, ClientConnectorError) as error:
            raise UpdateFailed(error) from error

Again I don't speak python so check thoroughly!

This should also provide a simple framework for adding other switches for settings.

[SamMousa]

Does this look good @cgarwood?

[bcutter]

Does this look good @cgarwood?

Just want to ask the same question @cgarwood 2,5 months later :-)

[cgarwood]

😬 so far behind on projects.. just merged your other PR and did a new release of the library, 0.0.9. Please update the requirements in manifest.json and give it a quick test then I think we're good to merge.

[SamMousa]

@cgarwood Done!

[chpego]

Hi @SamMousa , did you try to fix the format of your commit ?

[SamMousa]

Not sure what you mean? I thought this one was done, but haven't looked at it in a long while!

[chpego]

Not sure what you mean? I thought this one was done, but haven't looked at it in a long while!

the check formatting action failed on some parts of the code you pushed :
https://github.com/cgarwood/homeassistant-fullykiosk/runs/4289157913?check_suite_focus=true#step:4:61

[SamMousa]

Yeah will do when I have time!

[cgarwood]

Had some free time so I committed the code formatting changes 🙂

HACS validation test failure is unrelated to this PR