<html><body>
<p><tt><br>
> > + <br>
> > + except Exception, detail:<br>
> > + logger.error("Exception: %s", detail)<br>
> > + status = FAIL<br>
> <br>
> Instead of calling undefine before each call to raise Exception(), why <br>
> not put the undefine call in the except block? This means you'll only <br>
> need to call undefine() once for any exception. And if this test case <br>
> is extended in the future, it'll safe guard against an accidental <br>
> exclusion of the undefine() call when an exception occurs.</tt><br>
<br>
<tt>There is an exception raised if define fails. We don't want to call undefine if the VM wasn't defined.</tt><br>
<br>
<tt>-Sharad<br>
<br>
</tt></body></html>