gickowtf / pixoo-homeassistant

Home Assistant Integration for Divoom Pixoo 64
MIT License
112 stars 12 forks source link

Adds some very simple tests #29

Closed ireneybean closed 6 months ago

ireneybean commented 6 months ago

I thought it might be useful to have some very basic unit tests in place since there is discussion of making changes to the config structure but still having basic backwards compatibility for at least some period of time. I also thought that this would be a good way for me to get more familiar with the existing code.

Please don't hesitate to challenge me on any decisions I have made in this code ... I'm not as familiar with python/homeassistant dev best practices as I am in other ecosystems. If I've done anything unusual here it's likely due to unfamiliarity.

This is hardly full test coverage or anything, but I figured it'd be good to get something merged and iterate from there. Thanks for your consideration.