[Libvirt-cim] [PATCH] [TEST] #7 Add new test to verify enum of DiskRASD to have EmulatedType=0 for Disk and EmulatedType=1 for CDROM

Guo Lian Yun yunguol at cn.ibm.com
Mon Jan 12 02:10:17 UTC 2009


libvirt-cim-bounces at redhat.com wrote on 2009-01-09 18:31:43:

> +1 for me, but see the comments inline below:
> 
> yunguol at cn.ibm.com wrote:
> > # HG changeset patch
> > # User Guolian Yun <yunguol at cn.ibm.com>
> > # Date 1231210158 28800
> > # Node ID b592961ccaac67ad3cfd8876beb22beec0c28492
> > # Parent  64b84ef28b22d17dd22027c632a9dc44d642d2b5
> > [TEST] #7 Add new test to verify enum of DiskRASD to have 
> EmulatedType=0 for Disk and EmulatedType=1 for CDROM
> >
> > Updates form 6 to 7:
> > Correct exception description
> >
> > Signed-off-by: Guolian Yun <yunguol at cn.ibm.com>
> >
> >
> > +
> > +    emu_types = [0, 1]
> > +    try:
> > +        for exp_emu_type in emu_types:
> > +            cxml = get_class(options.virt)(default_dom, 
> emu_type=exp_emu_type)
> > 
> When you update the test case for Xen and other virt types, can you 
> change the above to the following :
> virt_xml = get_class(options.virt)
> 
> cxml = virt_xml(default_dom, emu_type=exp_emu_type)
> 
  Good catch. This sounds good to me. I will update it later.
  Thanks!
> 
> -- 
> Thanks and Regards,
> Deepti B. Kalakeri
> IBM Linux Technology Center
> deeptik at linux.vnet.ibm.com
> 
> _______________________________________________
> Libvirt-cim mailing list
> Libvirt-cim at redhat.com
> https://www.redhat.com/mailman/listinfo/libvirt-cim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-cim/attachments/20090112/f1c77956/attachment.htm>


More information about the Libvirt-cim mailing list