set-user-password: Refine the code logic

1. Remove the code of logging in guest with old password as it is
   meaningless
2. Move the restore vm logic to finally block

Signed-off-by: Fangge Jin <[email protected]>

Author: fangge1212

libvirt/tests/src/virsh_cmd/domain/virsh_set_user_password.py

@@ -92,26 +92,13 @@ def run(test, params, env):
                 logging.debug(e)
                 test.fail("Failed to login with new password")
 
-            # Login with old password
-            try:
-                session = remote.wait_for_login("ssh", vm_ip, "22", set_user_name, ori_passwd,
-                                                r"[\#\$]\s*$", timeout=10)
-                session.close()
-            except remote.LoginAuthenticationError:
-                logging.debug("Login with old password failed as expected.")
-
-            # Change the password back in VM
-            ret = virsh.set_user_password(vm_name, set_user_name, ori_passwd, False,
-                                          option=option, debug=True)
-            libvirt.check_exit_status(ret)
-
-            # Login with the original password
-            try:
-                session = remote.wait_for_login("ssh", vm_ip, "22", set_user_name, ori_passwd,
-                                                r"[\#\$]\s*$", timeout=30)
-                session.close()
-            except remote.LoginAuthenticationError as e:
-                logging.debug(e)
+    finally:
+        # Recover VM
+        if vm.is_alive():
+            # always restore root password in case previously case execution is broken
+            if status_error != "yes":
+                virsh.set_user_password(vm_name, set_user_name, ori_passwd, False,
+                                        option=option, debug=True)
 
             if start_ga:
                 # Stop guest agent in vm
@@ -126,12 +113,5 @@ def run(test, params, env):
                     test.error("Deleting user '%s' got failed: '%s'" %
                                (set_user_name, output))
                 session.close()
-    finally:
-        # Recover VM
-        if vm.is_alive():
-            # always restore root password in case previously case execution is broken
-            if status_error != "yes":
-                virsh.set_user_password(vm_name, set_user_name, ori_passwd, False,
-                                        option=option, debug=True)
             vm.destroy(gracefully=False)
         vmxml_bak.sync()