alvawei / smerge

3 stars 3 forks source link

Test.java and BiggerTest.java #6

Open sarahsz opened 6 years ago

sarahsz commented 6 years ago

The first issue is that the tests are in the smerge/src/ folder. While this doesn't break your code, it does cause confusion as to the organization of this project. This was mentioned before, so I will move on to the another issue of the naming of the Test files you have created. Test.java and BiggerTest.java doesn't really communicate to someone who may want to add on to your project what the tests are testing. This is a relatively easy fix, but still important!