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

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Mon Dec 29 22:32:31 UTC 2008


>> The indention is off here. Also, you'll need to use % not commas. This 
>> should be:
>>
>> raise Exception("EmulatedType Mismatch: got %d, \
>> expected %d" % (rasd['EmulatedType'],
>> exp_emu_type))
> We need not have to use % , comma works. Also, we can avoid slash. The 

I thought using commas only worked for the logger() function.  For 
Exception(), I think you need to use % or else the formatting is off.

I could be wrong about that though.  Daisy - you'll want to double check 
that.

> above can be rewritten as follows:
> raise Exception("EmulatedType Mismatch: got %d, "
> "expected %d", (rasd['EmulatedType'],
> exp_emu_type))


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




More information about the Libvirt-cim mailing list