command-line-chess icon indicating copy to clipboard operation
command-line-chess copied to clipboard

Published 20 hours ago • verified publisher icon marcusbuffett

Adding unittests

Open ClasherKasten opened this issue 2 years ago • 4 comments

There are not really tests in this repository which makes it a bit hard to ensure there are no bugs and/or unintentional behavior.

To fix this, unittests should be written. When this is done, integration tests can be done too.

~~Right now there is only one (unnecessary) test.~~ All new tests should be created in a new directory called tests/ and because it is already in use, pytest should be used as test-framework.

ClasherKasten avatar Sep 24 '22 07:09 ClasherKasten

Hi, I am want to solve this, can you assign me?

GrbavaCigla avatar Oct 05 '22 22:10 GrbavaCigla

Looks good, do you need more tests created? Interesting project!

programmin1 avatar Oct 12 '22 04:10 programmin1

Looks good, do you need more tests created? Interesting project!

First: Nice that you find this project interesting. 👍

#46 already covers many parts of the code, but it definitely does not cover everything. So new PRs fixing that would be very welcome.

ClasherKasten avatar Oct 12 '22 05:10 ClasherKasten

Ok check out https://github.com/marcusbuffett/command-line-chess/pull/50 ?

programmin1 avatar Oct 22 '22 06:10 programmin1