----- Original Message -----
From: "Yu Mingfei" <yuming...@cn.fujitsu.com>
To: "Chris Evich" <cev...@redhat.com>, "Lucas Meneghel Rodrigues" 
<l...@redhat.com>, "Alex Jia" <a...@redhat.com>
Cc: autotest@test.kernel.org
Sent: Tuesday, July 10, 2012 8:56:39 AM
Subject: [PATCH 0/3]Add virsh domname testcases for libvirt

Virsh domname can get vm's name by its running-id or uuid.
These patchs are added to test it.


Right, but rename is irrelevant with virsh domname, as you said, this patchset 
should 
test if virsh domname can get vm's name by its running-id or uuid. IMHO, you 
should
add get_name() function or something like that into VM class then write test 
case to
verify it.

Alex


Yu Mingfei (3):
   virt/libvirt_vm.py: add rename function to VM class
   virt/subtests.cfg.sample: add testcases to config file
   client/tests/libvirt/tests/virsh_domname.py: add test file for virsh domname

  client/tests/libvirt/tests/virsh_domname.py |  122 +++++++++++++++++++++++++++
  client/virt/libvirt_vm.py                   |   60 +++++++++++++
  client/virt/subtests.cfg.sample             |   56 ++++++++++++
  3 files changed, 238 insertions(+), 0 deletions(-)
  create mode 100644 client/tests/libvirt/tests/virsh_domname.py


-- 
Best Regards
Yu Mingfei

_______________________________________________
Autotest mailing list
Autotest@test.kernel.org
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to