-
Notifications
You must be signed in to change notification settings - Fork 84
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
ITwitchAPI is not mockable #83
Comments
Pretty sure its in the works. Latest nuget doesnt have the ability. V3 when its ready should. @lucky can verify as he is working on it. |
I think you tagged the wrong person |
.... Sry ya lucky7even . tagging on mobile sucks |
Hi @Mabiro we are working on the testability of the library at the moment. You will find in the Dev branch there is a test project going in. You are right about the concrete items and we are working on those as well. The library wasn't designed to be testable from the beginning. Slowly improving. |
Thank you for the quick reply. I'll take a look at the dev branch! |
I am looking forward to using TwitchLib.Api for a project I am currently working on, but it does look like ITwitchAPI can't correctly be mocked and used in unit tests.
Having ITwitchAPI define concrete type as properties makes it impossible to mock the underlying implementation.
Am I missing something here? Are there plans to make it more unit-test friendly?
The text was updated successfully, but these errors were encountered: