search

CaptainCoderOrg / CapMan

A Pac Man Inspired Game for Learning and Fun!
3 stars 4 forks source link

Add More Tests #45

Closed DerekGooding closed 6 months ago

DerekGooding commented 6 months ago

Coverage is low. Create more tests for existing methods and codebase.

jcollard commented 6 months ago

I feel this issue is too vague to implement. I recommend identifying specific pieces of logic that are untested and reopening issues targeting those specific pieces of logic.

DerekGooding commented 6 months ago

Fair. You mentioned it on stream but I made a poor go at the specifics.

jcollard commented 6 months ago

Perhaps this Issue could be rewritten as a task to provide test coverage statistics on build? Then, we would have a better idea of where we should focus tests.

DerekGooding commented 6 months ago

Now you know why I was asking you about how you felt about test coverage on stream. You seemed against it. But yes, I'm down with this as a first step. I'll add a new issue to capture this idea.