Unit testing about todolist HOT 12 OPEN

Great, I'll try to implement this solution, and then add some tests ✌️

from todolist.

Thank so much for your contribution @remcoe, I will review in the next 1 or 2 days.

from todolist.

Hi @amoraitis , I'm down to add some unit tests too. Any particular areas I could focus on?

from todolist.

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.

from todolist.

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:

PlatformNotSupportedException: LocalDB is not supported on this Platform.

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.

from todolist.

@maxalmonte14
Temporary solution: I think that you can use InMemoryDatabase for linux without adding it to Version Control.

I will, also, move to sqlite on the next commit

from todolist.

I'm still on this, I'm having a hard time trying to test async methods, also trying to mock dependencies like DbContext, I have to get use to C# testing tools, but eventually I will make a PR. ✌️

from todolist.

@maxalmonte14 please use docs. Start with this one

from todolist.

I created a PR :) #31

from todolist.

Hi, do you still need help with this?

from todolist.

@jezmaghuyop This will remain open, we always want better test coverage for the project.

So, feel free to add you tests.

from todolist.

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!

from todolist.