[User Story] As a Developer, I want better unit tests on logical classes to reduce the chance of re-introducing a bug

[srjheam]

As previously discussed in #29, improving tests and reducing code repetition is increasingly necessary.

[nickmartin1ee7]

A unit test where attempting to add two projects would be beneficial.

[srjheam]

A unit test where attempting to add two projects would be beneficial.

Thanks, I'll make sure to do that.

[srjheam]

I was basing my tests on Code2Gether-Discord-Bot.Library.Models.ProjectManager class but this class is always called by Code2Gether-Discord-Bot.Library.BusinessLogic namespace's classes, so @nickmartin1ee7 I'd to know whether should I basing my tests in each BusinessLogic classes, ProjectManager classes or both?

[nickmartin1ee7]

Specifically I am testing the logic for the commands in #29.
However, ProjectManager should probably have tests too. But that is something that can be improved in a different branch for this story.

[srjheam]

ATM I'm writing the tests for the Model.ProjectManager class and refactoring the way to create fake/mock/stub objects.
I'm gonna share my work soon.

[srjheam]

Why was this problem closed, @nickmartin1ee7? I thought it was about refactoring all the tests, at least I was doing it.

[nickmartin1ee7]

Closed automatically. Reopening.