It's good to test this blank-name  case.  Doesn't <code>virsh_instance.dumpxml(vm_name)</code> throw an exception or something if the <code>vm_name</code> is blank?  Maybe should it?  I'm wondering out-loud if maybe this would be more helpful as a <code>self.assertRaises()</code> statement.  Just a suggestion.