-
Notifications
You must be signed in to change notification settings - Fork 62
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
Fixes for initialization of heated seats #189
Conversation
Can you please add a test for the case where it broke before where the data isn't initialized yet? |
We're close to pushing a new release to address the login errors. Any chance to get that test in place so I can merge this too? |
Sorry for the delayed reply. I think the changes I made on 47e5f23 in test_heated_seat.py (line 33) address this. It's the same approach used by the initialization tests for the other components. |
I mean do we have a test in place to make sure we don't get the exception on startup #186 ? In theory after your fixes, you won't get the exception but this is to prevent a regression in the future. |
The test I reference would catch that. If anything sets the car state during initialization that assertion will fail. |
Ok. Thanks! |
Fixes for #186