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

Introduce assert_receive_tesla_call #724

Open
visciang opened this issue Oct 25, 2024 · 3 comments
Open

Introduce assert_receive_tesla_call #724

visciang opened this issue Oct 25, 2024 · 3 comments

Comments

@visciang
Copy link
Contributor

visciang commented Oct 25, 2024

Hi!

May I ask you if it's possible to introduce a Tesla.Test.assert_receive_tesla_call() along the line of Test.Test.assert_received_tesla_call().

I have some asynchronous code I'm test an it would be nice to assert_receive instead of just assert_received.

Maybe it could have also an additional optional timeout option to be passed to the underlying assert_receive.

@yordis
Copy link
Member

yordis commented Oct 25, 2024

PR welcome; I kept it minimal based on immediate needs! Please let me know if you are capable to contribute, I can help you with it; otherwise, I will get to this eventually, trying to clean up as much as I can at the moment

@visciang
Copy link
Contributor Author

PR welcome; I kept it minimal based on immediate needs! Please let me know if you are capable to contribute, I can help you with it; otherwise, I will get to this eventually, trying to clean up as much as I can at the moment

Perfect!
I'll open a PR shortly, feel free to adjust as you like

@visciang
Copy link
Contributor Author

FYI: #725

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

No branches or pull requests

2 participants