On 03/01/2012 02:19 PM, guyanhua wrote: > This patch adds four test cases for "virsh uri" command. > > Use four cases:(1) Call virsh uri > (2) Call virsh -c remote_uri uri > (3) Call virsh uri with an unexpected option > (4) Call virsh uri with libvirtd service stop > > Signed-off-by: Gu Yanhua<[email protected]> > --- > client/tests/libvirt/tests/virsh_uri.py | 55 > +++++++++++++++++++++++++++++++ > 1 files changed, 55 insertions(+), 0 deletions(-) > create mode 100644 client/tests/libvirt/tests/virsh_uri.py > > diff --git a/client/tests/libvirt/tests/virsh_uri.py > b/client/tests/libvirt/tests/virsh_uri.py > new file mode 100644 > index 0000000..a239cd4 > --- /dev/null > +++ b/client/tests/libvirt/tests/virsh_uri.py > @@ -0,0 +1,55 @@ > +import logging > +from autotest_lib.client.common_lib import utils, error > +from autotest_lib.client.virt import libvirt_vm > + > +def run_virsh_uri(test, params, env): > + """ > + Test the command virsh uri > + > + (1) Call virsh uri > + (2) Call virsh -c remote_uri uri > + (3) Call virsh uri with an unexpected option > + (4) Call virsh uri with libvirtd service stop > + """ > + > + def virsh_uri(cmd): > + cmd_result = utils.run(cmd, ignore_status=True) > + logging.debug("Output: %s", cmd_result.stdout.strip()) > + logging.debug("Error: %s", cmd_result.stderr.strip()) > + logging.debug("Status: %d", cmd_result.exit_status) > + return cmd_result.exit_status, cmd_result.stdout.strip() > + > + # Prepare libvirtd service > + check_libvirtd = params.has_key("libvirtd") > + if check_libvirtd: > + libvirtd = params.get("libvirtd") > + if libvirtd == "off": > + libvirt_vm.service_libvirtd_control("stop") > + > + # Run test case > + option = params.get("options") > + check_target_uri = params.has_key("target_uri") > + if check_target_uri: > + target_uri = params.get("target_uri") > + cmd = "virsh -c %s uri" % target_uri > + else: > + cmd = "virsh uri %s" % option > + > + status, uri_test = virsh_uri(cmd) > + > + # Recover libvirtd service start > + if libvirtd == "off": > + libvirt_vm.service_libvirtd_control("start") > + > + # Check status_error > + status_error = params.get("status_error") > + if status_error == "yes": > + if status == 0: > + raise error.TestFail("Command 'virsh uri %s' succeeded " > + "(incorrect command)" % option) We probably need also to record some expected errors case, for example: else: logging.info("xxxxxxxxxxx This is a expected error xxxxxxxx") > + elif status_error == "no": > + if cmp(target_uri, uri_test) != 0: > + raise error.TestFail("Virsh cmd gives wrong uri.") > + if status != 0: > + raise error.TestFail("Command 'virsh uri %s' failed " > + "(correct command)" % option) elif status == 0: logging.info("xxxxxx succeed to run xxxxxxxxx")
I found we haven't any 'info' information is written into debug/xxxx.xxx.xxxx.INFO. Others are fine for me. Thanks, Alex > -- > 1.7.1 > > _______________________________________________ > Autotest mailing list > [email protected] > http://test.kernel.org/cgi-bin/mailman/listinfo/autotest _______________________________________________ Autotest mailing list [email protected] http://test.kernel.org/cgi-bin/mailman/listinfo/autotest
