mauricioaniche / sem-pacman-python

PacMan game used in the Software Engineering Methods (CS minor, TU Delft)
12 stars 22 forks source link

Add unittests, in which the two failing tests are also fixed #6

Closed Tstienstra closed 4 years ago

Tstienstra commented 4 years ago

Since unittests is used nowadays in this course, it seemed good to add an unittest version of the tests as well. Also fixed the two failing tests in that one. Problem was that play gave back 3 arguments which got assigned to 2.

mauricioaniche commented 4 years ago

@Tstienstra great PR! Does this mean we can get rid of the pytest tests? I'm totally fine if we delete them!

Would you mind doing it? (All you need to do is to do the changes in your branch, and it will get automatically reflected in this PR!)

Tstienstra commented 4 years ago

@mauricioaniche Also deleted the pytest tests.