dalinicus / homeassistant-acinfinity

AC Infinity integration for Home Assistant for UIS based controllers
MIT License
75 stars 4 forks source link

Switch to using an update coordinator instead of manual refreshs #23

Closed dalinicus closed 11 months ago

dalinicus commented 11 months ago

boy howdy was this more involved than I expected

Also updated tests to be more efficient by sharing some of the logic

resolves #22

codecov[bot] commented 11 months ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b350c15) 100.00% compared to head (78abbe2) 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #23 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 11 10 -1 Lines 448 478 +30 ========================================= + Hits 448 478 +30 ``` | [Files Changed](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson) | Coverage Δ | | |---|---|---| | [custom\_components/ac\_infinity/\_\_init\_\_.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/ac\_infinity.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvYWNfaW5maW5pdHkucHk=) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/binary\_sensor.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvYmluYXJ5X3NlbnNvci5weQ==) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/const.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvY29uc3QucHk=) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/number.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvbnVtYmVyLnB5) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/select.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvc2VsZWN0LnB5) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/sensor.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvc2Vuc29yLnB5) | `100.00% <100.00%> (ø)` | | | [custom\_components/ac\_infinity/time.py](https://app.codecov.io/gh/dalinicus/homeassistant-acinfinity/pull/23?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Ryan+Mattson#diff-Y3VzdG9tX2NvbXBvbmVudHMvYWNfaW5maW5pdHkvdGltZS5weQ==) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.