-
Notifications
You must be signed in to change notification settings - Fork 36
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
Unit tests #58
Comments
I can get started on this 👍 |
Do you have a preferred framework for the unit test? Xunit, Nunit, MStest? |
I can start this too! Please let us know a preferred framework. |
To add, I think it would help with organization if we moved the code into a |
Ok. Let's do this. |
Anyone else interested in adding Unit tests? The next one I am planning is Tests for WeatherUtilities Edit: Unit tests and new hint pattern for missing creds done in #85 |
I will work on Kanye |
@jinkim-tech Kanye is done already in pr #83, but there are plenty of utilities which could use the same treatment and should be similar :) For instance, CountryInfo, Covid19Info, and a number of the other utilities are all pretty similar and need tests too |
Checklist for Utilities and whether or not they have tests
|
Please feel free to add unit tests, thank you!
The text was updated successfully, but these errors were encountered: