Check for terminated processes properly
This commit is contained in:
parent
b7bef16137
commit
78e6deae3a
3 changed files with 20 additions and 13 deletions
|
|
@ -15,9 +15,11 @@ sub run {
|
|||
|
||||
# Remove the user clear the system and force the gnome-initial-setup
|
||||
# to start on new boot.
|
||||
assert_script_run("loginctl kill-user $username");
|
||||
# that might switch us back to GDM, switch away if so
|
||||
assert_script_run("loginctl kill-user $username", timeout => 120);
|
||||
# closing the user might switch us back to GDM, let's go back there
|
||||
$self->root_console(tty => 3) if (check_screen 'graphical_login', timeout => 30);
|
||||
# repeat checking that all processes are terminated, then proceed to delete the user
|
||||
assert_script_run("(for i in {1..30}; do ! pgrep -u $username >/dev/null && exit 0; sleep 1; done; pgrep -au $username; exit 1)");
|
||||
assert_script_run("userdel -r $username");
|
||||
|
||||
# Reboot the system to trigger GNOME initial setup
|
||||
|
|
|
|||
|
|
@ -55,13 +55,16 @@ sub handle_privacy {
|
|||
record_soft_failure("Geoservices are not enabled by default which was expected.");
|
||||
assert_and_click("gis_radio_geoservice_enable");
|
||||
}
|
||||
unless (check_screen("gis_reporting_enabled", 2)) {
|
||||
record_soft_failure("Autoreporting is not enabled by default which was expected.");
|
||||
assert_and_click("gis_radio_reporting_enable");
|
||||
# Skip this test on Silverblue
|
||||
unless (get_var("CANNED")) {
|
||||
unless (check_screen("gis_reporting_enabled", 2)) {
|
||||
record_soft_failure("Autoreporting is not enabled by default which was expected.");
|
||||
assert_and_click("gis_radio_reporting_enable");
|
||||
}
|
||||
}
|
||||
|
||||
# Disable automatic reporting to test that we can turn options off
|
||||
assert_and_click("gis_radio_reporting_disable");
|
||||
assert_and_click("gis_radio_geoservice_disable");
|
||||
|
||||
assert_and_click("next_button");
|
||||
}
|
||||
|
|
|
|||
|
|
@ -82,10 +82,10 @@ sub run {
|
|||
# Optional extra proof on the DNF side:
|
||||
assert_script_run(q{dnf repolist --enabled | grep -E 'fedora-cisco-openh264|google-chrome|rpmfusion'});
|
||||
|
||||
# Check the geolocation service are enabled for the user
|
||||
# Check the geolocation service are disabled for the user (the previous test did it)
|
||||
assert_as_user_eq($user,
|
||||
q{gsettings get org.gnome.system.location enabled},
|
||||
'true'
|
||||
'false'
|
||||
);
|
||||
|
||||
# Do not check for keyboard layout and locales because
|
||||
|
|
@ -106,11 +106,13 @@ sub run {
|
|||
}
|
||||
|
||||
|
||||
# Check that automatic reporting is disabled.
|
||||
assert_as_user_eq($user,
|
||||
q{gsettings get org.gnome.desktop.privacy report-technical-problems},
|
||||
'false'
|
||||
);
|
||||
# Check that automatic reporting is enabled when not in Silverblue.
|
||||
unless (get_var('CANNED')) {
|
||||
assert_as_user_eq($user,
|
||||
q{gsettings get org.gnome.desktop.privacy report-technical-problems},
|
||||
'false'
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
sub test_flags {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue