enforce standard coding on all Perl files

lib/anaconda.pm

@@ -178,7 +178,7 @@ sub custom_blivet_add_partition {
 sub custom_add_partition {
     # Used to add partition on Rocky custom-gui partitioning screen
     # in Anaconda. Should be called when custom-gui is displayed and free space is selected.
-    # You can pass device type for partition (needle tagged anaconda_custom_devicetype_$devicetype should exist), size of that partition in MiBs, desired filesystem of that partition
+# You can pass device type for partition (needle tagged anaconda_custom_devicetype_$devicetype should exist), size of that partition in MiBs, desired filesystem of that partition
     # (anaconda_custom_part_fs_$filesystem should exist) and mountpoint of that partition (e. g. string "/boot").
     my %args = (
         devicetype => "",
@@ -368,7 +368,7 @@ sub get_full_repo {
     if ($repourl !~ m/^(nfs|hd:)/) {
         # Everything variant doesn't exist for modular composes atm,
         # only Server
-        $repourl .= "/".get_var("ARCH")."/os";
+        $repourl .= "/" . get_var("ARCH") . "/os";
     }
     return $repourl;
 }

lib/anacondatest.pm

@@ -73,21 +73,21 @@ sub post_fail_hook {
     }
 
     if ($dracut) {
-        upload_logs "/run/initramfs/rdsosreport.txt", failok=>1;
+        upload_logs "/run/initramfs/rdsosreport.txt", failok => 1;
         # that's all that's really useful, so...
         return;
     }
 
-    upload_logs "/tmp/X.log", failok=>1;
+    upload_logs "/tmp/X.log", failok => 1;
-    upload_logs "/tmp/anaconda.log", failok=>1;
+    upload_logs "/tmp/anaconda.log", failok => 1;
-    upload_logs "/tmp/packaging.log", failok=>1;
+    upload_logs "/tmp/packaging.log", failok => 1;
-    upload_logs "/tmp/storage.log", failok=>1;
+    upload_logs "/tmp/storage.log", failok => 1;
-    upload_logs "/tmp/syslog", failok=>1;
+    upload_logs "/tmp/syslog", failok => 1;
-    upload_logs "/tmp/program.log", failok=>1;
+    upload_logs "/tmp/program.log", failok => 1;
-    upload_logs "/tmp/dnf.log", failok=>1;
+    upload_logs "/tmp/dnf.log", failok => 1;
-    upload_logs "/tmp/dnf.librepo.log", failok=>1;
+    upload_logs "/tmp/dnf.librepo.log", failok => 1;
-    upload_logs "/tmp/dnf.rpm.log", failok=>1;
+    upload_logs "/tmp/dnf.rpm.log", failok => 1;
-    upload_logs "/tmp/dbus.log", failok=>1;
+    upload_logs "/tmp/dbus.log", failok => 1;
 
     if ($has_traceback) {
         # Upload Anaconda traceback logs
@@ -131,7 +131,7 @@ sub root_console {
         # tty3 has a shell on all f31+ installer and live images
         send_key "ctrl-alt-f3";
     }
-    console_login(user=>"root", timeout=>$args{timeout});
+    console_login(user => "root", timeout => $args{timeout});
 }
 
 1;

lib/cockpit.pm

@@ -24,7 +24,7 @@ sub start_cockpit {
     assert_screen "cockpit_login", 60;
     # this happened on early Modular Server composes...
     record_soft_failure "Unbranded Cockpit" if (match_has_tag "cockpit_login_unbranded");
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     if ($login) {
         type_safely "root";
         wait_screen_change { send_key "tab"; };
@@ -57,6 +57,6 @@ sub check_updates {
     my $logfile = shift;
     sleep 2;
     my $checkresult = script_run "dnf check-update > $logfile";
-    upload_logs "$logfile", failok=>1;
+    upload_logs "$logfile", failok => 1;
-    return($checkresult);
+    return ($checkresult);
 }

lib/freeipa.pm

@@ -44,7 +44,7 @@ sub start_webui {
     disable_firefox_studies;
     type_string "startx /usr/bin/firefox -width 1024 -height 768 https://ipa001.test.openqa.fedoraproject.org\n";
     assert_screen ["freeipa_webui_login", $user_screen], 60;
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     # softfail on kerberos ticket bugs meaning we get auto-logged in
     # as the requested user when we don't expect to be
     if (match_has_tag $user_screen) {

lib/installedtest.pm

@@ -43,7 +43,7 @@ sub post_fail_hook {
         assert_script_run "dhclient";
     }
     else {
-        $self->root_console(tty=>6);
+        $self->root_console(tty => 6);
         # fix up keyboard layout, if we failed before the test did this
         # itself; if it's already been done, should be safe, will just
         # fail and carry on
@@ -88,7 +88,7 @@ sub post_fail_hook {
     }
 
     if ($dracut) {
-        upload_logs "/run/initramfs/rdsosreport.txt", failok=>1;
+        upload_logs "/run/initramfs/rdsosreport.txt", failok => 1;
         # that's all that's really useful, so...
         return;
     }
@@ -125,14 +125,14 @@ sub post_fail_hook {
     }
 
     # Sometimes useful for diagnosing FreeIPA issues
-    upload_logs "/etc/nsswitch.conf", failok=>1;
+    upload_logs "/etc/nsswitch.conf", failok => 1;
 
     if (get_var("FLAVOR") eq "updates-everything-boot-iso") {
         # for installer creation test
         script_run "df -h";
-        upload_logs "/root/imgbuild/pylorax.log", failok=>1;
+        upload_logs "/root/imgbuild/pylorax.log", failok => 1;
-        upload_logs "/root/imgbuild/lorax.log", failok=>1;
+        upload_logs "/root/imgbuild/lorax.log", failok => 1;
-        upload_logs "/root/imgbuild/program.log", failok=>1;
+        upload_logs "/root/imgbuild/program.log", failok => 1;
     }
 
     if (get_var("TEST") eq "live_build") {
@@ -149,10 +149,10 @@ sub post_fail_hook {
     }
 }
 
-    # For update tests, let's do the update package info log stuff,
+# For update tests, let's do the update package info log stuff,
-    # it may be useful for diagnosing the cause of the failure
+# it may be useful for diagnosing the cause of the failure
-    advisory_get_installed_packages;
+advisory_get_installed_packages;
-    advisory_check_nonmatching_packages(fatal=>0);
+advisory_check_nonmatching_packages(fatal => 0);
 
 1;
 

lib/modularity.pm

@@ -37,7 +37,7 @@ sub parse_module_list {
 sub is_listed {
     my ($module, $stream, $listref) = @_;
     my $found = 0;
-    foreach (@{ $listref }) {
+    foreach (@{$listref}) {
         if ($_->{module} eq $module and $_->{stream} eq $stream) {
             $found = 1;
         }

lib/utils.pm

@@ -22,9 +22,9 @@ sub run_with_error_check {
         # by using 'unless' and 'expect_not_found=>1' here we avoid
         # the web UI showing each failure to see the error message as
         # a 'failed match'
-        die "Error screen appeared" unless (wait_serial($error_screen, timeout=>5, expect_not_found=>1));
+        die "Error screen appeared" unless (wait_serial($error_screen, timeout => 5, expect_not_found => 1));
         $func->();
-        die "Error screen appeared" unless (wait_serial($error_screen, timeout=>5, expect_not_found=>1));
+        die "Error screen appeared" unless (wait_serial($error_screen, timeout => 5, expect_not_found => 1));
     }
     else {
         die "Error screen appeared" if (check_screen $error_screen, 5);
@@ -40,7 +40,7 @@ sub type_safely {
     type_string($string, wait_screen_change => 3, max_interval => 20);
     # similarity level 45 as there will commonly be a flashing
     # cursor and the default level (47) is slightly too tight
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
 }
 
 # high-level 'type this string extremely safely and rather slow'
@@ -50,7 +50,7 @@ sub type_very_safely {
     type_string($string, wait_screen_change => 1, max_interval => 1);
     # similarity level 45 as there will commonly be a flashing
     # cursor and the default level (47) is slightly too tight
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
 }
 
 sub get_release_number {
@@ -60,13 +60,13 @@ sub get_release_number {
     my $version = get_var("VERSION");
     my $rawrel = get_var("RAWREL", "Rawhide");
     return $rawrel if ($version eq "Rawhide");
-    return $version
+    return $version;
 }
 
 sub get_version_major {
     my $version = get_var('VERSION');
     my $version_major = substr($version, 0, index($version, q/./));
-  return $version_major
+    return $version_major;
 }
 
 sub get_code_name {
@@ -74,11 +74,11 @@ sub get_code_name {
     my $version = get_var('VERSION');
     my $version_major = get_version_major;
 
-    given($version_major){
+    given ($version_major) {
         when ('9') { $code_name = 'Blue Onyx'; }
         when ('10') { $code_name = 'Smoky Quartz'; }
         when ('11') { $code_name = 'Lavender Calcite'; }
-        default{
+        default {
             $code_name = 'Green Obsidian';
         }
     }
@@ -129,7 +129,7 @@ sub boot_to_login_screen {
     $args{timeout} //= 300;
     if (testapi::is_serial_terminal) {
         # For serial console, just wait for the login prompt
-        unless (wait_serial "login:", timeout=>$args{timeout}) {
+        unless (wait_serial "login:", timeout => $args{timeout}) {
             die "No login prompt shown on serial console.";
         }
     }
@@ -188,7 +188,7 @@ sub desktop_switch_layout {
 sub _console_login_finish {
     # The check differs according to the console used.
     if (testapi::is_serial_terminal) {
-        unless (wait_serial("-bash-.*[#\$]", timeout=>5, expect_not_found=>1)) {
+        unless (wait_serial("-bash-.*[#\$]", timeout => 5, expect_not_found => 1)) {
             record_soft_failure "It looks like profile sourcing failed";
         }
     }
@@ -219,7 +219,7 @@ sub console_login {
     # enable a new proper login based on the user variable.
     if (get_var("SERIAL_CONSOLE")) {
         # Check for the usual prompt.
-        if (wait_serial("~\][#\$]", timeout=>5, quiet=>1)) {
+        if (wait_serial("~\][#\$]", timeout => 5, quiet => 1)) {
             type_string "logout\n";
             # Wait a bit to let the logout properly finish.
             sleep 10;
@@ -233,7 +233,7 @@ sub console_login {
         # Let's perform a simple login test. This is the same as
         # whoami, but has the advantage of existing in installer env
         assert_script_run "id -un";
-        unless (wait_serial $args{user}, timeout=>5) {
+        unless (wait_serial $args{user}, timeout => 5) {
             die "Logging onto the serial console has failed.";
         }
     }
@@ -525,9 +525,9 @@ sub _repo_setup_compose {
     script_run "sed -i -e 's,^metalink,#metalink,g' -e 's,^mirrorlist,#mirrorlist,g' -e 's,^#baseurl.*basearch,baseurl=${location}/Modular/\$basearch,g' -e 's,^#baseurl.*source,baseurl=${location}/Modular/source,g' /etc/yum.repos.d/{fedora-modular,fedora-rawhide-modular}.repo", 0;
 
     # this can be used for debugging if something is going wrong
-#    unless (script_run 'pushd /etc/yum.repos.d && tar czvf yumreposd.tar.gz * && popd') {
+    #    unless (script_run 'pushd /etc/yum.repos.d && tar czvf yumreposd.tar.gz * && popd') {
-#        upload_logs "/etc/yum.repos.d/yumreposd.tar.gz";
+    #        upload_logs "/etc/yum.repos.d/yumreposd.tar.gz";
-#    }
+    #    }
 }
 
 sub _repo_setup_updates {
@@ -538,9 +538,9 @@ sub _repo_setup_updates {
     my $currrel = get_var("CURRREL", "0");
     repos_mirrorlist();
     # this can be used for debugging repo config if something is wrong
-#    unless (script_run 'pushd /etc/yum.repos.d && tar czvf yumreposd.tar.gz * && popd') {
+    #    unless (script_run 'pushd /etc/yum.repos.d && tar czvf yumreposd.tar.gz * && popd') {
-#        upload_logs "/etc/yum.repos.d/yumreposd.tar.gz";
+    #        upload_logs "/etc/yum.repos.d/yumreposd.tar.gz";
-#    }
+    #    }
     if ($version > $currrel) {
         # Disable updates-testing so other bad updates don't break us
         # this will do nothing on upgrade tests as we're on a stable
@@ -735,18 +735,18 @@ sub gnome_initial_setup {
         # https://fedoraproject.org//wiki/Changes/ReduceInitialSetupRedundancy
         # https://bugzilla.redhat.com/show_bug.cgi?id=1474787 ,
         # except 'language' is never *really* skipped (see above)
-        @nexts = grep {$_ ne 'keyboard'} @nexts;
+        @nexts = grep { $_ ne 'keyboard' } @nexts;
-        @nexts = grep {$_ ne 'timezone'} @nexts;
+        @nexts = grep { $_ ne 'timezone' } @nexts;
     }
     else {
         # 'timezone' and 'software' are suppressed for the 'existing user'
         # form of g-i-s
-        @nexts = grep {$_ ne 'software'} @nexts;
+        @nexts = grep { $_ ne 'software' } @nexts;
-        @nexts = grep {$_ ne 'timezone'} @nexts;
+        @nexts = grep { $_ ne 'timezone' } @nexts;
     }
     # 'additional software sources' screen does not display on F28+:
     # https://bugzilla.gnome.org/show_bug.cgi?id=794825
-    @nexts = grep {$_ ne 'software'} @nexts;
+    @nexts = grep { $_ ne 'software' } @nexts;
 
     # note: in g-i-s 3.37.91 and later, the first screen in systemwide
     # mode has a "Start Setup" button, not a "Next" button
@@ -767,7 +767,7 @@ sub gnome_initial_setup {
     }
     # GDM 3.24.1 dumps a cursor in the middle of the screen here...
     mouse_hide if ($args{prelogin});
-    for my $n (1..scalar(@nexts)) {
+    for my $n (1 .. scalar(@nexts)) {
         # click 'Next' $nexts times, moving the mouse to avoid
         # highlight problems, sleeping to give it time to get
         # to the next screen between clicks
@@ -794,7 +794,7 @@ sub gnome_initial_setup {
     unless (get_var("VNC_CLIENT")) {
         # click 'Skip' one time (this is the 'goa' screen). We don't
         # get it on VNC_CLIENT case as network isn't working (yet)
-        mouse_set(100,100);
+        mouse_set(100, 100);
         wait_screen_change { assert_and_click "skip_button"; };
     }
     send_key "ret";
@@ -843,7 +843,7 @@ sub anaconda_create_user {
         @_
     );
     my $user_login = get_var("USER_LOGIN") || "test";
-    assert_and_click("anaconda_install_user_creation", timeout=>$args{timeout});
+    assert_and_click("anaconda_install_user_creation", timeout => $args{timeout});
     assert_screen "anaconda_install_user_creation_screen";
     # wait out animation
     wait_still_screen 2;
@@ -901,8 +901,8 @@ sub check_desktop {
 }
 
 sub download_modularity_tests {
-# Download the modularity test script, place in the system and then
+    # Download the modularity test script, place in the system and then
-# modify the access rights to make it executable.
+    # modify the access rights to make it executable.
     my ($whitelist) = @_;
     # we need python3-yaml for the script to run
     assert_script_run 'dnf -y install python3-yaml', 180;
@@ -914,8 +914,8 @@ sub download_modularity_tests {
 }
 
 sub quit_firefox {
-# Quit Firefox, handling the 'close multiple tabs' warning screen if
+    # Quit Firefox, handling the 'close multiple tabs' warning screen if
-# it shows up
+    # it shows up
     send_key "ctrl-q";
     # expect to get to either the tabs warning or a console
     if (check_screen ["user_console", "root_console", "firefox_close_tabs"], 30) {
@@ -927,14 +927,14 @@ sub quit_firefox {
 }
 
 sub start_with_launcher {
-# Get the name of the needle with a launcher, find the launcher in the menu
+    # Get the name of the needle with a launcher, find the launcher in the menu
-# and click on it to start the application. This function works for the
+    # and click on it to start the application. This function works for the
-# Gnome desktop.
+    # Gnome desktop.
 
     # $launcher holds the launcher needle, but some of the apps are hidden in a submenu
     # so this must be handled first to find the launcher needle.
 
-    my ($launcher,$submenu,$group) = @_;
+    my ($launcher, $submenu, $group) = @_;
     $submenu //= '';
     $group //= '';
     my $desktop = get_var('DESKTOP');
@@ -979,7 +979,7 @@ sub start_with_launcher {
         assert_and_click $launcher;
         wait_still_screen 5;
     }
-    elsif ($desktop eq 'kde'){
+    elsif ($desktop eq 'kde') {
         # Click on the KDE launcher icon
         assert_and_click 'kde_menu_launcher';
         wait_still_screen 2;
@@ -1006,7 +1006,7 @@ sub start_with_launcher {
 
 
 sub quit_with_shortcut {
-# Quit the application using the Alt-F4 keyboard shortcut
+    # Quit the application using the Alt-F4 keyboard shortcut
     send_key 'alt-f4';
     wait_still_screen 5;
     assert_screen 'workspace';
@@ -1035,12 +1035,12 @@ sub advisory_get_installed_packages {
         # occasionally, for some reason, it's unhappy about sorting;
         # we shouldn't fail the test in this case, just upload the
         # files so we can see why...
-        upload_logs "/tmp/allpkgs.txt", failok=>1;
+        upload_logs "/tmp/allpkgs.txt", failok => 1;
-        upload_logs "/var/log/updatepkgs.txt", failok=>1;
+        upload_logs "/var/log/updatepkgs.txt", failok => 1;
     }
     # we'll try and upload the output even if comm 'failed', as it
     # does in fact still write it in some cases
-    upload_logs "/var/log/testedpkgs.txt", failok=>1;
+    upload_logs "/var/log/testedpkgs.txt", failok => 1;
 }
 
 sub advisory_check_nonmatching_packages {
@@ -1068,8 +1068,8 @@ sub advisory_check_nonmatching_packages {
     script_run 'for pkg in $(cat /var/log/updatepkgnames.txt); do rpm -q $pkg && rpm -q $pkg --last | head -1 | cut -d" " -f1 | xargs rpm -q --qf "%{SOURCERPM} %{EPOCH} %{NAME}-%{VERSION}-%{RELEASE}\n" >> /tmp/installedupdatepkgs.txt; done';
     script_run 'sort -u -o /tmp/installedupdatepkgs.txt /tmp/installedupdatepkgs.txt';
     # for debugging, may as well always upload these, can't hurt anything
-    upload_logs "/tmp/installedupdatepkgs.txt", failok=>1;
+    upload_logs "/tmp/installedupdatepkgs.txt", failok => 1;
-    upload_logs "/var/log/updatepkgs.txt", failok=>1;
+    upload_logs "/var/log/updatepkgs.txt", failok => 1;
     # if any line appears in installedupdatepkgs.txt but not updatepkgs.txt,
     # we have a problem.
     if (script_run 'comm -23 /tmp/installedupdatepkgs.txt /var/log/updatepkgs.txt > /var/log/installednotupdatedpkgs.txt') {
@@ -1081,7 +1081,7 @@ sub advisory_check_nonmatching_packages {
     # this exits 1 if the file is zero-length, 0 if it's longer
     # if it's 0, that's *BAD*: we want to upload the file and fail
     unless (script_run 'test -s /var/log/installednotupdatedpkgs.txt') {
-        upload_logs "/var/log/installednotupdatedpkgs.txt", failok=>1;
+        upload_logs "/var/log/installednotupdatedpkgs.txt", failok => 1;
         my $message = "Package(s) from update not installed when it should have been! See installednotupdatedpkgs.txt";
         if ($args{fatal}) {
             set_var("_ACNMP_DONE", "1");
@@ -1372,12 +1372,12 @@ sub click_unwanted_notifications {
     my @closed;
     while ($count > 0 && check_screen "desktop_update_notification_popup", 5) {
         $count -= 1;
-        push (@closed, 'update');
+        push(@closed, 'update');
         click_lastmatch;
     }
     if (check_screen "akonadi_migration_agent_running", 5) {
         click_lastmatch;
-        push (@closed, 'akonadi');
+        push(@closed, 'akonadi');
     }
     return @closed;
 }

main.pm

@@ -33,7 +33,7 @@ testapi::set_distribution(fedoradistribution->new());
 # Stolen from openSUSE.
 sub unregister_needle_tags($) {
     my $tag = shift;
-    my @a   = @{ needle::tags($tag) };
+    my @a = @{needle::tags($tag)};
     for my $n (@a) { $n->unregister(); }
 }
 
@@ -49,14 +49,14 @@ sub unregister_needle_tags($) {
 # 'LANGUAGE-' at all.
 sub unregister_prefix_tags {
     my ($prefix, $valueref) = @_;
-    NEEDLE: for my $needle ( needle::all() ) {
+  NEEDLE: for my $needle (needle::all()) {
         my $unregister = 0;
-        for my $tag ( @{$needle->{'tags'}} ) {
+        for my $tag (@{$needle->{'tags'}}) {
             if ($tag =~ /^\Q$prefix/) {
                 # We have at least one tag matching the prefix, so we
                 # *MAY* want to un-register the needle
                 $unregister = 1;
-                for my $value ( @{$valueref} ) {
+                for my $value (@{$valueref}) {
                     # At any point if we hit a prefix-value match, we
                     # know we need to keep this needle and can skip
                     # to the next
@@ -85,13 +85,13 @@ sub cleanup_needles() {
 
     # Unregister desktop needles of other desktops when DESKTOP is specified
     if (get_var('DESKTOP')) {
-        unregister_prefix_tags('DESKTOP', [ get_var('DESKTOP') ])
+        unregister_prefix_tags('DESKTOP', [get_var('DESKTOP')]);
     }
 
     # Unregister non-language-appropriate needles. See unregister_except_
     # tags for details; basically all needles with at least one LANGUAGE-
     # tag will be unregistered unless they match the current langauge.
-    my $langref = [ get_var('LANGUAGE') || 'english' ];
+    my $langref = [get_var('LANGUAGE') || 'english'];
     unregister_prefix_tags('LANGUAGE', $langref);
 }
 $needle::cleanuphandler = \&cleanup_needles;
@@ -195,15 +195,15 @@ sub load_install_tests() {
     my $partitioning = get_var('PARTITIONING');
     # if PARTITIONING is unset, or one of [...], use disk_guided_empty,
     # which is the simplest / 'default' case.
-    if (! $partitioning || $partitioning ~~ ['guided_empty', 'guided_free_space']) {
+    if (!$partitioning || $partitioning ~~ ['guided_empty', 'guided_free_space']) {
         $storage = "tests/disk_guided_empty.pm";
     }
     else {
-        $storage = "tests/disk_".$partitioning.".pm";
+        $storage = "tests/disk_" . $partitioning . ".pm";
     }
     autotest::loadtest $storage;
 
-    if (get_var("ENCRYPT_PASSWORD")){
+    if (get_var("ENCRYPT_PASSWORD")) {
         autotest::loadtest "tests/disk_guided_encrypted.pm";
     }
 
@@ -330,7 +330,7 @@ sub load_postinstall_tests() {
     }
     autotest::loadtest $storagepost if ($storagepost);
 
-    if (get_var("UEFI") &! get_var("NO_UEFI_POST") &! get_var("START_AFTER_TEST")) {
+    if (get_var("UEFI") & !get_var("NO_UEFI_POST") & !get_var("START_AFTER_TEST")) {
         autotest::loadtest "tests/uefi_postinstall.pm";
     }
 

tests/_advisory_post.pm

@@ -5,7 +5,7 @@ use utils;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # figure out which packages from the update actually got installed
     # (if any) as part of this test
     advisory_get_installed_packages;
@@ -15,7 +15,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_advisory_update.pm

@@ -7,7 +7,7 @@ sub run {
     my $self = shift;
     # do the 'repo setup' steps, which set up a repo containing the
     # update packages and run 'dnf update'
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     repo_setup;
     if (get_var("ADVISORY_BOOT_TEST")) {
         # to test boot stuff - in case the update touched grub2, or dracut,
@@ -25,7 +25,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_anaconda_kdump_disable.pm

@@ -16,7 +16,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_anaconda_kdump_enable.pm

@@ -16,7 +16,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_anaconda_network_enable.pm

@@ -22,7 +22,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_anaconda_network_static.pm

@@ -30,7 +30,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_boot_to_anaconda.pm

@@ -77,7 +77,7 @@ sub run {
     # call do_bootloader with postinstall=0, the params, and the mutex,
     # unless we're a VNC install client (no bootloader there)
     unless (get_var("VNC_CLIENT")) {
-        do_bootloader(postinstall=>0, params=>$params, mutex=>$mutex, timeout=>$timeout);
+        do_bootloader(postinstall => 0, params => $params, mutex => $mutex, timeout => $timeout);
     }
 
     # Read variables for identification tests (see further).
@@ -99,7 +99,7 @@ sub run {
                 # we direct the installer to virtio-console1, and use
                 # virtio-console as a root console
                 select_console('virtio-console1');
-                unless (wait_serial "Use text mode", timeout=>120) { die "Anaconda has not started."; }
+                unless (wait_serial "Use text mode", timeout => 120) { die "Anaconda has not started."; }
                 type_string "2\n";
                 unless (wait_serial "Installation") { die "Text version of Anaconda has not started."; }
             }
@@ -130,17 +130,17 @@ sub run {
                 my $relnum = get_release_number;
                 my $dclick = 0;
                 $dclick = 1 if (get_var("DESKTOP") eq "kde" && $relnum > 33);
-                assert_and_click("live_start_anaconda_icon", dclick=>$dclick);
+                assert_and_click("live_start_anaconda_icon", dclick => $dclick);
                 unless (check_screen "anaconda_select_install_lang", 180) {
                     # click it again - on KDE since 2019-10 or so it seems
                     # like the first attempt sometimes just doesn't work
-                    assert_and_click("live_start_anaconda_icon", dclick=>$dclick, timeout=>300);
+                    assert_and_click("live_start_anaconda_icon", dclick => $dclick, timeout => 300);
                 }
             }
             my $language = get_var('LANGUAGE') || 'english';
             # wait for anaconda to appear; we click to work around
             # RHBZ #1566066 if it happens
-            assert_and_click("anaconda_select_install_lang", timeout=>300);
+            assert_and_click("anaconda_select_install_lang", timeout => 300);
 
             # Select install language
             wait_screen_change { assert_and_click "anaconda_select_install_lang_input"; };
@@ -191,7 +191,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_check_install_source.pm

@@ -81,7 +81,7 @@ sub run {
     }
     # just for convenience - sometimes it's useful to see this log
     # for a success case
-    upload_logs "/tmp/packaging.log", failok=>1;
+    upload_logs "/tmp/packaging.log", failok => 1;
     send_key "ctrl-alt-f6";
 
     # Anaconda hub
@@ -90,7 +90,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_collect_data.pm

@@ -4,7 +4,7 @@ use testapi;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>4);
+    $self->root_console(tty => 4);
     assert_script_run 'top -i -n20 -b > /var/tmp/top.log', 120;
     upload_logs '/var/tmp/top.log';
     unless (get_var("CANNED")) {
@@ -43,7 +43,7 @@ sub run {
 }
 
 sub test_flags {
-    return { 'ignore_failure' => 1 };
+    return {'ignore_failure' => 1};
 }
 
 1;

tests/_console_avc_crash.pm

@@ -5,7 +5,7 @@ use testapi;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # if this is a non-English, non-switched layout, load US layout
     # at this point
     # FIXME: this is all kind of a mess, as on such configs we need

tests/_console_login.pm

@@ -11,14 +11,14 @@ sub run {
     # with a password that requires Arabic support.
     # Such attempt to log in would always fail.
     if (get_var("LANGUAGE") ne "arabic" && get_var("USER_LOGIN") ne "false") {
-            console_login(user=>get_var("USER_LOGIN", "test"), password=>get_var("USER_PASSWORD", "weakpassword"));
+        console_login(user => get_var("USER_LOGIN", "test"), password => get_var("USER_PASSWORD", "weakpassword"));
     }
     if (get_var("ROOT_PASSWORD")) {
-        console_login(user=>"root", password=>get_var("ROOT_PASSWORD"));
+        console_login(user => "root", password => get_var("ROOT_PASSWORD"));
     }
 }
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 1;
 # vim: set sw=4 et:

tests/_console_shutdown.pm

@@ -10,7 +10,7 @@ sub run {
     # use the desktops' graphical shutdown methods, we just go to a
     # console and run 'poweroff'. We can write separate tests for
     # properly testing shutdown/reboot/log out from desktops.
-    $self->root_console(tty=>4);
+    $self->root_console(tty => 4);
     # disable the hidden grub menu on Workstation, so post-install
     # tests that need to edit boot params will see it. Don't use
     # assert_script_run as this will fail when it's not set
@@ -23,7 +23,7 @@ sub run {
 # even if shutdown fails. we should have a separate test for shutdown/
 # logout/reboot stuff, might need some refactoring.
 sub test_flags {
-    return { 'norollback' => 1, 'ignore_failure' => 1 };
+    return {'norollback' => 1, 'ignore_failure' => 1};
 }
 
 1;

tests/_console_wait_login.pm

@@ -11,7 +11,7 @@ sub run {
 
     # handle bootloader, if requested
     if (get_var("GRUB_POSTINSTALL")) {
-        do_bootloader(postinstall=>1, params=>get_var("GRUB_POSTINSTALL"), timeout=>$wait_time);
+        do_bootloader(postinstall => 1, params => get_var("GRUB_POSTINSTALL"), timeout => $wait_time);
         $wait_time = 240;
     }
 
@@ -24,23 +24,23 @@ sub run {
     boot_to_login_screen(timeout => $wait_time);
 
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
 
     # do user login unless USER_LOGIN is set to string 'false'
     unless (get_var("USER_LOGIN") eq "false") {
         # this avoids us waiting 90 seconds for a # to show up
         my $origprompt = $testapi::distri->{serial_term_prompt};
         $testapi::distri->{serial_term_prompt} = '$ ';
-        console_login(user=>get_var("USER_LOGIN", "test"), password=>get_var("USER_PASSWORD", "weakpassword"));
+        console_login(user => get_var("USER_LOGIN", "test"), password => get_var("USER_PASSWORD", "weakpassword"));
         $testapi::distri->{serial_term_prompt} = $origprompt;
     }
     if (get_var("ROOT_PASSWORD")) {
-        console_login(user=>"root", password=>get_var("ROOT_PASSWORD"));
+        console_login(user => "root", password => get_var("ROOT_PASSWORD"));
     }
 }
 
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 
 1;

tests/_coreos_install.pm

@@ -12,7 +12,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_do_install_and_reboot.pm

@@ -137,14 +137,14 @@ sub run {
     # there are various things we might have to do at a console here
     # before we actually reboot. let's figure them all out first...
     my @actions;
-    push (@actions, 'consoletty0') if (get_var("ARCH") eq "aarch64");
+    push(@actions, 'consoletty0') if (get_var("ARCH") eq "aarch64");
-    push (@actions, 'abrt') if (get_var("ABRT", '') eq "system");
+    push(@actions, 'abrt') if (get_var("ABRT", '') eq "system");
-    push (@actions, 'rootpw') if (get_var("INSTALLER_NO_ROOT"));
+    push(@actions, 'rootpw') if (get_var("INSTALLER_NO_ROOT"));
-    push (@actions, 'stagingrepos') if (get_var("DNF_CONTENTDIR"));
+    push(@actions, 'stagingrepos') if (get_var("DNF_CONTENTDIR"));
     # memcheck test doesn't need to reboot at all. Rebooting from GUI
     # for lives is unreliable. And if we're already doing something
     # else at a console, we may as well reboot from there too
-    push (@actions, 'reboot') if (!get_var("MEMCHECK") && (get_var("LIVE") || @actions));
+    push(@actions, 'reboot') if (!get_var("MEMCHECK") && (get_var("LIVE") || @actions));
     # our approach for taking all these actions doesn't work on VNC
     # installs, fortunately we don't need any of them in that case
     # yet, so for now let's just flush the list here if we're VNC
@@ -159,14 +159,14 @@ sub run {
     }
     # OK, if we're here, we got actions, so head to a console. Switch
     # to console after liveinst sometimes takes a while, so 30 secs
-    $self->root_console(timeout=>30);
+    $self->root_console(timeout => 30);
     # this is something a couple of actions may need to know
     my $mount = "/mnt/sysimage";
     if (get_var("CANNED")) {
         # finding the actual host system root is fun for ostree...
         $mount = "/mnt/sysimage/ostree/deploy/fedora*/deploy/*.?";
     }
-    if (grep {$_ eq 'consoletty0'} @actions) {
+    if (grep { $_ eq 'consoletty0' } @actions) {
         # somehow, by this point, localized keyboard layout has been
         # loaded for this tty, so for French and Arabic at least we
         # need to load the 'us' layout again for the next command to
@@ -180,15 +180,15 @@ sub run {
         # regenerate the bootloader config
         assert_script_run "chroot $mount grub2-mkconfig -o /boot/efi/EFI/rocky/grub.cfg";
     }
-    if (grep {$_ eq 'abrt'} @actions) {
+    if (grep { $_ eq 'abrt' } @actions) {
         # Chroot in the newly installed system and switch on ABRT systemwide
         assert_script_run "chroot $mount abrt-auto-reporting 1";
     }
-    if (grep {$_ eq 'rootpw'} @actions) {
+    if (grep { $_ eq 'rootpw' } @actions) {
         my $root_password = get_var("ROOT_PASSWORD") || "weakpassword";
         assert_script_run "echo 'root:$root_password' | chpasswd -R $mount";
     }
-    if (grep {$_ eq 'stagingrepos'} @actions) {
+    if (grep { $_ eq 'stagingrepos' } @actions) {
         if (get_version_major() < 9) {
             assert_script_run 'sed -i -e "s/^mirrorlist/#mirrorlist/g;s,^#\(baseurl=http[s]*://\),\1,g" ' . $mount . '/etc/yum.repos.d/Rocky-BaseOS.repo';
             assert_script_run 'sed -i -e "s/^mirrorlist/#mirrorlist/g;s,^#\(baseurl=http[s]*://\),\1,g" ' . $mount . '/etc/yum.repos.d/Rocky-AppStream.repo';
@@ -202,11 +202,11 @@ sub run {
         }
         assert_script_run 'printf "stg/rocky\n" > ' . $mount . '/etc/dnf/vars/contentdir';
     }
-    type_string "reboot\n" if (grep {$_ eq 'reboot'} @actions);
+    type_string "reboot\n" if (grep { $_ eq 'reboot' } @actions);
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_graphical_input.pm

@@ -50,7 +50,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_graphical_wait_login.pm

@@ -17,17 +17,17 @@ sub run {
 
     # handle bootloader, if requested
     if (get_var("GRUB_POSTINSTALL")) {
-        do_bootloader(postinstall=>1, params=>get_var("GRUB_POSTINSTALL"), timeout=>$wait_time);
+        do_bootloader(postinstall => 1, params => get_var("GRUB_POSTINSTALL"), timeout => $wait_time);
         $wait_time = 300;
     }
 
     # Handle pre-login initial setup if we're doing INSTALL_NO_USER
     if (get_var("INSTALL_NO_USER") && !get_var("_SETUP_DONE")) {
         if (get_var("DESKTOP") eq 'gnome') {
-            gnome_initial_setup(prelogin=>1, timeout=>$wait_time);
+            gnome_initial_setup(prelogin => 1, timeout => $wait_time);
         }
         else {
-            anaconda_create_user(timeout=>$wait_time);
+            anaconda_create_user(timeout => $wait_time);
             # wait out animation
             wait_still_screen 3;
             assert_and_click "initialsetup_finish_configuration";
@@ -125,7 +125,7 @@ sub run {
         # root user now, for subsequent tests to work
         if (get_var("IMAGE_DEPLOY")) {
             send_key "ctrl-alt-f3";
-            console_login(user=>get_var("USER_LOGIN", "test"), password=>get_var("USER_PASSWORD", "weakpassword"));
+            console_login(user => get_var("USER_LOGIN", "test"), password => get_var("USER_PASSWORD", "weakpassword"));
             type_string "sudo su\n";
             type_string "$password\n";
             my $root_password = get_var("ROOT_PASSWORD") || "weakpassword";
@@ -137,11 +137,11 @@ sub run {
     # Move the mouse somewhere it won't highlight the match areas
     mouse_set(300, 800);
     # KDE can take ages to start up
-    check_desktop(timeout=>120);
+    check_desktop(timeout => 120);
 }
 
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 
 1;

tests/_installer_build.pm

@@ -24,13 +24,13 @@ sub run {
     $cmd .= " --repo=/etc/yum.repos.d/advisory.repo --repo=/etc/yum.repos.d/workarounds.repo ./results";
     assert_script_run $cmd, 1500;
     # good to have the log around for checks
-    upload_logs "pylorax.log", failok=>1;
+    upload_logs "pylorax.log", failok => 1;
     assert_script_run "mv results/images/boot.iso ./${advortask}-netinst-${arch}.iso";
     upload_asset "./${advortask}-netinst-x86_64.iso";
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_iot_zezere_remote.pm

@@ -56,7 +56,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_iot_zezere_server.pm

@@ -39,7 +39,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_live_build.pm

@@ -66,7 +66,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_memcheck.pm

@@ -1,17 +1,17 @@
 use base "anacondatest";
- use strict;
+use strict;
- use testapi;
+use testapi;
 
- sub run {
+sub run {
     my $self = shift;
     $self->root_console();
     upload_logs '/tmp/memory.dat';
- }
+}
 
- sub test_flags {
+sub test_flags {
-     return { fatal => 1 };
+    return {fatal => 1};
- }
+}
 
- 1;
+1;
 
- # vim: set sw=4 et:
+# vim: set sw=4 et:

tests/_podman_client.pm

@@ -8,7 +8,7 @@ use utils;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # wait for server to be set up
     mutex_lock "podman_server_ready";
     mutex_unlock "podman_server_ready";
@@ -23,7 +23,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_post_network_static.pm

@@ -13,7 +13,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_setup_browser.pm

@@ -6,7 +6,7 @@ use utils;
 sub run {
     my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
 
     # set up appropriate repositories
     repo_setup();
@@ -27,7 +27,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 
 1;

tests/_software_selection.pm

@@ -33,13 +33,13 @@ sub run {
 
     # In Rocky, graphical-server starts out selected in the DVD ISO so if that's
     # what we're looking for we're done
-    if (!check_screen("anaconda_".$packageset."_selected", 1)) {
+    if (!check_screen("anaconda_" . $packageset . "_selected", 1)) {
-      send_key_until_needlematch("anaconda_".$packageset."_highlighted", "down", 20);
+        send_key_until_needlematch("anaconda_" . $packageset . "_highlighted", "down", 20);
         send_key "spc";
     }
 
     # check that desired environment is selected
-    assert_screen "anaconda_".$packageset."_selected";
+    assert_screen "anaconda_" . $packageset . "_selected";
     assert_and_click "anaconda_spoke_done";
 
     # Anaconda hub
@@ -48,7 +48,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_staging_repos_disable.pm

@@ -5,17 +5,17 @@ use utils;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>4);
+    $self->root_console(tty => 4);
     # Point at default repositories by modifying contentdir
     # NOTE: This will leave repos pointing at primary dl server instead
     #       of mirrorlist.
     script_run 'printf "pub/rocky\n" > /etc/dnf/vars/contentdir';
     script_run 'dnf clean all';
-    script_run 'dnf repoinfo'
+    script_run 'dnf repoinfo';
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_staging_repos_enable.pm

@@ -5,7 +5,7 @@ use utils;
 
 sub run {
     my $self = shift;
-    $self->root_console(tty=>4);
+    $self->root_console(tty => 4);
     # Point at staging repositories by modifying contentdir
     if (get_version_major() < 9) {
         script_run "sed -i 's/^mirrorlist/#mirrorlist/g' /etc/yum.repos.d/Rocky-*";
@@ -20,7 +20,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_support_server.pm

@@ -104,7 +104,7 @@ file:///ks.cfg\n  initrdefi rocky/initrd.img\n}' >> /var/lib/tftpboot/grub.cfg";
 }
 
 sub run {
-    my $self=shift;
+    my $self = shift;
     # disable systemd-resolved, it conflicts with dnsmasq
     unless (script_run "systemctl is-active systemd-resolved.service") {
         script_run "systemctl stop systemd-resolved.service";
@@ -127,7 +127,7 @@ sub run {
     ## ISCSI
 
     # start up iscsi target
-    #assert_script_run "printf '<target iqn.2016-06.local.domain:support.target1>\n    backing-store /dev/vdb\n    incominguser test weakpassword\n</target>' > /etc/tgt/conf.d/openqa.conf";
+#assert_script_run "printf '<target iqn.2016-06.local.domain:support.target1>\n    backing-store /dev/vdb\n    incominguser test weakpassword\n</target>' > /etc/tgt/conf.d/openqa.conf";
     assert_script_run "targetcli /backstores/block create dev=/dev/vdb name=vdb";
     assert_script_run "targetcli /iscsi create wwn=iqn.2016-06.local.domain:support.target1";
     assert_script_run "targetcli /iscsi/iqn.2016-06.local.domain:support.target1/tpg1/acls create iqn.2016-06.local.domain:support.target1";
@@ -186,7 +186,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_vnc_client_connect.pm

@@ -20,7 +20,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_vnc_client_connect_pre.pm

@@ -7,7 +7,7 @@ use utils;
 sub run {
     my $self = shift;
     boot_to_login_screen(timeout => 300);
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     setup_tap_static('172.16.2.115', 'vnc002.test.openqa.fedoraproject.org');
     # test test: check if we can see the server
     assert_script_run "ping -c 2 172.16.2.114";
@@ -18,7 +18,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_vncconnect_client_setup.pm

@@ -13,7 +13,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/_vncconnect_client_setup_pre.pm

@@ -7,7 +7,7 @@ use utils;
 sub run {
     my $self = shift;
     boot_to_login_screen(timeout => 300);
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     setup_tap_static('172.16.2.117', 'vnc004.test.openqa.fedoraproject.org');
     # install tigervnc (Boxes doesn't do reverse VNC)
     assert_script_run "dnf -y install tigervnc", 180;
@@ -17,7 +17,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/anaconda_help.pm

@@ -56,10 +56,10 @@ sub run {
     # on GNOME installs (Workstation Live and Silverblue) we don't
     # need to set a root password or create a user; on other flavors
     # we must
-    unless (get_var("DESKTOP") eq "gnome" ) {
+    unless (get_var("DESKTOP") eq "gnome") {
         # In Rocky ISO you will finish testplan on Create User and need to shift-tab to select
         # Root password
-        if ((get_var("DISTRI") eq "rocky" )) {
+        if ((get_var("DISTRI") eq "rocky")) {
             send_key_until_needlematch("anaconda_main_hub_root_password", "shift-tab");
         }
         assert_and_click "anaconda_main_hub_root_password";

tests/applications/evince/aaa_setup.pm

@@ -25,13 +25,13 @@ sub download_testdata {
     # Change ownership and attributes
     assert_script_run("chown -R test:test openqa_testdata");
     # Move the test file into a correct location.
-    assert_script_run("cp openqa_testdata/evince/evince.pdf Documents")
+    assert_script_run("cp openqa_testdata/evince/evince.pdf Documents");
 }
 
 sub run {
     my $self = shift;
     # Switch to console
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # Perform git test
     check_and_install_git();
     # Download the test data

tests/applications/evince/about.pm

@@ -6,23 +6,23 @@ use utils;
 # This part of the suite tests if About works.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Open the menu by clicking on the Burger icon
+    # Open the menu by clicking on the Burger icon
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# In the menu, select the About item.
+    # In the menu, select the About item.
-assert_and_click("evince_menu_about", button => "left", timeout => 30);
+    assert_and_click("evince_menu_about", button => "left", timeout => 30);
 
-# Check that the About section has been displayed.
+    # Check that the About section has been displayed.
-assert_screen("evince_about_shown");
+    assert_screen("evince_about_shown");
 
-# Click on Credits button to see the second part of the dialogue.
+    # Click on Credits button to see the second part of the dialogue.
-assert_and_click("evince_about_credits", button => "left", timeout => 30);
+    assert_and_click("evince_about_credits", button => "left", timeout => 30);
 
-# Check that Credits are accessible and visible, too.
+    # Check that Credits are accessible and visible, too.
-assert_screen("evince_credits_shown");
+    assert_screen("evince_credits_shown");
 
 }
 

tests/applications/evince/add_annotation.pm

@@ -6,37 +6,37 @@ use utils;
 # This part of the suite tests an annotation can be added to the displayed content.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Click on the Pencil button.
+    # Click on the Pencil button.
-assert_and_click("evince_add_annotation", button => "left", timeout => 30);
+    assert_and_click("evince_add_annotation", button => "left", timeout => 30);
 
-# Click on Note text.
+    # Click on Note text.
-assert_and_click("evince_add_annotation_text", button => "left", timeout => 30);
+    assert_and_click("evince_add_annotation_text", button => "left", timeout => 30);
 
-# Select location to add annotation.
+    # Select location to add annotation.
-assert_and_click("evince_select_annotation_place", button => "left", timeout => 30);
+    assert_and_click("evince_select_annotation_place", button => "left", timeout => 30);
 
-# Enter some text to the annotation.
+    # Enter some text to the annotation.
-type_very_safely("Add note");
+    type_very_safely("Add note");
 
-# Check that the annotation window has appeared with that text.
+    # Check that the annotation window has appeared with that text.
-assert_screen("evince_annotation_added");
+    assert_screen("evince_annotation_added");
 
-# Close the annotation.
+    # Close the annotation.
-assert_and_click("evince_close_annotation", button => "left", timeout => 30);
+    assert_and_click("evince_close_annotation", button => "left", timeout => 30);
 
-# Check that the annotation is still placed in the document.
+    # Check that the annotation is still placed in the document.
-assert_screen("evince_annotation_placed");
+    assert_screen("evince_annotation_placed");
 
-# Open the annotation's context menu.
+    # Open the annotation's context menu.
-assert_and_click("evince_annotation_placed", button => "right", timeout => 30);
+    assert_and_click("evince_annotation_placed", button => "right", timeout => 30);
 
-# Remove the annotation.
+    # Remove the annotation.
-assert_and_click("evince_remove_annotation", button => "left", timeout => 30);
+    assert_and_click("evince_remove_annotation", button => "left", timeout => 30);
 
-# Check that the annotation has been removed.
+    # Check that the annotation has been removed.
-assert_screen("evince_annotation_removed");
+    assert_screen("evince_annotation_removed");
 
 }
 

tests/applications/evince/change_zoom.pm

@@ -6,17 +6,17 @@ use utils;
 # This part of the suite tests that Evince can change the zoom of the document.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
 
-# Click on zoom menu to open choices.
+    # Click on zoom menu to open choices.
-assert_and_click("evince_change_zoom", button => "left", timeout => 30);
+    assert_and_click("evince_change_zoom", button => "left", timeout => 30);
 
-# Select 200%.
+    # Select 200%.
-assert_and_click("evince_select_zoom_200", button => "left", timeout => 30);
+    assert_and_click("evince_select_zoom_200", button => "left", timeout => 30);
 
-# Check that the document zoom was changed.
+    # Check that the document zoom was changed.
-assert_screen("evince_document_zoom_200");
+    assert_screen("evince_document_zoom_200");
 
 }
 

tests/applications/evince/dual_mode.pm

@@ -6,28 +6,28 @@ use utils;
 # This part of the suite tests that Evince is able to display content in a two-page mode.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Click on the Zoom menu to change a different zoom for next steps.
+    # Click on the Zoom menu to change a different zoom for next steps.
-assert_and_click("evince_change_zoom", button => "left", timeout => 30);
+    assert_and_click("evince_change_zoom", button => "left", timeout => 30);
 
-# Select the Fit Width option to be able to see the whole layout.
+    # Select the Fit Width option to be able to see the whole layout.
-assert_and_click("evince_select_zoom_fitwidth", button => "left", timeout => 30);
+    assert_and_click("evince_select_zoom_fitwidth", button => "left", timeout => 30);
 
-#Dismiss the dialogue
+    #Dismiss the dialogue
-send_key("esc");
+    send_key("esc");
 
-# Enter the menu
+    # Enter the menu
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
 
-# Select the Dual mode
+    # Select the Dual mode
-assert_and_click("evince_menu_dual", button => "left", timeout => 30);
+    assert_and_click("evince_menu_dual", button => "left", timeout => 30);
 
-# Dismiss the menu
+    # Dismiss the menu
-send_key("esc");
+    send_key("esc");
 
-# Check that the content is displayed in dual mode.
+    # Check that the content is displayed in dual mode.
-assert_screen("evince_dual_mode", timeout => 30);
+    assert_screen("evince_dual_mode", timeout => 30);
 }
 
 sub test_flags {

tests/applications/evince/help.pm

@@ -6,18 +6,18 @@ use utils;
 # This part of the suite tests that Evince can display the Help pages.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Open menu with Burger icon.
+    # Open menu with Burger icon.
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Select the Help item in the menu.
+    # Select the Help item in the menu.
-assert_and_click("evince_menu_help", button => "left", timeout => 30);
+    assert_and_click("evince_menu_help", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Check that Help has been shown.
+    # Check that Help has been shown.
-assert_screen("evince_help_shown", timeout => 30);
+    assert_screen("evince_help_shown", timeout => 30);
 }
 
 sub test_flags {

tests/applications/evince/night_mode.pm

@@ -6,20 +6,20 @@ use utils;
 # This part of the suite tests that Evince can be switched into night mode.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Click on the Menu button.
+    # Click on the Menu button.
-assert_and_click("gnome_burger_menu", timeout => 30, button => "left");
+    assert_and_click("gnome_burger_menu", timeout => 30, button => "left");
 
-# Click on the Night mode to select it.
+    # Click on the Night mode to select it.
-assert_and_click("evince_toggle_night_mode", button => "left", timeout => 30);
+    assert_and_click("evince_toggle_night_mode", button => "left", timeout => 30);
 
-# The menu stays opened, so hit Esc to dismiss it.
+    # The menu stays opened, so hit Esc to dismiss it.
-send_key("esc");
+    send_key("esc");
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Check that night mode has been activated.
+    # Check that night mode has been activated.
-assert_screen("evince_night_mode", timeout => 30);
+    assert_screen("evince_night_mode", timeout => 30);
 
 }
 

tests/applications/evince/properties.pm

@@ -6,18 +6,18 @@ use utils;
 # This part of the suite tests that Evince can show Document Properties.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Open the menu.
+    # Open the menu.
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Select the Properties item.
+    # Select the Properties item.
-assert_and_click("evince_menu_properties", button => "left", timeout => 30);
+    assert_and_click("evince_menu_properties", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Check that Properties are shown.
+    # Check that Properties are shown.
-assert_screen("evince_properties_shown", timeout => 30);
+    assert_screen("evince_properties_shown", timeout => 30);
 }
 
 sub test_flags {

tests/applications/evince/rotate.pm

@@ -7,27 +7,27 @@ use utils;
 
 sub rotate_content {
 
-# Open the application menu
+    # Open the application menu
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
 
-# Click with the *left* button (needle click area might need some correction)
+    # Click with the *left* button (needle click area might need some correction)
-assert_and_click("evince_menu_rotate", button => "left", timeout => 30);
+    assert_and_click("evince_menu_rotate", button => "left", timeout => 30);
 }
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Rotate the content once.
+    # Rotate the content once.
-rotate_content();
+    rotate_content();
 
-# Check that the window content has been rotated.
+    # Check that the window content has been rotated.
-assert_screen("evince_content_rotated_once", timeout => 30);
+    assert_screen("evince_content_rotated_once", timeout => 30);
 
-# Rotate the content again.
+    # Rotate the content again.
-rotate_content();
+    rotate_content();
 
-# Check that the window content has been rotated.
+    # Check that the window content has been rotated.
-assert_screen("evince_content_rotated_twice", timeout => 30);
+    assert_screen("evince_content_rotated_twice", timeout => 30);
 }
 
 sub test_flags {

tests/applications/evince/save_as.pm

@@ -6,27 +6,27 @@ use utils;
 # This part of the suite tests tests that Evince can Save the document As another document.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Open the menu.
+    # Open the menu.
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
 
-# Select Save As
+    # Select Save As
-assert_and_click("evince_menu_saveas", button => "left", timeout => 30);
+    assert_and_click("evince_menu_saveas", button => "left", timeout => 30);
 
-# Type a new name.
+    # Type a new name.
-type_very_safely("alternative");
+    type_very_safely("alternative");
 
-# Click on the Save button
+    # Click on the Save button
-assert_and_click("gnome_button_save_blue", button => "left", timeout => 30);
+    assert_and_click("gnome_button_save_blue", button => "left", timeout => 30);
 
-# Now the document is saved under a different name. We will switch to the
+    # Now the document is saved under a different name. We will switch to the
-# terminal console to check that it has been created.
+    # terminal console to check that it has been created.
-$self->root_console(tty=>3);
+    $self->root_console(tty => 3);
-assert_script_run("ls /home/test/Documents/alternative.pdf");
+    assert_script_run("ls /home/test/Documents/alternative.pdf");
 
-# Now, check that the new file does not differ from the original one.
+    # Now, check that the new file does not differ from the original one.
-assert_script_run("diff /home/test/Documents/evince.pdf /home/test/Documents/alternative.pdf");
+    assert_script_run("diff /home/test/Documents/evince.pdf /home/test/Documents/alternative.pdf");
 }
 
 sub test_flags {

tests/applications/evince/search.pm

@@ -6,18 +6,18 @@ use utils;
 # This part of the suite tests the ability to search string in the text.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Click on the Search button to search for text
+    # Click on the Search button to search for text
-assert_and_click("evince_search_button", button => "left", timeout => 30);
+    assert_and_click("evince_search_button", button => "left", timeout => 30);
 
-# Type *pages*.
+    # Type *pages*.
-type_very_safely("pages");
+    type_very_safely("pages");
-# Press Enter.
+    # Press Enter.
-send_key("ret");
+    send_key("ret");
 
-# Check that the typed text has been found.
+    # Check that the typed text has been found.
-assert_screen("evince_search_found", timeout => 30);
+    assert_screen("evince_search_found", timeout => 30);
 
 }
 

tests/applications/evince/shortcuts.pm

@@ -6,30 +6,30 @@ use utils;
 # This part of the suite tests that Evince displays shortcuts.
 
 sub run {
-my $self = shift;
+    my $self = shift;
 
-# Open the menu
+    # Open the menu
-assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
+    assert_and_click("gnome_burger_menu", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Select the Keyboard Shortcuts item
+    # Select the Keyboard Shortcuts item
-assert_and_click("evince_menu_shortcuts", button => "left", timeout => 30);
+    assert_and_click("evince_menu_shortcuts", button => "left", timeout => 30);
-wait_still_screen 2;
+    wait_still_screen 2;
 
-# Check that Shortcuts has been shown
+    # Check that Shortcuts has been shown
-assert_screen("evince_shortcuts_shown");
+    assert_screen("evince_shortcuts_shown");
 
-# Click on number 2 to arrive to the second page
+    # Click on number 2 to arrive to the second page
-assert_and_click("evince_shortcuts_second", button => "left", timeout => 30);
+    assert_and_click("evince_shortcuts_second", button => "left", timeout => 30);
 
-# Check that Shortcuts 2 has been shown
+    # Check that Shortcuts 2 has been shown
-assert_screen("evince_shortcuts_second_shown");
+    assert_screen("evince_shortcuts_second_shown");
 
-# Click on number 3 to arrive to the second page
+    # Click on number 3 to arrive to the second page
-assert_and_click("evince_shortcuts_third", button => "left", timeout => 30);
+    assert_and_click("evince_shortcuts_third", button => "left", timeout => 30);
 
-# Check that Shortcuts 3 has been shown
+    # Check that Shortcuts 3 has been shown
-assert_screen("evince_shortcuts_third_shown");
+    assert_screen("evince_shortcuts_third_shown");
 }
 
 sub test_flags {

tests/apps_gnome_preset.pm

@@ -16,7 +16,7 @@ sub run {
 # so there is no need to continue.
 # Also, when subsequent tests fail, the suite will revert to this state for further testing.
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 
 1;

tests/apps_startstop/gnome/terminal.pm

@@ -27,7 +27,7 @@ sub run {
 # so there is no need to continue.
 # Also, when subsequent tests fail, the suite will revert to this state for further testing.
 sub test_flags {
-    return { fatal => 1, milestone => 1 };
+    return {fatal => 1, milestone => 1};
 }
 
 1;

tests/apps_startstop/kde/firewall.pm

@@ -14,7 +14,7 @@ sub run {
     # Firewall requires password to be entered and confirmed to start.
     # View password
     assert_screen "auth_required";
-    my $password = get_var('ROOT_PASSWORD','weakpassword');
+    my $password = get_var('ROOT_PASSWORD', 'weakpassword');
     type_very_safely $password;
     send_key 'ret';
     sleep 5;

tests/autocloud.pm

@@ -23,7 +23,7 @@ sub run {
     assert_script_run "chmod ugo+w /dev/" . $serialdev;
     # let's go to another tty and login as regular user
     send_key "alt-f2";
-    console_login(user=>"test", password=>"weakpassword");
+    console_login(user => "test", password => "weakpassword");
     assert_script_run "curl -O https://fedorapeople.org/groups/qa/tunirtests.tar.gz";
     assert_script_run "tar xvf tunirtests.tar.gz";
     assert_script_run "sudo python3 -m unittest tunirtests.atomictests.TestAtomic01Status -v";
@@ -45,24 +45,24 @@ sub run {
     assert_script_run "sudo python3 -m unittest tunirtests.cloudservice.TestServiceDisable -v";
     type_string "sudo reboot\n";
     boot_to_login_screen(timeout => 180);
-    console_login(user=>"root", password=>"weakpassword");
+    console_login(user => "root", password => "weakpassword");
     # we need to use script_run as regular user again
     assert_script_run "sudo chmod ugo+w /dev/" . $serialdev;
     # let's go to another tty and login as regular user again
     send_key "alt-f2";
-    console_login(user=>"test", password=>"weakpassword");
+    console_login(user => "test", password => "weakpassword");
     _soft_fail_run "tunirtests.testreboot.TestReboot";
     assert_script_run "sudo python3 -m unittest tunirtests.cloudservice.TestServiceManipulation -v";
     # this test only works properly as a regular user
     _soft_fail_run "tunirtests.cloudtests.TestJournalWrittenAfterReboot", 0;
     type_string "sudo reboot\n";
     boot_to_login_screen(timeout => 180);
-    console_login(user=>"root", password=>"weakpassword");
+    console_login(user => "root", password => "weakpassword");
     # we need to use script_run as regular user again
     assert_script_run "sudo chmod ugo+w /dev/" . $serialdev;
     # let's go to another tty and login as regular user again
     send_key "alt-f2";
-    console_login(user=>"test", password=>"weakpassword");
+    console_login(user => "test", password => "weakpassword");
     assert_script_run "sudo python3 -m unittest tunirtests.cloudservice.TestServiceAfter -v";
     assert_script_run "sudo python3 -m unittest tunirtests.atomictests.TestDockerInstalled -v";
     assert_script_run "sudo python3 -m unittest tunirtests.atomictests.TestDockerStorageSetup -v";
@@ -76,7 +76,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_package_install_remove.pm

@@ -6,7 +6,7 @@ use utils;
 sub run {
     my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
 
     # This test case tests that packages can be correctly installed and removed.
     # We will test by installing two packages - ftp and mc.
@@ -49,7 +49,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_reboot_unmount.pm

@@ -19,21 +19,21 @@ sub test_routine {
 sub run {
     my $self = shift;
     # switch to TTY3 for both graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # Run test for the first time
     test_routine();
     # Reboot the system.
     type_safely "reboot\n";
     # This time, we will need to login manually.
     boot_to_login_screen;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
 
     # Run the tests for the second time.
     test_routine();
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_selinux.pm

@@ -4,15 +4,15 @@ use testapi;
 use utils;
 
 sub run {
-    my $self=shift;
+    my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     validate_script_output 'getenforce', sub { $_ =~ m/Enforcing/ };
 }
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_service_manipulation.pm

@@ -6,7 +6,7 @@ use utils;
 sub run {
     my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # we could make this slightly more 'efficient' by assuming chronyd
     # is always going to be enabled/running at first, but it's safer
     # to force an expected starting state.
@@ -14,7 +14,7 @@ sub run {
     script_run "systemctl disable chronyd.service";
     script_run "reboot", 0;
     boot_to_login_screen;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # note the use of ! here is a bash-ism, but it sure makes life easier
     assert_script_run '! systemctl is-enabled chronyd.service';
     assert_script_run '! systemctl is-active chronyd.service';
@@ -33,14 +33,14 @@ sub run {
     assert_script_run '! ps -C chronyd';
     script_run "reboot", 0;
     boot_to_login_screen;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     assert_script_run 'systemctl is-enabled chronyd.service';
     assert_script_run 'systemctl is-active chronyd.service';
     assert_script_run 'ps -C chronyd';
     script_run "systemctl disable chronyd.service";
     script_run "reboot", 0;
     boot_to_login_screen;
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     assert_script_run '! systemctl is-enabled chronyd.service';
     assert_script_run '! systemctl is-active chronyd.service';
     assert_script_run '! ps -C chronyd';
@@ -48,7 +48,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_services_start.pm

@@ -5,7 +5,7 @@ use testapi;
 sub run {
     my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # "Job foo.service/start deleted to break ordering cycle"-type
     # message in the log indicates a service got taken out of the boot
     # process to resolve some kind of dependency loop, see e.g.
@@ -42,7 +42,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_system_logging.pm

@@ -3,9 +3,9 @@ use strict;
 use testapi;
 
 sub run {
-    my $self=shift;
+    my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # Check that journalctl DOESN'T output "No entries". This is also the case when journal files are missing.
     # NOTE: We are quietly assuming that something was logged in journal in last 30 minutes. Should be boot log,
     # switch to TTY3 etc.
@@ -16,7 +16,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/base_update_cli.pm

@@ -6,7 +6,7 @@ use packagetest;
 sub run {
     my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # enable test repos and install test packages
     prepare_test_packages;
     # check rpm agrees they installed good
@@ -24,7 +24,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/btrfs_preserve_home_check_content.pm

@@ -4,9 +4,9 @@ use testapi;
 use utils;
 
 sub run {
-    my $self=shift;
+    my $self = shift;
     # switch to TTY3 for both, graphical and console tests
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     # The pre-created image has a special file left in the home
     # directory. This checks that the file has been left there
     # correctly after system reinstall.
@@ -15,7 +15,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/database_client.pm

@@ -6,7 +6,7 @@ use tapnet;
 use utils;
 
 sub run {
-    my $self=shift;
+    my $self = shift;
     # use compose repo, disable u-t, etc.
     repo_setup();
     # install postgresql
@@ -19,7 +19,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_background.pm

@@ -14,7 +14,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_browser.pm

@@ -30,54 +30,54 @@ sub run {
     check_desktop;
     send_key 'alt-f1';
     # wait out animations
-    wait_still_screen(stilltime=>4, similarity_level=>45);
+    wait_still_screen(stilltime => 4, similarity_level => 45);
     assert_and_click 'browser_launcher';
     assert_screen 'browser', 45;
     # firefox is quite grindy on startup, let it settle
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     # open a new tab so we don't race with the default page load
     # (also focuses the location bar for us)
     _open_new_tab;
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     sleep 3;
     # check FAS, typing slowly to avoid errors
     type_very_safely "https://admin.fedoraproject.org/accounts/\n";
     assert_screen "browser_fas_home";
     _open_new_tab;
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     sleep 2;
     type_very_safely "https://kernel.org\n";
     assert_and_click "browser_kernelorg_patch";
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     assert_and_click "browser_download_save";
     sleep 2;
     send_key 'ret';
-    wait_still_screen(stilltime=>3, similarity_level=>45);
+    wait_still_screen(stilltime => 3, similarity_level => 45);
     # browsers do...something...when the download completes, and we
     # expect there's a single click to make it go away and return
     # browser to a state where we can open a new tab
     assert_and_click "browser_download_complete";
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     # we'll check it actually downloaded later
     # add-on test: at present all desktops we test (KDE, GNOME) are
     # using Firefox by default so we do this unconditionally, but we
     # may need to conditionalize it if we ever test desktops whose
     # default browser doesn't support add-ons or uses different ones
     _open_new_tab;
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     sleep 2;
     type_very_safely "https://addons.mozilla.org/en-US/firefox/addon/ublock-origin/\n";
     assert_and_click "firefox_addon_add";
     assert_and_click "firefox_addon_install";
     assert_and_click "firefox_addon_success";
     # go to a console and check download worked
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     my $user = get_var("USER_LOGIN", "test");
     assert_script_run "test -e /home/$user/Downloads/patch-*.xz";
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_login.pm

@@ -88,7 +88,7 @@ sub login_user {
         type_very_safely "$password\n";
     }
     type_very_safely "$password\n";
-    check_desktop(timeout=>60) if ($args{checklogin});
+    check_desktop(timeout => 60) if ($args{checklogin});
     wait_still_screen 5;
 }
 
@@ -199,7 +199,7 @@ sub run {
     our $desktop = get_var("DESKTOP");
     # replace the wallpaper with a black image, this should work for
     # all desktops. Takes effect after a logout / login cycle
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     assert_script_run "dnf -y install GraphicsMagick", 300;
     assert_script_run "gm convert -size 1024x768 xc:black /usr/share/backgrounds/black.png";
     assert_script_run 'for i in /usr/share/backgrounds/f*/default/*.png; do ln -sf /usr/share/backgrounds/black.png $i; done';
@@ -207,7 +207,7 @@ sub run {
         # use solid blue background for SDDM
         assert_script_run "sed -i -e 's,image,solid,g' /usr/share/sddm/themes/01-breeze-fedora/theme.conf.user";
     }
-    adduser(name=>"Jack Sparrow", login=>"jack", password=>$jackpass);
+    adduser(name => "Jack Sparrow", login => "jack", password => $jackpass);
     if ($desktop eq "gnome") {
         # suppress the Welcome Tour for new users in GNOME 40+
         assert_script_run 'printf "[org.gnome.shell]\nwelcome-dialog-last-shown-version=\'4294967295\'\n" > /usr/share/glib-2.0/schemas/org.gnome.shell.gschema.override';
@@ -215,12 +215,12 @@ sub run {
         # In Gnome, we can create a passwordless user that can provide his password upon
         # the first login. So we can create the second user in this way to test this feature
         # later.
-        adduser(name=>"Jim Eagle", login=>"jim", password=>"askuser");
+        adduser(name => "Jim Eagle", login => "jim", password => "askuser");
     }
     else {
         # In KDE, we can also create a passwordless user, but we cannot log into the system
         # later, so we will create the second user the standard way.
-        adduser(name=>"Jim Eagle", login=>"jim", password=>$jimpass);
+        adduser(name => "Jim Eagle", login => "jim", password => $jimpass);
     }
 
     # Clean boot the system, and note what accounts are listed on the login screen.
@@ -230,7 +230,7 @@ sub run {
     boot_to_login_screen;
 
     # Log in with the first user account.
-    login_user(user=>"jack", password=>$jackpass);
+    login_user(user => "jack", password => $jackpass);
     check_user_logged_in("jack");
     # Log out the user.
     logout_user();
@@ -238,18 +238,18 @@ sub run {
     # Log in with the second user account. The second account, Jim Eagle,
     if ($desktop eq "gnome") {
         # If we are in Gnome, we will this time assign a password on first log-in.
-        login_user(user=>"jim", password=>$jimpass, method=>"create");
+        login_user(user => "jim", password => $jimpass, method => "create");
     }
     else {
         # If not, we are in KDE and we will log in normally.
-        login_user(user=>"jim", password=>$jimpass);
+        login_user(user => "jim", password => $jimpass);
     }
     check_user_logged_in("jim");
     # And this time reboot the system using the menu.
     reboot_system();
 
     # Try to log in with either account, intentionally entering the wrong password.
-    login_user(user=>"jack", password=>"wrongpassword", checklogin=>0);
+    login_user(user => "jack", password => "wrongpassword", checklogin => 0);
     my $relnum = get_release_number;
     if ($desktop eq "gnome" && $relnum < 34) {
         # In GDM before F34, a message is shown about an unsuccessful login
@@ -265,13 +265,13 @@ sub run {
     # Now, log into the system again using the correct password. This will
     # only work if we were correctly denied login with the wrong password,
     # if we were let in with the wrong password it'll fail
-    login_user(user=>"jim", password=>$jimpass);
+    login_user(user => "jim", password => $jimpass);
     check_user_logged_in("jim");
 
     # Lock the screen and unlock again.
     lock_screen();
     # Use the password to unlock the screen.
-    login_user(user=>"jim", password=>$jimpass, method=>"unlock");
+    login_user(user => "jim", password => $jimpass, method => "unlock");
 
     # Switch user tests
     if ($desktop eq "gnome") {
@@ -286,7 +286,7 @@ sub run {
         # Initiate switch user
         switch_user();
         # Now, we get a new login screen, so let's do the login into the new session.
-        login_user(user=>"jack", password=>$jackpass);
+        login_user(user => "jack", password => $jackpass);
         # Check that it is a new session, the terminal window should not be visible.
         if (check_screen "user_confirm_jim") {
             die "The session was not switched!";
@@ -298,7 +298,7 @@ sub run {
         logout_user();
         # Now, let us log into the original session, this time, the terminal window
         # should still be visible.
-        login_user(user=>"jim", password=>$jimpass);
+        login_user(user => "jim", password => $jimpass);
         assert_screen "user_confirm_jim";
 
         # We will also test another alternative - switching the user from
@@ -306,7 +306,7 @@ sub run {
         lock_screen();
         send_key "ret";
         switch_user();
-        login_user(user=>"jack", password=>$jackpass);
+        login_user(user => "jack", password => $jackpass);
         check_user_logged_in("jack");
     }
     # Power off the machine
@@ -314,7 +314,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_notifications.pm

@@ -17,14 +17,14 @@ sub run {
     my $version_major = get_version_major;
     # for the live image case, handle bootloader here
     if (get_var("BOOTFROM")) {
-        do_bootloader(postinstall=>1, params=>'3');
+        do_bootloader(postinstall => 1, params => '3');
     }
     else {
-        do_bootloader(postinstall=>0, params=>'3');
+        do_bootloader(postinstall => 0, params => '3');
     }
     boot_to_login_screen;
     # use tty1 to avoid RHBZ #1821499 on F32 Workstation live
-    $self->root_console(tty=>1);
+    $self->root_console(tty => 1);
     # ensure we actually have some package updates available
     prepare_test_packages;
     if ($desktop eq 'gnome') {
@@ -43,11 +43,11 @@ sub run {
             # get notifications, see:
             # https://wiki.gnome.org/Design/Apps/Software/Updates#Tentative_Design
             my $now = script_output 'date +%s';
-            my $yyday = $now - 2*24*60*60;
+            my $yyday = $now - 2 * 24 * 60 * 60;
-            my $longago = $now - 14*24*60*60;
+            my $longago = $now - 14 * 24 * 60 * 60;
             # have to log in as the user to do this
             script_run 'exit', 0;
-            console_login(user=>get_var('USER_LOGIN', 'test'), password=>get_var('USER_PASSWORD', 'weakpassword'));
+            console_login(user => get_var('USER_LOGIN', 'test'), password => get_var('USER_PASSWORD', 'weakpassword'));
             script_run "gsettings set org.gnome.software check-timestamp ${yyday}", 0;
             script_run "gsettings set org.gnome.software update-notification-timestamp ${longago}", 0;
             script_run "gsettings set org.gnome.software online-updates-timestamp ${longago}", 0;
@@ -55,7 +55,7 @@ sub run {
             script_run "gsettings set org.gnome.software install-timestamp ${longago}", 0;
             wait_still_screen 5;
             script_run 'exit', 0;
-            console_login(user=>'root', password=>get_var('ROOT_PASSWORD', 'weakpassword'));
+            console_login(user => 'root', password => get_var('ROOT_PASSWORD', 'weakpassword'));
         }
     }
     # can't use assert_script_run here as long as we're on tty1
@@ -76,14 +76,14 @@ sub run {
         type_very_safely get_var("USER_PASSWORD", "weakpassword");
         send_key 'ret';
     }
-    check_desktop(timeout=>90);
+    check_desktop(timeout => 90);
     # now, WE WAIT. Because KDE on F34+ shows a notification only
     # briefly we will keep an eye out and record if we saw it (logic
     # around this comes later). But we wait the whole ten minutes even
     # if we see it  so we catch any unwanted notifications that appear
     # shortly after boot
     my $seen = 0;
-    for my $n (1..16) {
+    for my $n (1 .. 16) {
         $seen = 1 if (check_screen "desktop_update_notification", 30);
         mouse_set 20, 20;
         send_key "spc";
@@ -148,7 +148,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_printing.pm

@@ -8,7 +8,7 @@ sub run {
     # Prepare the environment:
     #
     # Become root
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
 
     # Create a text file with content to print
     script_run "cd /home/test/";
@@ -37,12 +37,12 @@ sub run {
 
     # Open the text editor and print the file.
     wait_screen_change { send_key "alt-f2"; };
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     type_very_safely "$editor /home/test/testfile.txt\n";
-    wait_still_screen(stilltime=>5, similarity_level=>44);
+    wait_still_screen(stilltime => 5, similarity_level => 44);
     # Print the file using the Cups-PDF printer
     send_key "ctrl-p";
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     if ($desktop eq 'gnome') {
         assert_and_click "printing_select_pdfprinter";
     }
@@ -52,7 +52,7 @@ sub run {
         # We only check that it is correct.
         assert_screen "printing_pdfprinter_ready";
     }
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     assert_and_click "printing_print";
     # Exit the application
     send_key "alt-f4";
@@ -63,7 +63,7 @@ sub run {
 
     # Open the pdf file and check the print
     send_key "alt-f2";
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     # output location is different for cups-pdf 3.0.1-12 or later (we
     # checked this above)
     if ($cpdfvercmp eq "12") {
@@ -73,11 +73,11 @@ sub run {
     else {
         type_safely "$viewer /home/test/Desktop/testfile-job_1.pdf\n";
     }
-    wait_still_screen(stilltime=>5, similarity_level=>45);
+    wait_still_screen(stilltime => 5, similarity_level => 45);
     # Resize the window, so that the size of the document fits the bigger space
     # and gets more readable.
     send_key $maximize;
-    wait_still_screen(stilltime=>2, similarity_level=>45);
+    wait_still_screen(stilltime => 2, similarity_level => 45);
     # make sure we're at the start of the document
     send_key "ctrl-home" if ($desktop eq "kde");
     # Check the printed pdf.
@@ -86,7 +86,7 @@ sub run {
 
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_terminal.pm

@@ -31,7 +31,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/desktop_update_graphical.pm

@@ -10,7 +10,7 @@ sub run {
     my $relnum = get_release_number;
     my $version_major = get_version_major;
     # use a tty console for repo config and package prep
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     assert_script_run 'dnf config-manager --set-disabled updates-testing';
     prepare_test_packages;
     # get back to the desktop
@@ -49,7 +49,7 @@ sub run {
     # may be waiting some time at a 'Software catalog is being
     # loaded' screen.
     if ($desktop eq 'gnome' || ($desktop eq 'kde' && $relnum > 33)) {
-        for my $n (1..5) {
+        for my $n (1 .. 5) {
             last if (check_screen 'desktop_package_tool_update', 120);
             mouse_set 10, 10;
             mouse_hide;
@@ -109,7 +109,7 @@ sub run {
         # FIXME: also sometimes the update apply button just doesn't
         # work, so keep clicking till it does:
         # https://bugzilla.redhat.com/show_bug.cgi?id=1943943
-        for my $n (1..10) {
+        for my $n (1 .. 10) {
             sleep 2;
             assert_screen ['kde_offline_update_reboot', 'desktop_package_tool_update_apply'];
             # break out if we reached the reboot button
@@ -127,12 +127,12 @@ sub run {
         assert_screen 'desktop_package_tool_update_done', 180;
     }
     # back to console to verify updates
-    $self->root_console(tty=>3);
+    $self->root_console(tty => 3);
     verify_updated_packages;
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_btrfs.pm

@@ -36,7 +36,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_btrfs_preserve_home.pm

@@ -14,7 +14,7 @@ sub run {
     if (get_var("UEFI")) {
         # if we're running on UEFI, let us reformat the UEFI first
         # Select the UEFI partition if it is not selected by default
-        if (not (check_screen "anaconda_blivet_part_efi_selected", 30)) {
+        if (not(check_screen "anaconda_blivet_part_efi_selected", 30)) {
             assert_and_click "anaconda_blivet_part_inactive_efi";
             wait_still_screen 5;
         }
@@ -23,7 +23,7 @@ sub run {
         assert_and_click "anaconda_blivet_part_edit";
         # Select the Format option
         assert_and_click "anaconda_blivet_part_format";
-    if (not (check_screen "anaconda_blivet_part_fs_efi_filesystem_selected", 30)) {
+        if (not(check_screen "anaconda_blivet_part_fs_efi_filesystem_selected", 30)) {
             assert_and_click "anaconda_blivet_part_fs_select";
             assert_and_click "anaconda_blivet_part_fs_efi_filesystem";
         }
@@ -50,7 +50,7 @@ sub run {
     # Select the Format option
     assert_and_click "anaconda_blivet_part_format";
     # Open the filesystem types and select ext4, if not selected
-    if (not (check_screen "anaconda_blivet_part_fs_ext4_selected",30)) {
+    if (not(check_screen "anaconda_blivet_part_fs_ext4_selected", 30)) {
         assert_and_click "anaconda_blivet_part_fs_select";
         assert_and_click "anaconda_blivet_part_fs_ext4";
     }
@@ -65,7 +65,7 @@ sub run {
     assert_and_click "anaconda_blivet_volumes_icon";
 
     # Select the home partition
-    if (not (check_screen "anaconda_blivet_part_home_selected")) {
+    if (not(check_screen "anaconda_blivet_part_home_selected")) {
         assert_and_click "anaconda_blivet_part_inactive_home";
     }
     # Go to the partition settings
@@ -113,7 +113,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_ext3.pm

@@ -30,7 +30,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_lvm_ext4.pm

@@ -35,7 +35,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_lvmthin.pm

@@ -38,7 +38,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_resize_lvm.pm

@@ -86,7 +86,7 @@ sub run {
 }
 
 sub test_flags {
-        return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_software_raid.pm

@@ -7,7 +7,7 @@ sub run {
     my $self = shift;
     # Go to INSTALLATION DESTINATION and ensure two disks are selected.
     # Because PARTITIONING starts with 'custom_blivet', this will select blivet-gui.
-    select_disks(disks=>2);
+    select_disks(disks => 2);
     assert_and_click "anaconda_spoke_done";
 
     if (get_var("UEFI")) {
@@ -30,7 +30,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_standard_partition_ext4.pm

@@ -31,7 +31,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_standard_partition_ext4_postinstall.pm

@@ -22,7 +22,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_with_swap.pm

@@ -31,7 +31,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_blivet_xfs.pm

@@ -30,7 +30,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_btrfs.pm

@@ -24,7 +24,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_btrfs_postinstall.pm

@@ -9,7 +9,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_btrfs_preserve_home.pm

@@ -72,7 +72,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_ext3.pm

@@ -26,7 +26,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_ext3_postinstall.pm

@@ -9,7 +9,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_gui_lvm_ext4.pm

@@ -35,7 +35,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_gui_software_raid.pm

@@ -7,7 +7,7 @@ sub run {
     my $self = shift;
     # Go to INSTALLATION DESTINATION and ensure two disks are selected.
     # Because PARTITIONING starts with 'custom', this will select blivet-gui.
-    select_disks(disks=>2);
+    select_disks(disks => 2);
     assert_and_click "anaconda_spoke_done";
 
     if (get_var("UEFI")) {
@@ -31,7 +31,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_gui_standard_partition_ext4.pm

@@ -32,7 +32,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_gui_xfs.pm

@@ -31,7 +31,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_iscsi.pm

@@ -11,7 +11,7 @@ sub run {
     # Anaconda hub
     # Go to INSTALLATION DESTINATION and ensure one regular disk
     # and the iscsi target are selected.
-    select_disks(iscsi=>\%iscsi);
+    select_disks(iscsi => \%iscsi);
     assert_and_click "anaconda_spoke_done";
     # now we're at custom part. let's use standard partitioning for
     # simplicity
@@ -25,7 +25,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_iscsi_postinstall.pm

@@ -12,7 +12,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_lvm_ext4.pm

@@ -28,7 +28,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_lvm_ext4_postinstall.pm

@@ -6,7 +6,7 @@ use utils;
 sub run {
     my $self = shift;
     unless (check_screen "root_console", 0) {
-        $self->root_console(tty=>4);
+        $self->root_console(tty => 4);
     }
     assert_screen "root_console";
     my $devboot = 'vda1';
@@ -29,7 +29,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_lvmthin.pm

@@ -24,7 +24,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;

tests/disk_custom_lvmthin_postinstall.pm

@@ -6,7 +6,7 @@ use utils;
 sub run {
     my $self = shift;
     unless (check_screen "root_console", 0) {
-        $self->root_console(tty=>4);
+        $self->root_console(tty => 4);
     }
     assert_screen "root_console";
     # check that lvmthinpool is present:
@@ -16,7 +16,7 @@ sub run {
 }
 
 sub test_flags {
-    return { fatal => 1 };
+    return {fatal => 1};
 }
 
 1;