search

alaurenz / metrobike

0 stars 1 forks source link

Add unit testing for ui utility class #121

Closed dutchscout closed 11 years ago

dutchscout commented 11 years ago

We need to increase our testing coverage!

dutchscout commented 11 years ago

I got this file started with one test case, but I'm not sure whether I'll have time to finish it myself.

sw11 commented 11 years ago

Can you tell me what should I need to do about this?

dutchscout commented 11 years ago

Each of the methods (except a private constructor) need unit tests made for them, covering cases that should work, cases that may fail, edge cases, and typical cases, etc etc.

I'm not sure who will write these tests, but they need to be written this phase so we can reach our test coverage goals.

On May 19, 2013, at 1:09, sw11 notifications@github.com wrote:

Can you tell me what should I need to do about this?

— Reply to this email directly or view it on GitHubhttps://github.com/alaurenz/metrobike/issues/121#issuecomment-18114015 .

sw11 commented 11 years ago

MapSetting class cannot be tested since it needs googlemap object. But historytest class is added.