[Cosmo-dev] Bug fixes and unit tests

[Ted Leung]

I wanted to highlight something that we discussed in the Cosmo  
meeting today:

Brian Moseley suggested that we check in unit test for every bug that  
we fixed. The team agreed to this. We should be using a standard  
format commit message, including the bug number and the bug  
description from Bugzilla. When closing the bug in bugzilla, we  
should be entering the revision number which fixes the bug. In order  
for that revision number to be meaningful, we should attempt to  
commit all fix related code and unit test code in a single revision.

Improving our test coverage is an investment that pays itself back  
over and over again - let's all help each other adapt to this practice.

Also,  when Chandler goes into feature freeze, all checkins must be  
code-reviewed before they can be committed -- patches containing the  
fixes are attached to bugzilla bugs and the bugzilla mechanisms are  
used to review the patches.    I think that this is a very helpful  
process and wanted to know what people thought about adopting this idea.

Ted