-
Notifications
You must be signed in to change notification settings - Fork 128
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 testing #1
Comments
Hi buddy, I like your project, looks very nice indeed 👌 I would like to contribute to it, especially in this particular issue (which is testing coverage). I have a little question about it, I cloned the project and when I tried to run it, just exploded 😅 I'm using an Ubuntu 16.04 based Linux distribution and I got the following error after logging in:
Does that mean that the app doesn't run on Linux? The answer looks pretty obvious to me, it would be nice adding Linux support, Thanks to .NET Core should be easy peasy. |
@maxalmonte14 I will, also, move to sqlite on the next commit |
Great, I'll try to implement this solution, and then add some tests ✌️ |
I'm still on this, I'm having a hard time trying to test async methods, also trying to mock dependencies like |
@maxalmonte14 please use docs. Start with this one |
I created a PR :) #31 |
Thank so much for your contribution @remcoe, I will review in the next 1 or 2 days. |
Hi, do you still need help with this? |
@jezmaghuyop This will remain open, we always want better test coverage for the project. So, feel free to add you tests. |
Hi @amoraitis , I'm down to add some unit tests too. Any particular areas I could focus on? |
Hi, please check for areas not covered yet. I am pretty sure that the TodoItemService is not covered. |
Hi @amoraitis. Will you please assign this issue to me? I am using #hacktoberfest to get into the OSS community. I think this is a great task for me to work on. BTW, can you also create an issue item to port the code to dot net 7? If you create that issue, you can assign that one to me too. Thanks! |
**Is your feature request related to a problem?
We need unit tests to cover the project with.
Currently covered
The text was updated successfully, but these errors were encountered: