We have a unit test, but this code is only executed when dry run isn't specified, and the tests have to be run in dry run mode.

thanks,
ginnie

On 11/30/10 13:18, Alok Aggarwal wrote:
Hi Ginnie,

Looks fine to me.

Should there be a unittest that catches breakages
in this code down the line?

Thanks,
Alok

On Tue, 30 Nov 2010, Ginnie Wray wrote:

Hi -

I've posted a code review for the following CR:
http://monaco.sfbay.sun.com/detail.jsf?cr=7003323


Webrev located at:

http://cr.opensolaris.org/~ginnie/7003323/

Alok has verified that the fix works. I've tested against the nose test suite.

thanks,
ginnie
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to