feat: add AWAY state configuration - Githubissues

palazzem / ha-econnect-alarm

Home Assistant integration that provides a full-fledged Alarm Panel to control your Elmo/IESS alarm systems.

BSD 3-Clause "New" or "Revised" License

10 stars 5 forks source link

feat: add AWAY state configuration #107

Closed palazzem closed 7 months ago

palazzem commented 7 months ago

Related Issues

Closes #96

Proposed Changes:

Adds arm AWAY configuration in the OptionsFlow step with the following logic:

After version update, the setting is empty
If the setting is not set (empty), the device arms all sectors (backward compatibility)
If the setting is set, only selected sectors are armed

Example

Testing:

Arming the system without setting the option should arm all sectors
Arming the system with the option set, should arm only selected sectors

Extra Notes (optional):

n/a

Checklist

[x] Related issues and proposed changes are filled
[x] Tests are defining the correct and expected behavior
[x] Code is well-documented via docstrings

coveralls commented 7 months ago

Pull Request Test Coverage Report for Build 6797771020

4 of 4 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.249%

Totals
Change from base Build 6797209446:	0.2%
Covered Lines:	504
Relevant Lines:	536

💛 - Coveralls