fix(device): avoid state update while arming or disarming

[palazzem]

Related Issues

• Closes Alarm state moves between states when activated via HomeKit #154

Proposed Changes:

This change fixes a race condition that happens when the device arms the system, but before this happens an update is triggered and brings the system to a wrong state. Furthermore, avoids the AlarmDevice to set "Arm Away" wrong state.

Testing:

From "disarm", trigger the "arm home" state and be sure it doesn't transition back to "disarm", or in a wrong "arm away" state.

Extra Notes (optional):

This change creates a dependency between the AlarmDevice and the arm/disarm caller as it is expected who calls one of these methods, to set the right ending state. While this is not great and would be better to decouple it, it's a good enough solution. At some point, we may want to remove AlarmDevice transforming it in an actual entity.

Checklist

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

[coveralls]

Pull Request Test Coverage Report for Build 8189992854

Details

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.007%) to 93.986%

---

Totals
Change from base Build 8189954330:	0.007%
Covered Lines:	674
Relevant Lines:	721

---

💛 - Coveralls