alvawei / smerge

3 stars 3 forks source link

Testing is all over the place #2

Open Juliusc01 opened 6 years ago

Juliusc01 commented 6 years ago

Tests are in different places: conflicts/test, reports/tests/test, src/test/java/smerge/test, and I can’t easily tell what test is for what, or why they are seperated . It would be nice if in each test you commented what specifically it was testing, this would make it clearer and also might help realize if you have redundant tests, and centralize the tests you have.

connerknight commented 6 years ago

decided to look at this to make sure your tests were clear and centrally located for ease of access.

sarahsz commented 6 years ago

perhaps having a centralized test directory would be the best approach ~ when looking for tests the first place that people look is typically in the src/test folder