Brookke / Lorem-Ipsum-4

2017 SEPR project assessment 4
1 stars 0 forks source link

Player testing #76

Closed Brookke closed 7 years ago

Brookke commented 7 years ago

Small additions to player testing should bump our coverage up a little.

Although I am against writing additional code so that things can be tested, I have added a few try catch blocks to the player class that now output warnings if things like the speech manager are missing. I believe that this is correct behaviour as the player should still function without the speech box manager and certain parts of the game snapshot.

Brookke commented 7 years ago

In the master branch at least it takes the player from 44% coverage to 57%

beno11224 commented 7 years ago

can we just merge this then? or was there something still to do?

joeShuff commented 7 years ago

^^^^