The biggest problem I've found trying to go back and fill in unit tests is you can either cover the obvious things that won't break unless there are API changes, but the type of stuff that more often breaks can't often be deduced from the code (not to mention you can't find existing bugs when the implementation is your spec.<br>
<br><div class="gmail_quote">On Fri, Feb 12, 2010 at 1:54 PM, Max Kanat-Alexander <span dir="ltr"><<a href="mailto:mkanat@bugzilla.org">mkanat@bugzilla.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On 02/11/2010 10:13 PM, Gabor Szabo wrote:<br>
> and the total test coverage of Bugzilla is 18.4%<br>
<br>
</div>        Yeah, just to say again what others have said, this is a problem, for sure.<br>
<div class="im"><br>
> Are there tests other than those in t/*?<br>
<br>
</div>        There are the Selenium tests that Byron pointed out--those are our main<br>
method of QA at the moment.<br>
<br>
        The nice part about the Selenium tests is that they make sure that the<br>
actual UI and the JavaScript in Bugzilla works. That is, we wouldn't<br>
really have "full coverage" unless we tested those as well.<br>
<br>
        Having unit tests for every single method and function in Bugzilla<br>
would be great, though. The fact is, the lack of TDD in our development<br>
process is one of the things that makes the review process so difficult.<br>
But the number of tests that we'd have to write to truly have full<br>
coverage is also something that's nearly overwhelming for us, because we<br>
just don't have the current manpower to do it.<br>
<br>
        If somebody wanted to work on getting Bugzilla fully unit-tested, I'd<br>
be behind it 100% of the way. (Note that to get really full coverage on<br>
the Perl code, we'd have to move all the .cgi code into Controller<br>
modules, but that's something that I've want to do  anyway.)<br>
<br>
        -Max<br>
<font color="#888888">--<br>
<a href="http://www.everythingsolved.com/" target="_blank">http://www.everythingsolved.com/</a><br>
Competent, Friendly Bugzilla and Perl Services. Everything Else, too.<br>
</font><div><div></div><div class="h5">-<br>
To view or change your list settings, click here:<br>
<<a href="http://bugzilla.org/cgi-bin/mj_wwwusr?user=ahdevans@gmail.com" target="_blank">http://bugzilla.org/cgi-bin/mj_wwwusr?user=ahdevans@gmail.com</a>><br>
</div></div></blockquote></div><br>