Re: [Libvirt-cim] [PATCH] [TEST] #3 Fix HostSystem-01_enum.py to work with sblim base provider installed and without

> if ret == PASS and len(hs) == 0: > + name = 'Linux_ComputerSystem' > + system = linux_cs > + if ret == FAIL and len(hs) == 1: > + name = hostname > + system = hs[0] > + if system.CreationClassName != name or system.Name != host: > + logger.error("Exp %s, got %s" % (name, > system.CreationClassName)) > + logger.error("Exp %s, got %s" % (host, system.Name)) > + status = FAIL > + else: > + logger.info("%s is %s" % (name, host)) > + status = PASS > + > + if ret == PASS and len(hs) == 1: > + if linux_cs.CreationClassName != 'Linux_ComputerSystem' \ > + or linux_cs.Name != host \ > + or hs[0].CreationClassName != hostname\ > + or hs[0].Name != host: > + logger.error("Exp Linux_ComputerSystem, got %s" % > linux_cs.CreationClassName) > + logger.error("Exp %s, got %s" % (host, linux_cs.Name)) > + logger.error("Exp %s, got %s" % (hostname, > hs[0].CreationClassName)) > + logger.error("Exp %s, got %s" % (host, hs[0].Name)) > + status = FAIL > When sblim-base-provider is installed, I dont think we will have HostSystem and Linux_ComputerSystem both returning the info. If both of them return values, then in that case the above check should return a FAIL instead of verifying the values, thoughts ?? Also, if my understanding is not complete, atleast the above checks should be separated.