Re: [Libvirt-cim] [PATCH] [TEST] #3 Fix ECTP 01_forward.py to support system with multiple networks defined

> - verify_ectp_list[hs_ccn] = cs_fields > + cn_names = ["VirtualSystemMigrationService", "ComputerSystem", > "DiskPool", > + "NetworkPool", "ProcessorPool", "MemoryPool"] We Can omit the inclusion of "DiskPool","NetworkPool", "ProcessorPool" for libvirt-cim < 680. Otherwise the test will fail when we check for the list for the values after the checks for the values returned from the association is made.

> + if status == PASS: > + for k, l in verify_ectp_list.iteritems(): > + if len(l) != 0: > + raise Exception("%s items weren't returned: %s", > k, l) The above Exception should be : raise Exception("%s items weren't returned: %s" %( k, l)) otherwise, the exception statement is will be printed as follow: Exception: ("%s items weren't returned: %s", 'Xen_ProcessorPool', [CIMInstance(classname=u'Xen_ProcessorPool', ...)]) > + status = FAIL > This tc gives a false positive with libvirt-cim version < 680 . The status = FAIL should come before the raise exception. After moving the status= FAIL before the raise statement the tc fails with the following error: ERROR - Exception: ("%s items weren't returned: %s", 'Xen_ProcessorPool', [CIMInstance(classname=u'Xen_ProcessorPool', ...)])