[Libvirt-cim] [PATCH 0 of 4] Updating 01_forward.py of EAFP.

Kaitlin Rupert kaitlin at linux.vnet.ibm.com
Fri Jul 11 21:28:28 UTC 2008


Checking the capabilities and reserved of the resource pools is a good 
idea, but I think it might be a better approach to put these checks in 
ResourcePool/enum_01.py.

EAFP tests should be concerned with verifying that the pool returned by 
the association is the pool we expected for the given device.

While the properties of the resource pool instance need to be valid, I 
think checking these values clutters this test and makes it difficult to 
read.
-- 
Kaitlin Rupert
IBM Linux Technology Center
kaitlin at linux.vnet.ibm.com




More information about the Libvirt-cim mailing list