Add Rachio Flex Schedules

[brg468]

Proposed change

Add Rachio Flex Schedules as switches.

Follow-up to #33421

Type of change

• Dependency upgrade
• Bugfix (non-breaking change which fixes an issue)
• New integration (thank you!)
• New feature (which adds functionality to an existing integration)
• Breaking change (fix/feature causing existing functionality to break)
• Code quality improvements to existing code or addition of tests

Additional information

• This PR fixes or closes issue: fixes #
• This PR is related to issue:
• Link to documentation pull request:

Checklist

• The code change is tested and works locally.
• Local tests pass. Your PR cannot be merged unless tests pass
• There is no commented out code in this PR.
• I have followed the development checklist
• The code has been formatted using Black (black --fast homeassistant tests)
• Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

• Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

• The manifest file has all fields filled out correctly.
  Updated and included derived files by running: python3 -m script.hassfest.
• New or updated dependencies have been added to requirements_all.txt.
  Updated by running python3 -m script.gen_requirements_all.
• Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

• No score or internal
• 🥈 Silver
• 🥇 Gold
• 🏆 Platinum

[probot-home-assistant]

Hey there @bdraco, mind taking a look at this pull request as its been labeled with a integration (rachio) you are listed as a codeowner for? Thanks!

[bdraco]

Looks good, I'll have some time to test tonight

[bdraco]

Just tested, flex schedules are not showing up.

[bdraco]

{
	"flexScheduleRules": [{
		"id": "<snip>",
		"cycleSoak": false,
		"startDay": 30,
		"name": "Main",
		"totalDuration": 20161,
		"operator": "AFTER",
		"startYear": 2019,
		"enabled": true,
		"summary": "As needed end by sunrise",
		"startMonth": 11,
		"externalName": "Main",
		"scheduleJobTypes": ["ANY"],
		"startDate": 1575108000000,
		"type": "FLEX",
		"zones": [{
			"zoneId": "<snip>",
			"sortOrder": 0,
			"duration": 2086
		}, {
			"zoneId": "<snip>",
			"sortOrder": 3,
			"duration": 1390
		}, {
			"zoneId": "<snip>",
			"sortOrder": 4,
			"duration": 2086
		}, {
			"zoneId": "<snip>",
			"sortOrder": 1,
			"duration": 4421
		}, {
			"zoneId": "<snip>",
			"sortOrder": 2,
			"duration": 3920
		}, {
			"zoneId": "<snip>",
			"sortOrder": 5,
			"duration": 2086
		}, {
			"zoneId": "<snip>",
			"sortOrder": 6,
			"duration": 2086
		}, {
			"zoneId": "<snip>",
			"sortOrder": 7,
			"duration": 2086
		}]
	}]
}

[bdraco]

Just tested, flex schedules are not showing up.

Disregard: I missed the merge conflict so it wasn't up to date. (testing error on my part)

[bdraco]

Verified ok running from app -> shows in Home Assistant

[brg468]

Ha phew! I just built a new schedule to test and it was there so I was about to be real confused.

[bdraco]

Verified turning off in app shows in Home Assistant

[bdraco]

Verified on in home assistant show on in app

[bdraco]

Verified off in home assistant show on in app

[bdraco]

TEST PASS

[brg468]

TEST PASS

👍

[MartinHjelmare]

A coroutine function can't be a callback.

• Please remove the async in async def.
• Rename the function to _async_handle_update.
• Change self.schedule_update_ha_state to self.async_write_ha_state inside.
• Do this for all the switch classes and the abstract method.
• Rename _handle_any_update too, similarly.
• Decorate all these methods with @callback.

[bdraco]

Thanks @MartinHjelmare

@brg468 Do you want to do a PR for this?, or I can take care of it needed.

[brg468]

If you had any other changes you were planning go ahead, if not I can do it in a bit.

[bdraco]

I'm working on fixing something in homekit so you'll probably get to it before I do.