[Libvirt-cim] [PATCH] [TEST] (#2) LXC currently doesn't support VNC, so skip appropriated tests

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Mon Nov 30 21:15:15 UTC 2009


Richard Maciel wrote:
> # HG changeset patch
> # User Richard Maciel <rmaciel at linux.vnet.ibm.com>
> # Date 1259357082 7200
> # Node ID 9e0d6ff3f5f6ec6653059ef1422ed6b70f8728f5
> # Parent  5467bdbfdaf8a3716cbbf489410b15025b23c7ce
> [TEST] (#2) LXC currently doesn't support VNC, so skip appropriated tests

These look good. Although, no need to have the message span multiple 
lines.  For each test case, you can do the following instead:

logger.info("VNC is not supported by LXC, hence skipping the tc ....")

This will fit within 80 character limit.   And looks cleaner, since you 
don't have a string spanning multiple lines.

-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list