[Libvirt-cim] [PATCH] [TEST] Fixing 40_RSC_start.py tc

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Thu Jun 19 17:31:23 UTC 2008


>          if rc != 0:
> -            rc = XFAIL_RC(bug)
> -            raise Exception("Unable start dom %s using RequestedStateChange()", 
> -                            default_dom)
> +            status = XFAIL_RC(bug)
> +            raise Exception("RequestedStateChange() could not be used to start "
> +                            "domain:", default_dom)
> +

I know this was originally in the code (and my mistake too, I believe), 
but can you fix the strings the the raise call to use %s instead of a comma:

raise Exception("RequestedStateChange() could not be used to start " 

                 "domain: %s" % default_dom)

As the code is now, it prints like:

('RequestedStateChange() could not be used to start domain:', 'test_domain')

This isn't bad, but the following is cleaner:

('RequestedStateChange() could not be used to start domain: test_domain')

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




More information about the Libvirt-cim mailing list