Skip to content
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

feat: add min_to_full_charge attribute #505

Merged
merged 4 commits into from
Feb 9, 2023

Conversation

InTheDaylight14
Copy link
Contributor

Passing minutes_to_full_charge as an attribute of time_charge_complete sensor.

Still investigating the behavior seen in #497

@InTheDaylight14
Copy link
Contributor Author

  /home/runner/work/tesla/tesla/.venv/lib/python3.10/site-packages/_pytest/python.py:184: PytestUnhandledCoroutineWarning: async def functions are not natively supported and have been skipped.
  You need to install a suitable plugin for your async framework, for example:
    - anyio
    - pytest-asyncio
    - pytest-tornasync
    - pytest-trio
    - pytest-twisted
    warnings.warn(PytestUnhandledCoroutineWarning(msg.format(nodeid)))

I added a mark in the test_sensor.py so the tests added in this PR are not skipped. I'm sure there is a better way to mark all the tests so they are not skipped.

@alandtse
Copy link
Owner

alandtse commented Feb 9, 2023

Oh, I think we just need to add pytest-asyncio or something to the dependencies for pytest.

@alandtse alandtse merged commit 316ad90 into alandtse:dev Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants