1
0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2024-12-22 10:23:09 +00:00

Conditionalize FreeIPA UI change, add 4.8.9 update to workarounds

The FreeIPA UI change that the previous commit adapted to is in
4.8.9. That's stable for Rawhide and F33 already, but still in
testing for F32, and won't go to F31. So we need to make the
change conditional on release number, and we also add the update
to workarounds for F32 so we don't have to do something awkward
while we wait for it to go stable.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2020-08-21 14:00:42 -07:00
parent 5713631a9c
commit b9f6ecd72d
2 changed files with 7 additions and 4 deletions

View File

@ -440,9 +440,9 @@ sub setup_workaround_repo {
assert_script_run "pushd /opt/workarounds_repo"; assert_script_run "pushd /opt/workarounds_repo";
my %workarounds = ( my %workarounds = (
"31" => [], "31" => [],
# krb5 reversions to fix RHBZ#1868482 # FreeIPA 4.8.9 to make the UI consistent across releases
"32" => ["FEDORA-2020-d10a284af3"], "32" => ["FEDORA-2020-dc32a8de93"],
"33" => ["FEDORA-2020-58970d693c"] "33" => []
); );
# then we'll download each update for our release: # then we'll download each update for our release:
my $advisories = $workarounds{$version}; my $advisories = $workarounds{$version};

View File

@ -24,7 +24,10 @@ sub run {
assert_and_click "freeipa_webui_reset_password_link"; assert_and_click "freeipa_webui_reset_password_link";
wait_still_screen 3; wait_still_screen 3;
type_safely "batterystaple"; type_safely "batterystaple";
type_safely "\t"; # The next box we need to type into was moved in FreeIPA 4.8.9,
# which is in F32+ but not F31
my $relnum = get_release_number;
$relnum < 32 ? type_safely "\t\t" : type_safely "\t";
type_safely "loremipsum"; type_safely "loremipsum";
wait_screen_change { send_key "tab"; }; wait_screen_change { send_key "tab"; };
type_safely "loremipsum"; type_safely "loremipsum";