This adds a new test suite, run for Workstation and KDE live
images, which does not create a user during install. It then
expects initial-setup (KDE) or gnome-initial-setup (Workstation)
to appear after install, creates a user, and proceeds with
normal boot.
Note the ARM image test already covers the initial-setup text
mode, and the ARM minimal image is the only case where that
actually matters (it's not included in Server).
Run the new tests, check they work. Run all old
tests, check the changes didn't break them.
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
Looks OK.
| lib/utils.pm | ||
|---|---|---|
| 416–419 | I don't know why they are keep showing it. Wouldn't something like "Help" button on last screen in g-i-s be enough? | |
| lib/utils.pm | ||
|---|---|---|
| 416–419 | They don't 'keep' showing it. There's two flows: g-i-s -> gdm -> desktop + getting_started it's only shown once each time, the order's just a bit different. I could split the code into a helper function in util and call it from both places, but it seems a bit short to bother. | |
| Path | Packages | |||
|---|---|---|---|---|
| M | lib/utils.pm (117 lines) | |||
| A | M | needles/anaconda/initialsetup_finish_configuration.json (15 lines) | ||
| A | M | (bin) | needles/anaconda/initialsetup_finish_configuration.png | |
| M | templates (28 lines) | |||
| M | tests/_do_install_and_reboot.pm (55 lines) | |||
| M | tests/_graphical_wait_login.pm (54 lines) |
| Commit | Tree | Parents | Author | Summary | Date |
|---|---|---|---|---|---|
| 16648b7e554c | 5f3f1f700524 | 638adda97ac6 | Adam Williamson | Add tests with no user created during install (Show More…) | Apr 3 2017, 11:23 PM |
| 1 | package utils; | 1 | package utils; | ||
|---|---|---|---|---|---|
| 2 | 2 | | |||
| 3 | use strict; | 3 | use strict; | ||
| 4 | 4 | | |||
| 5 | use base 'Exporter'; | 5 | use base 'Exporter'; | ||
| 6 | use Exporter; | 6 | use Exporter; | ||
| 7 | 7 | | |||
| 8 | use lockapi; | 8 | use lockapi; | ||
| 9 | use testapi; | 9 | use testapi; | ||
| 10 | our @EXPORT = qw/run_with_error_check type_safely type_very_safely desktop_vt boot_to_login_screen console_login console_switch_layout desktop_switch_layout console_loadkeys_us do_bootloader get_milestone boot_decrypt check_release menu_launch_type start_cockpit repo_setup/; | 10 | our @EXPORT = qw/run_with_error_check type_safely type_very_safely desktop_vt boot_to_login_screen console_login console_switch_layout desktop_switch_layout console_loadkeys_us do_bootloader get_milestone boot_decrypt check_release menu_launch_type start_cockpit repo_setup gnome_initial_setup anaconda_create_user/; | ||
| 11 | 11 | | |||
| 12 | sub run_with_error_check { | 12 | sub run_with_error_check { | ||
| 13 | my ($func, $error_screen) = @_; | 13 | my ($func, $error_screen) = @_; | ||
| 14 | die "Error screen appeared" if (check_screen $error_screen, 5); | 14 | die "Error screen appeared" if (check_screen $error_screen, 5); | ||
| 15 | $func->(); | 15 | $func->(); | ||
| 16 | die "Error screen appeared" if (check_screen $error_screen, 5); | 16 | die "Error screen appeared" if (check_screen $error_screen, 5); | ||
| 17 | } | 17 | } | ||
| 18 | 18 | | |||
| ▲ Show 20 Lines • Show All 341 Lines • ▼ Show 20 Line(s) | 358 | sub repo_setup { | |||
| 360 | get_var("ADVISORY") ? _repo_setup_updates : _repo_setup_compose; | 360 | get_var("ADVISORY") ? _repo_setup_updates : _repo_setup_compose; | ||
| 361 | # This repo does not always exist for Rawhide or Branched, and | 361 | # This repo does not always exist for Rawhide or Branched, and | ||
| 362 | # some things (at least realmd) try to update the repodata for | 362 | # some things (at least realmd) try to update the repodata for | ||
| 363 | # it even though it is disabled, and fail. At present none of the | 363 | # it even though it is disabled, and fail. At present none of the | ||
| 364 | # tests needs it, so let's just unconditionally nuke it. | 364 | # tests needs it, so let's just unconditionally nuke it. | ||
| 365 | assert_script_run "rm -f /etc/yum.repos.d/fedora-cisco-openh264.repo"; | 365 | assert_script_run "rm -f /etc/yum.repos.d/fedora-cisco-openh264.repo"; | ||
| 366 | } | 366 | } | ||
| 367 | 367 | | |||
| 368 | sub gnome_initial_setup { | ||||
| 369 | # Handle gnome-initial-setup, with variations for the pre-login | ||||
| 370 | # mode (when no user was created during install) and post-login | ||||
| 371 | # mode (when user was created during install) | ||||
| 372 | my %args = ( | ||||
| 373 | prelogin => 0, | ||||
| 374 | timeout => 120, | ||||
| 375 | @_ | ||||
| 376 | ); | ||||
| 377 | my $nexts = 3; | ||||
| 378 | if ($args{prelogin}) { | ||||
| 379 | $nexts = 5; | ||||
| 380 | } | ||||
| 381 | assert_screen "next_button", $args{timeout}; | ||||
| 382 | # wait a bit in case of animation | ||||
| 383 | wait_still_screen 3; | ||||
| 384 | for my $n (1..$nexts) { | ||||
| 385 | # click 'Next' $nexts times, moving the mouse to avoid | ||||
| 386 | # highlight problems, sleeping to give it time to get | ||||
| 387 | # to the next screen between clicks | ||||
| 388 | mouse_set(100, 100); | ||||
| 389 | wait_screen_change { assert_and_click "next_button"; }; | ||||
| 390 | # for Japanese, we need to workaround a bug on the keyboard | ||||
| 391 | # selection screen | ||||
| 392 | if ($n == 1 && get_var("LANGUAGE") eq 'japanese') { | ||||
| 393 | if (!check_screen 'initial_setup_kana_kanji_selected', 5) { | ||||
| 394 | record_soft_failure 'kana kanji not selected: bgo#776189'; | ||||
| 395 | assert_and_click 'initial_setup_kana_kanji'; | ||||
| 396 | } | ||||
| 397 | } | ||||
| 398 | } | ||||
| 399 | # click 'Skip' one time | ||||
| 400 | mouse_set(100,100); | ||||
| 401 | wait_screen_change { assert_and_click "skip_button"; }; | ||||
| 402 | send_key "ret"; | ||||
| 403 | if ($args{prelogin}) { | ||||
| 404 | # create user | ||||
| 405 | my $user_login = get_var("USER_LOGIN") || "test"; | ||||
| 406 | my $user_password = get_var("USER_PASSWORD") || "weakpassword"; | ||||
| 407 | type_very_safely $user_login; | ||||
| 408 | wait_screen_change { assert_and_click "next_button"; }; | ||||
| 409 | type_very_safely $user_password; | ||||
| 410 | send_key "tab"; | ||||
| 411 | type_very_safely $user_password; | ||||
| 412 | wait_screen_change { assert_and_click "next_button"; }; | ||||
| 413 | send_key "ret"; | ||||
| 414 | } | ||||
| 415 | else { | ||||
| 416 | # wait for the stupid 'help' screen to show and kill it | ||||
| 417 | assert_screen "getting_started"; | ||||
| 418 | send_key "alt-f4"; | ||||
| 419 | wait_still_screen 5; | ||||
They don't 'keep' showing it. There's two flows: g-i-s -> gdm -> desktop + getting_started it's only shown once each time, the order's just a bit different. I could split the code into a helper function in util and call it from both places, but it seems a bit short to bother. | |||||
| 420 | # don't do it again on second load | ||||
| 421 | } | ||||
| 422 | set_var("_setup_done", 1); | ||||
| 423 | } | ||||
| 424 | | ||||
| 425 | sub _type_user_password { | ||||
| 426 | # convenience function used by anaconda_create_user, not meant | ||||
| 427 | # for direct use | ||||
| 428 | my $user_password = get_var("USER_PASSWORD") || "weakpassword"; | ||||
| 429 | if (get_var("SWITCHED_LAYOUT")) { | ||||
| 430 | # we double the password, the second time using the native | ||||
| 431 | # layout, so the password has both ASCII and native characters | ||||
| 432 | desktop_switch_layout "ascii", "anaconda"; | ||||
| 433 | type_very_safely $user_password; | ||||
| 434 | desktop_switch_layout "native", "anaconda"; | ||||
| 435 | type_very_safely $user_password; | ||||
| 436 | } | ||||
| 437 | else { | ||||
| 438 | type_very_safely $user_password; | ||||
| 439 | } | ||||
| 440 | } | ||||
| 441 | | ||||
| 442 | sub anaconda_create_user { | ||||
| 443 | # Create a user, in the anaconda interface. This is here because | ||||
| 444 | # the same code works both during install and for initial-setup, | ||||
| 445 | # which runs post-install, so we can share it. | ||||
| 446 | my %args = ( | ||||
| 447 | timeout => 90, | ||||
| 448 | @_ | ||||
| 449 | ); | ||||
| 450 | my $user_login = get_var("USER_LOGIN") || "test"; | ||||
| 451 | assert_and_click "anaconda_install_user_creation", '', $args{timeout}; | ||||
| 452 | assert_screen "anaconda_install_user_creation_screen"; | ||||
| 453 | # wait out animation | ||||
| 454 | wait_still_screen 2; | ||||
| 455 | type_very_safely $user_login; | ||||
| 456 | type_very_safely "\t\t\t\t"; | ||||
| 457 | _type_user_password(); | ||||
| 458 | wait_screen_change { send_key "tab"; }; | ||||
| 459 | wait_still_screen 2; | ||||
| 460 | _type_user_password(); | ||||
| 461 | # even with all our slow typing this still *sometimes* seems to | ||||
| 462 | # miss a character, so let's try again if we have a warning bar. | ||||
| 463 | # But not if we're installing with a switched layout, as those | ||||
| 464 | # will *always* result in a warning bar at this point (see below) | ||||
| 465 | if (!get_var("SWITCHED_LAYOUT") && check_screen "anaconda_warning_bar", 3) { | ||||
| 466 | wait_screen_change { send_key "shift-tab"; }; | ||||
| 467 | wait_still_screen 2; | ||||
| 468 | _type_user_password(); | ||||
| 469 | wait_screen_change { send_key "tab"; }; | ||||
| 470 | wait_still_screen 2; | ||||
| 471 | _type_user_password(); | ||||
| 472 | } | ||||
| 473 | assert_and_click "anaconda_install_user_creation_make_admin"; | ||||
| 474 | assert_and_click "anaconda_spoke_done"; | ||||
| 475 | # since 20170105, we will get a warning here when the password | ||||
| 476 | # contains non-ASCII characters. Assume only switched layouts | ||||
| 477 | # produce non-ASCII characters, though this isn't strictly true | ||||
| 478 | if (get_var('SWITCHED_LAYOUT') && check_screen "anaconda_warning_bar", 3) { | ||||
| 479 | wait_still_screen 1; | ||||
| 480 | assert_and_click "anaconda_spoke_done"; | ||||
| 481 | } | ||||
| 482 | } | ||||
| 1 | { | ||||
|---|---|---|---|---|---|
| 2 | "tags": [ | ||||
| 3 | "initialsetup_finish_configuration" | ||||
| 4 | ], | ||||
| 5 | "properties": [], | ||||
| 6 | "area": [ | ||||
| 7 | { | ||||
| 8 | "xpos": 852, | ||||
| 9 | "ypos": 732, | ||||
| 10 | "width": 148, | ||||
| 11 | "height": 12, | ||||
| 12 | "type": "match" | ||||
| 13 | } | ||||
| 14 | ] | ||||
| 15 | } | ||||
| Show First 20 Lines • Show All 178 Lines • ▼ Show 20 Line(s) | 177 | product => { | |||
|---|---|---|---|---|---|
| 179 | distri => "fedora", | 179 | distri => "fedora", | ||
| 180 | flavor => "Atomic-dvd_ostree-iso", | 180 | flavor => "Atomic-dvd_ostree-iso", | ||
| 181 | version => "*", | 181 | version => "*", | ||
| 182 | }, | 182 | }, | ||
| 183 | test_suite => { name => "install_default" }, | 183 | test_suite => { name => "install_default" }, | ||
| 184 | }, | 184 | }, | ||
| 185 | { | 185 | { | ||
| 186 | machine => { name => "64bit" }, | 186 | machine => { name => "64bit" }, | ||
| 187 | prio => 20, | ||||
| 188 | product => { | ||||
| 189 | arch => "x86_64", | ||||
| 190 | distri => "fedora", | ||||
| 191 | flavor => "Workstation-live-iso", | ||||
| 192 | version => "*", | ||||
| 193 | }, | ||||
| 194 | test_suite => { name => "install_no_user" }, | ||||
| 195 | }, | ||||
| 196 | { | ||||
| 197 | machine => { name => "64bit" }, | ||||
| 198 | prio => 21, | ||||
| 199 | product => { | ||||
| 200 | arch => "x86_64", | ||||
| 201 | distri => "fedora", | ||||
| 202 | flavor => "KDE-live-iso", | ||||
| 203 | version => "*", | ||||
| 204 | }, | ||||
| 205 | test_suite => { name => "install_no_user" }, | ||||
| 206 | }, | ||||
| 207 | { | ||||
| 208 | machine => { name => "64bit" }, | ||||
| 187 | prio => 40, | 209 | prio => 40, | ||
| 188 | product => { | 210 | product => { | ||
| 189 | arch => "x86_64", | 211 | arch => "x86_64", | ||
| 190 | distri => "fedora", | 212 | distri => "fedora", | ||
| 191 | flavor => "Workstation-live-iso", | 213 | flavor => "Workstation-live-iso", | ||
| 192 | version => "*", | 214 | version => "*", | ||
| 193 | }, | 215 | }, | ||
| 194 | test_suite => { name => "base_selinux" }, | 216 | test_suite => { name => "base_selinux" }, | ||
| ▲ Show 20 Lines • Show All 2181 Lines • ▼ Show 20 Line(s) | 2397 | { | |||
| 2376 | name => "memory_check", | 2398 | name => "memory_check", | ||
| 2377 | settings => [ | 2399 | settings => [ | ||
| 2378 | { key => "PACKAGE_SET", value => "default" }, | 2400 | { key => "PACKAGE_SET", value => "default" }, | ||
| 2379 | { key => "MEMCHECK", value => "1" }, | 2401 | { key => "MEMCHECK", value => "1" }, | ||
| 2380 | { key => "REPOSITORY_VARIATION", value => "%LOCATION%" }, | 2402 | { key => "REPOSITORY_VARIATION", value => "%LOCATION%" }, | ||
| 2381 | { key => "TEST_TARGET", value => "NONE" }, | 2403 | { key => "TEST_TARGET", value => "NONE" }, | ||
| 2382 | ], | 2404 | ], | ||
| 2383 | }, | 2405 | }, | ||
| 2406 | { | ||||
| 2407 | name => "install_no_user", | ||||
| 2408 | settings => [ | ||||
| 2409 | { key => "INSTALL_NO_USER", value => "1" }, | ||||
| 2410 | ], | ||||
| 2411 | }, | ||||
| 2384 | ], | 2412 | ], | ||
| 2385 | } | 2413 | } | ||
| 1 | use base "anacondatest"; | 1 | use base "anacondatest"; | ||
|---|---|---|---|---|---|
| 2 | use strict; | 2 | use strict; | ||
| 3 | use testapi; | 3 | use testapi; | ||
| 4 | use utils; | 4 | use utils; | ||
| 5 | 5 | | |||
| 6 | sub type_user_password { | | |||
| 7 | my $self = shift; | | |||
| 8 | my $user_password = get_var("USER_PASSWORD") || "weakpassword"; | | |||
| 9 | if (get_var("SWITCHED_LAYOUT")) { | | |||
| 10 | # we double the password, the second time using the native | | |||
| 11 | # layout, so the password has both ASCII and native characters | | |||
| 12 | desktop_switch_layout "ascii", "anaconda"; | | |||
| 13 | type_very_safely $user_password; | | |||
| 14 | desktop_switch_layout "native", "anaconda"; | | |||
| 15 | type_very_safely $user_password; | | |||
| 16 | } | | |||
| 17 | else { | | |||
| 18 | type_very_safely $user_password; | | |||
| 19 | } | | |||
| 20 | } | | |||
| 21 | | ||||
| 22 | sub run { | 6 | sub run { | ||
| 23 | my $self = shift; | 7 | my $self = shift; | ||
| 24 | # Anaconda hub | 8 | # Anaconda hub | ||
| 25 | assert_screen "anaconda_main_hub", 300; # | 9 | assert_screen "anaconda_main_hub", 300; # | ||
| 26 | 10 | | |||
| 27 | # Begin installation | 11 | # Begin installation | ||
| 28 | # Sometimes, the 'slide in from the top' animation messes with | 12 | # Sometimes, the 'slide in from the top' animation messes with | ||
| 29 | # this - by the time we click the button isn't where it was any | 13 | # this - by the time we click the button isn't where it was any | ||
| Show All 21 Lines | 33 | else { | |||
| 51 | # type super safely. This doesn't really slow the test down | 35 | # type super safely. This doesn't really slow the test down | ||
| 52 | # as we still get done before the install process is complete. | 36 | # as we still get done before the install process is complete. | ||
| 53 | type_very_safely $root_password; | 37 | type_very_safely $root_password; | ||
| 54 | wait_screen_change { send_key "tab"; }; | 38 | wait_screen_change { send_key "tab"; }; | ||
| 55 | type_very_safely $root_password; | 39 | type_very_safely $root_password; | ||
| 56 | } | 40 | } | ||
| 57 | assert_and_click "anaconda_spoke_done"; | 41 | assert_and_click "anaconda_spoke_done"; | ||
| 58 | 42 | | |||
| 59 | # Set user details | 43 | # Wait out animation | ||
| 60 | # wait out animation | | |||
| 61 | sleep 3; | 44 | sleep 3; | ||
| 62 | my $user_login = get_var("USER_LOGIN") || "test"; | 45 | # Set user details | ||
| 63 | unless ($user_login eq 'false') { | 46 | anaconda_create_user() unless (get_var("USER_LOGIN") eq 'false' || get_var("INSTALL_NO_USER")); | ||
| 64 | assert_and_click "anaconda_install_user_creation"; | | |||
| 65 | assert_screen "anaconda_install_user_creation_screen"; | | |||
| 66 | # wait out animation | | |||
| 67 | wait_still_screen 2; | | |||
| 68 | type_very_safely $user_login; | | |||
| 69 | type_very_safely "\t\t\t\t"; | | |||
| 70 | $self->type_user_password(); | | |||
| 71 | wait_screen_change { send_key "tab"; }; | | |||
| 72 | wait_still_screen 2; | | |||
| 73 | $self->type_user_password(); | | |||
| 74 | # even with all our slow typing this still *sometimes* seems to | | |||
| 75 | # miss a character, so let's try again if we have a warning bar. | | |||
| 76 | # But not if we're installing with a switched layout, as those | | |||
| 77 | # will *always* result in a warning bar at this point (see below) | | |||
| 78 | if (!get_var("SWITCHED_LAYOUT") && check_screen "anaconda_warning_bar", 3) { | | |||
| 79 | wait_screen_change { send_key "shift-tab"; }; | | |||
| 80 | wait_still_screen 2; | | |||
| 81 | $self->type_user_password(); | | |||
| 82 | wait_screen_change { send_key "tab"; }; | | |||
| 83 | wait_still_screen 2; | | |||
| 84 | $self->type_user_password(); | | |||
| 85 | } | | |||
| 86 | assert_and_click "anaconda_install_user_creation_make_admin"; | | |||
| 87 | assert_and_click "anaconda_spoke_done"; | | |||
| 88 | # since 20170105, we will get a warning here when the password | | |||
| 89 | # contains non-ASCII characters. Assume only switched layouts | | |||
| 90 | # produce non-ASCII characters, though this isn't strictly true | | |||
| 91 | if (get_var('SWITCHED_LAYOUT') && check_screen "anaconda_warning_bar", 3) { | | |||
| 92 | wait_still_screen 1; | | |||
| 93 | assert_and_click "anaconda_spoke_done"; | | |||
| 94 | } | | |||
| 95 | } | | |||
| 96 | 47 | | |||
| 97 | # Check username (and hence keyboard layout) if non-English | 48 | # Check username (and hence keyboard layout) if non-English | ||
| 98 | if (get_var('LANGUAGE')) { | 49 | if (get_var('LANGUAGE')) { | ||
| 99 | assert_screen "anaconda_install_user_created"; | 50 | assert_screen "anaconda_install_user_created"; | ||
| 100 | } | 51 | } | ||
| 101 | 52 | | |||
| 102 | # With the slow typing - especially with SWITCHED_LAYOUT - we | 53 | # With the slow typing - especially with SWITCHED_LAYOUT - we | ||
| 103 | # may not complete user creation until anaconda reaches post-install, | 54 | # may not complete user creation until anaconda reaches post-install, | ||
| Show All 37 Lines | |||||
| 1 | use base "installedtest"; | 1 | use base "installedtest"; | ||
|---|---|---|---|---|---|
| 2 | use strict; | 2 | use strict; | ||
| 3 | use testapi; | 3 | use testapi; | ||
| 4 | use utils; | 4 | use utils; | ||
| 5 | 5 | | |||
| 6 | sub run { | 6 | sub run { | ||
| 7 | my $self = shift; | 7 | my $self = shift; | ||
| 8 | # If KICKSTART is set, then the wait_time needs to consider the | 8 | # If KICKSTART is set, then the wait_time needs to consider the | ||
| 9 | # install time. if UPGRADE, we have to wait for the entire upgrade | 9 | # install time. if UPGRADE, we have to wait for the entire upgrade | ||
| 10 | # unless ENCRYPT_PASSWORD is set (in which case the postinstall | 10 | # unless ENCRYPT_PASSWORD is set (in which case the postinstall | ||
| 11 | # test does the waiting) | 11 | # test does the waiting) | ||
| 12 | my $wait_time = 300; | 12 | my $wait_time = 300; | ||
| 13 | $wait_time = 1800 if (get_var("KICKSTART")); | 13 | $wait_time = 1800 if (get_var("KICKSTART")); | ||
| 14 | $wait_time = 6000 if (get_var("UPGRADE") && !get_var("ENCRYPT_PASSWORD")); | 14 | $wait_time = 6000 if (get_var("UPGRADE") && !get_var("ENCRYPT_PASSWORD")); | ||
| 15 | 15 | | |||
| 16 | # Handle pre-login initial setup if we're doing INSTALL_NO_USER | ||||
| 17 | if (get_var("INSTALL_NO_USER") && !get_var("_setup_done")) { | ||||
| 18 | if (get_var("DESKTOP") eq 'gnome') { | ||||
| 19 | gnome_initial_setup(prelogin=>1, timeout=>$wait_time); | ||||
| 20 | } | ||||
| 21 | else { | ||||
| 22 | anaconda_create_user(timeout=>$wait_time); | ||||
| 23 | assert_and_click "initialsetup_finish_configuration"; | ||||
| 24 | set_var("_setup_done", 1); | ||||
| 25 | } | ||||
| 26 | $wait_time = 300; | ||||
| 27 | } | ||||
| 16 | # Wait for the login screen | 28 | # Wait for the login screen | ||
| 17 | boot_to_login_screen(timeout => $wait_time); | 29 | boot_to_login_screen(timeout => $wait_time); | ||
| 18 | # do user login unless USER_LOGIN is set to string 'false' | 30 | # do user login unless USER_LOGIN is set to string 'false' | ||
| 19 | unless (get_var("USER_LOGIN") eq "false") { | 31 | unless (get_var("USER_LOGIN") eq "false") { | ||
| 20 | if (get_var("DESKTOP") eq 'gnome') { | 32 | if (get_var("DESKTOP") eq 'gnome') { | ||
| 21 | # we have to hit enter to get the password dialog | 33 | # we have to hit enter to get the password dialog | ||
| 22 | send_key "ret"; | 34 | send_key "ret"; | ||
| 23 | } | 35 | } | ||
| Show All 13 Lines | |||||
| 37 | } | 49 | } | ||
| 38 | send_key "ret"; | 50 | send_key "ret"; | ||
| 39 | 51 | | |||
| 40 | # Handle initial-setup, for GNOME, unless START_AFTER_TEST | 52 | # Handle initial-setup, for GNOME, unless START_AFTER_TEST | ||
| 41 | # is set in which case it will have been done already. Always | 53 | # is set in which case it will have been done already. Always | ||
| 42 | # do it if ADVISORY is set, as for the update testing flow, | 54 | # do it if ADVISORY is set, as for the update testing flow, | ||
| 43 | # START_AFTER_TEST is set but a no-op and this hasn't happened | 55 | # START_AFTER_TEST is set but a no-op and this hasn't happened | ||
| 44 | if (get_var("DESKTOP") eq 'gnome' && (get_var("ADVISORY") || !get_var("START_AFTER_TEST"))) { | 56 | if (get_var("DESKTOP") eq 'gnome' && (get_var("ADVISORY") || !get_var("START_AFTER_TEST"))) { | ||
| 45 | # as this test gets loaded twice on the ADVISORY flow, | 57 | # as this test gets loaded twice on the ADVISORY flow, and | ||
| 46 | # keep track of whether this happened already | 58 | # we might be on the INSTALL_NO_USER flow, check whether | ||
| 47 | unless (get_var("_gis_done")) { | 59 | # this happened already | ||
| 48 | assert_screen "next_button", 120; | 60 | unless (get_var("_setup_done")) { | ||
| 49 | # wait a bit in case of animation | 61 | gnome_initial_setup(); | ||
| 50 | wait_still_screen 3; | | |||
| 51 | for my $n (1..3) { | | |||
| 52 | # click 'Next' three times, moving the mouse to avoid | | |||
| 53 | # highlight problems, sleeping to give it time to get | | |||
| 54 | # to the next screen between clicks | | |||
| 55 | mouse_set(100, 100); | | |||
| 56 | wait_screen_change { assert_and_click "next_button"; }; | | |||
| 57 | # for Japanese, we need to workaround a bug on the keyboard | | |||
| 58 | # selection screen | | |||
| 59 | if ($n == 1 && get_var("LANGUAGE") eq 'japanese') { | | |||
| 60 | if (!check_screen 'initial_setup_kana_kanji_selected', 5) { | | |||
| 61 | record_soft_failure 'kana kanji not selected: bgo#776189'; | | |||
| 62 | assert_and_click 'initial_setup_kana_kanji'; | | |||
| 63 | } | | |||
| 64 | } | 62 | } | ||
| 65 | } | 63 | } | ||
| 66 | # click 'Skip' one time | 64 | if (get_var("DESKTOP") eq 'gnome' && get_var("INSTALL_NO_USER")) { | ||
| 67 | mouse_set(100,100); | | |||
| 68 | wait_screen_change { assert_and_click "skip_button"; }; | | |||
| 69 | send_key "ret"; | | |||
| 70 | # wait for the stupid 'help' screen to show and kill it | 65 | # wait for the stupid 'help' screen to show and kill it | ||
| 71 | assert_screen "getting_started"; | 66 | assert_screen "getting_started"; | ||
| 72 | send_key "alt-f4"; | 67 | send_key "alt-f4"; | ||
| 73 | wait_still_screen 5; | 68 | wait_still_screen 5; | ||
| 74 | # don't do it again on second load | | |||
| 75 | set_var("_gis_done", 1); | | |||
| 76 | } | | |||
| 77 | } | 69 | } | ||
| 78 | 70 | | |||
| 79 | # Move the mouse somewhere it won't highlight the match areas | 71 | # Move the mouse somewhere it won't highlight the match areas | ||
| 80 | mouse_set(300, 200); | 72 | mouse_set(300, 200); | ||
| 81 | # KDE can take ages to start up | 73 | # KDE can take ages to start up | ||
| 82 | assert_screen "graphical_desktop_clean", 120; | 74 | assert_screen "graphical_desktop_clean", 120; | ||
| 83 | } | 75 | } | ||
| 84 | } | 76 | } | ||
| Show All 12 Lines | |||||
I don't know why they are keep showing it. Wouldn't something like "Help" button on last screen in g-i-s be enough?