Fix failure during loading due to MQTT init

[zvirja]

Fixes #238

Add MQTT to dependencies to make sure that component is loaded after it, so initialization does not fail

[zachowj]

https://developers.home-assistant.io/docs/creating_integration_manifest/#after-dependencies

[zvirja]

@zachowj Yeah, also saw that hint in the CI output. Brilliant!! I've tested it locally and it seems to work as a charm 😊

[zvirja]

@zachowj I believe that I fixed the ordering the that validation will be fine. I however don't know why the test is failing - it does not seem to be related to the change I did. Could you please help with that?

Thank you!

[zachowj]

Yeah, testing is broken at the moment. Your latest push removed the previous commit code.

[zvirja]

Yeah, I force pushed to clean up the mess and the previous approach I took. Feel free to add more to PR if needed - I am done 😊

[zachowj]

Oh sorry, I didn't follow. I thought awaiting for the mqtt to load was still needed in the setup, but I guess after_dependences handles that outside the code.

[zachowj]

thx

[zvirja]

Exactly, now HASS does it for us, so the fix is neat. If you find other issues like this - simply add them as well to after_deps - that should be scalable until you add all of them!

Thank you very much for the fast feedback loop. I appreciate it a lot! ❤️