search

spiceqa / autotest

Autotest - Fully automated tests on Linux
http://autotest.github.com/
Other
5 stars 2 forks source link

Spice: Copy & Paste Tests #27

Closed vi-patel closed 12 years ago

vi-patel commented 12 years ago

Adding 6 Tests to test spice vdagent by testing positive and negative scenarios for copying and pasting between the client and guest after a spice connection.

Also adding the spice copy-paste-disabled spice option for qemu-kvm.

krcmarik commented 12 years ago

Hi Vimal, I am looking at the commits, so far I've looked at format and I did following changes to cb.py and rv_copyandpaste.py scripts - http://fpaste.org/WZ4u/ and pylint score was raised to 8.16 from 5 for rv_copyandpaste.py and 9.05 for cb.py, mostly it's just divided lines not to have lines over 80 chars, removing unused vars and imports, not using capitals in function names, changing format of logging.info function parameters, etc.) You may consider to do the changes. I will test more. Thanks!

vi-patel commented 12 years ago

I found an erroneous line in kvm_vm.py, I will need to resubmit that also.

krcmarik commented 12 years ago

Applied to upstream/next, Well done, Thanks.