Hi Sue,
The changes look OK to me.
Can you get a comment for the setUpClass() function to explain
why that setup is necessary? Now, I know
you are trying make sure com.is_multihomed() function always returns
False for the tests because I read the bug. In the future, people who
is reading the code might not understand.
Thanks,
--Karen
On 09/12/11 13:51, Sue Sohn wrote:
Can I get a code review of the changes for:
7089691 unit_test for create_service shouldn't assume installadm is
installed
Webrev:
https://cr.opensolaris.org/action/browse/caiman/sohn/7089691/webrev.7089691
Thanks,
Sue
_______________________________________________
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
