search

doudz / homeassistant-zigate

zigate component for Home Assistant
MIT License
48 stars 15 forks source link

Ajout du config_flow et gestions des entités et des devices #101

Closed cyr-ius closed 3 years ago

cyr-ius commented 4 years ago

je vous propose l'ajout du config_flow ainsi que des options associés pour suivre la philosphie HomeAssistant J'ai associé le composant zigate.zigate au device fictif zigate J'ai ensuite associété tous les devices découverts à zigate et j'ai rattaché tous les sensors sous chaques devices. si vous souhaitez testé avant de merger , j'ai crée un package sur mon dépot ce qui permet de l'ajouter dans le plugin HACS

le problème identifié est la gestiion d'erreur en cas d'absence de la zigate. Je suis obligé de trapper toutes les exceptions avec un except: global , hors cela n'est pas autorisé dans les tests travis.

Tout le code ayant attrait aux sensors n'a pas été modifiés j'ai juste rajouté la propriété device_info pour faire le rattachement au device

Quelques captures d'écrans: image image image

image image image

doudz commented 4 years ago

Très bon boulot ! Il a un certain nombre de conflit, merci de faire un rebase

cyr-ius commented 4 years ago

J'ai rebasé et nettoyer tous les conflits

doudz commented 4 years ago

merci de corriger les erreurs signalé par Travis

cyr-ius commented 4 years ago

merci de corriger les erreurs signalé par Travis

J'ai corrigé l'erreur d'espace manquant par contre comme je le notais plus haut il me restera toujours l'erreur lié à l'exception dans les test Travis. Car j'ai ajouter un Try Except pour catcher l'éventuel indisponilité du zigate mais impossible d'attraper l'exception précise ZIGATE_NOT_FOUND. J'ai donc été obligé de faire un except global , ce que les tests Travi considère comme incorrect

./custom_components/zigate/config_flow.py:108:9: E722 do not use bare 'except' ./custom_components/zigate/init.py:133:5: E722 do not use bare 'except'

Comment faire mieux ?

doudz commented 3 years ago

closing since not updated