-
-
Notifications
You must be signed in to change notification settings - Fork 28.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove service helper (2) #16863
Remove service helper (2) #16863
Conversation
tests/components/climate/common.py
Outdated
@@ -0,0 +1,111 @@ | |||
"""Collection of helper methods.""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we update the description of each of these files to say that these are legacy helpers and should not be mimicked in new components.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 I've updated the descriptions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful
Description:
Additional work for: home-assistant/architecture#58
Instead of rewriting every test (what I did with #16832), I only moved the service helpers to
test/[component]/common.py
and rewrote the references, as suggested.Checklist:
tox
. Your PR cannot be merged unless tests pass