[Libvirt-cim] [PATCH 3 of 6] [TEST] Remove try_getinstance() from HostSystem - 05_hs_gi_errs.py

# HG changeset patch # User Kaitlin Rupert <karupert(a)us.ibm.com&gt; # Date 1229639712 28800 # Node ID 6422c5ef1c750424ceea505bd8b60deb3134e6c8 # Parent 1add9163d3135bb46a0648669de97ba04796181b [TEST] Remove try_getinstance() from HostSystem - 05_hs_gi_errs.py try_getinstance() is no longer needed - this function can be implemented using functions from enumclass.py. Plus, a conn needs to be passed to the function, which is poor function design. Removed the "invalid_instid_keyname" case - passing an invalid keyname only tests the CIMOM, it does not test the providers. Signed-off-by: Kaitlin Rupert <karupert(a)us.ibm.com&gt; diff -r 1add9163d313 -r 6422c5ef1c75 suites/libvirt-cim/cimtest/HostSystem/05_hs_gi_errs.py --- a/suites/libvirt-cim/cimtest/HostSystem/05_hs_gi_errs.py Thu Dec 18 14:35:11 2008 -0800 +++ b/suites/libvirt-cim/cimtest/HostSystem/05_hs_gi_errs.py Thu Dec 18 14:35:12 2008 -0800 @@ -23,21 +23,16 @@ # Test Case Info: # -------------- # This tc is used to verify if appropriate exceptions are -# returned by Xen_HostSystem on giving invalid inputs. +# returned by HostSystem on giving invalid inputs. # -# 1) Test by passing Invalid CCName Keyname +# 1) Test by passing invalid values for the following keys: +# # Input: # ------ -# wbemcli gi 'http://localhost:5988/root/virt:Xen_HostSystem.\ -# Wrong="Xen_HostSystem",Name="x3650"' -nl +# CreationClassName +# Name # -# Output: -# ------- -# error code : CIM_ERR_NOT_FOUND -# error desc : "No such instance (CreationClassName)" -# -# 2) Test by giving Invalid CCName Keyvalue -# Input: +# Format: # ------ # wbemcli gi 'http://localhost:5988/root/virt:Xen_HostSystem.\ # CreationClassName="Wrong",Name="x3650"' -nl @@ -45,53 +40,28 @@ # Output: # ------- # error code : CIM_ERR_NOT_FOUND -# error desc : "No such instance (CreationClassName)" +# error desc : "No such instance (CreationClassName)" (varies by key) # -# 3) Test by passing Invalid Name Keyname -# Input: -# ------ -# wbemcli gi 'http://localhost:5988/root/virt:Xen_HostSystem.\ -# CreationClassName="Xen_HostSystem",Wrong="x3650"' -nl -# -# Output: -# ------- -# error code : CIM_ERR_NOT_FOUND -# error desc : "No such instance (Name)" -# -# 4) Test by giving Invalid CCName Keyvalue -# Input: -# ------ -# wbemcli gi 'http://localhost:5988/root/virt:Xen_HostSystem.\ -# CreationClassName="Xen_HostSystem",Name="Wrong"' -nl -# -# Output: -# ------- -# error code : CIM_ERR_NOT_FOUND -# error desc : "No such instance (Name)" -# -# -Date 26.02.2008 import sys -import pywbem -from VirtLib import utils -from XenKvmLib import assoc -from XenKvmLib.common_util import get_host_info, try_getinstance +from pywbem import CIM_ERR_NOT_FOUND, CIMError +from pywbem.cim_obj import CIMInstanceName +from CimTest.ReturnCodes import PASS, FAIL, SKIP +from CimTest.Globals import logger +from XenKvmLib.common_util import get_host_info from XenKvmLib.classes import get_typed_class -from optparse import OptionParser -from CimTest.ReturnCodes import PASS, FAIL, SKIP -from CimTest.Globals import logger, CIM_USER, CIM_PASS, CIM_NS from XenKvmLib.const import do_main +from XenKvmLib.enumclass import GetInstance, CIM_CimtestClass, EnumInstances sup_types = ['Xen', 'KVM', 'XenFV', 'LXC'] -expr_values = { - "invalid_ccname" : {'rc' : pywbem.CIM_ERR_NOT_FOUND, - 'desc' : "No such instance " - "(CreationClassName)" }, - "invalid_name" : {'rc' : pywbem.CIM_ERR_NOT_FOUND, - 'desc' : "No such instance (Name)" } - } - +expected_values = { + "invalid_name" : {'rc' : CIM_ERR_NOT_FOUND, + 'desc' : "No such instance (Name)" }, + "invalid_ccname" : {'rc' : CIM_ERR_NOT_FOUND, + 'desc' : "No such instance " + "(CreationClassName)" } + } @do_main(sup_types) def main(): @@ -105,57 +75,47 @@ if status != PASS: return status - classname = host_inst.CreationClassName - host_name = host_inst.CreationClassName - #Test calls GetInstance() - no need to test GetInstance() of SBLIM providers - if (classname == "Linux_ComputerSystem"): + if (host_inst.Classname == "Linux_ComputerSystem"): return SKIP - conn = assoc.myWBEMConnection('http://%s' % options.ip, - (CIM_USER, CIM_PASS), CIM_NS) + key_vals = { 'Name' : host_inst.Name, + 'CreationClassName' : host_inst.CreationClassName, + } - # 1) Test by giving Invalid CCName Key Name - field = 'INVALID_CCName_KeyName' - keys = { field : classname, 'Name' : host_name } - ret_value = try_getinstance(conn, classname, keys, field_name=field, - expr_values=expr_values['invalid_ccname'], - bug_no="") - if ret_value != PASS: - logger.error("------ FAILED: Invalid CCName Key Name.------") - return ret_value + tc_scen = { + 'invalid_name' : 'Name', + 'invalid_ccname' : 'CreationClassName', + } - # 2) Test by passing Invalid CCName Key Value - field = 'INVALID_CCName_KeyValue' - keys = { 'CreationClassName' : field, 'Name' : host_name } - ret_value = try_getinstance(conn, classname, keys, field_name=field, - expr_values=expr_values['invalid_ccname'], - bug_no="") - if ret_value != PASS: - logger.error("------ FAILED: Invalid CCName Key Value.------") - return ret_value + for tc, field in tc_scen.iteritems(): + status = FAIL - # 3) Test by giving Invalid Name Key Name - field = 'INVALID_Name_KeyName' - keys = { 'CreationClassName' : classname, field : host_name} - ret_value = try_getinstance(conn, classname, keys, field_name=field, - expr_values=expr_values['invalid_name'], - bug_no="") - if ret_value != PASS: - logger.error("------ FAILED: Invalid Name Key Name.------") - return ret_value + keys = key_vals.copy() + keys[field] = tc + expr_values = expected_values[tc] - # 4) Test by passing Invalid Name Key Value - field = 'INVALID_Name_KeyValue' - keys = { 'CreationClassName' : classname, 'Name' : field } - ret_value = try_getinstance(conn, classname, keys, field_name=field, - expr_values=expr_values['invalid_name'], - bug_no="") - if ret_value != PASS: - logger.error("------ FAILED: Invalid Name Key Value.------") - return ret_value + ref = CIMInstanceName(host_inst.Classname, keybindings=keys) - return PASS + try: + inst = CIM_CimtestClass(options.ip, ref) + + except CIMError, (err_no, err_desc): + exp_rc = expr_values['rc'] + exp_desc = expr_values['desc'] + + if err_no == exp_rc and err_desc.find(exp_desc) >= 0: + logger.info("Got expected exception: %s %s", exp_desc, exp_rc) + status = PASS + else: + logger.error("Unexpected errno %s, desc %s", err_no, err_desc) + logger.error("Expected %s %s", exp_desc, exp_rc) + + if status != PASS: + logger.error("------ FAILED: %s ------", tc) + break + + return status if __name__ == "__main__": sys.exit(main())