Catch incomplete read and log which coil failed

[elupus]

This should allow easier fault tracing of issues with pumps crashing

See: home-assistant/core#118598

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.96%. Comparing base (7a4c599) to head (e6e8b15).
Report is 15 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   57.84%   57.96%   +0.12%     
==========================================
  Files          14       14              
  Lines        1243     1261      +18     
==========================================
+ Hits          719      731      +12     
- Misses        524      530       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[elupus]

Ps. I noticed we do retries in the modbus connection. I wonder if that make sense there?

[yozik04]

LGTM

[yozik04]

@elupus If you want to make a release please do it yourself. Bump app version in __init__.py and make a release with the same version. I am away from computer for couple weeks. 😄

[yozik04]

Ps. I noticed we do retries in the modbus connection. I wonder if that make sense there?

TCP can still fail. I think it is fine.

[elupus]

Will do. Right, forgot modbus reconnects by itself in the background. I wonder if that will hide the exception i'm looking for.