Skip to content

Commit

Permalink
Merge pull request #13917 from lemon-suse/Fix-remove-test-user-timeout
Browse files Browse the repository at this point in the history
Remove test user need logout firstly
  • Loading branch information
coolgw committed Dec 23, 2021
2 parents 0108669 + a4151e2 commit b70cc5d
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions lib/services/users.pm
Expand Up @@ -102,14 +102,8 @@ sub switch_users {
# handle welcome screen, when needed
handle_welcome_screen(timeout => 120) if (opensuse_welcome_applicable);
handle_gnome_activities;
switch_user;
send_key "esc";
assert_and_click "displaymanager-$username";
assert_screen "originUser-login-dm";
# for poo#88247, we have to restore current user's password before migration,
# so here need to use the original password.
type_password(get_required_var('FLAVOR') =~ /Migration/ ? "$password\n" : "$newpwd\n");
handle_gnome_activities;
handle_logout;
wait_still_screen 10;
}

# restore password to original value
Expand All @@ -129,7 +123,6 @@ sub restore_passwd {

# remove test user
sub remove_test_user {
assert_script_run("pkill -KILL -u $newUser");
assert_script_run("userdel -r $newUser");
}

Expand Down

0 comments on commit b70cc5d

Please sign in to comment.