search

alterm4nn / ChronoZoom

ChronoZoom is an interactive timeline for all of history.
http://www.chronozoom.com
160 stars 127 forks source link

Tours API: Write Unit Tests #636

Open charuch opened 11 years ago

charuch commented 11 years ago

Develop unit tests to test the new 2 verb PUT and the existing DELETE Tours operations.

M3tan3rd commented 11 years ago

Is UW intending on doing this?

jilarson commented 11 years ago

Yes. Charu is looking at this. On Jun 18, 2013 2:58 PM, "Peter S. Larsen" notifications@github.com wrote:

Is UW intending on doing this?

— Reply to this email directly or view it on GitHubhttps://github.com/alterm4nn/ChronoZoom/issues/636#issuecomment-19645410 .

javierluraschi commented 11 years ago

Triage: On the last triage session is was commented that this cut be won't-fixed. There are multiple code areas that will benefit from additional unit tests, therefore, it is still encouraged in the team; however, is too much overhead to add one bug for each unit-tests that could be added.

mirounga commented 11 years ago

Thou shalt not commit the code without unit test. How do you know it is working?