ucsb-cs56-projects / cs56-games-minesweeper

-
MIT License
0 stars 10 forks source link

Update JUnit tests (might require code refactoring) #11

Closed kjorg50 closed 8 years ago

kjorg50 commented 10 years ago

There are multiple cases of unit tests in GridTest.java that are empty and contain comments like //not able to test due to.... Either these "skeleton" tests should be removed, or you should refactor the code in a way that makes it more easy to test. See some of Andrew Berls' comments on pull request #5 for some helpful comments

~estimated 200

jaeaster commented 8 years ago

W16 approved