1
0
mirror of https://pagure.io/fedora-qa/os-autoinst-distri-fedora.git synced 2024-11-21 21:43:08 +00:00

Simplify blivet_btrfs_preserve_home, drop unneeded needles

There's no need to do all this 'check whether it's selected and
click it if not' stuff (for three different mount points). Just
always click it. If it's already selected, clicking it again
doesn't hurt (one of these stanzas even clicks it *even if it's
selected*!)

If we need to cover both cases, we just need two needles with
the same tag, we don't need separate code paths. In each case,
though, we actually haven't matched one of the needles for ages
(the most recent was part_boot_selected, but now we're using
GPT by default, we won't hit that any more as it'll be the BIOS
boot partition that's selected by default), so delete the needles
we aren't matching any more. If we *do* hit any case where we
need to handle the 'other' state, we can just add the alternative
needle with the same tag.

Signed-off-by: Adam Williamson <awilliam@redhat.com>
This commit is contained in:
Adam Williamson 2023-04-28 17:05:37 -07:00
parent 73e9c5c27e
commit c7e0af6e8c
15 changed files with 15 additions and 68 deletions

View File

@ -10,6 +10,6 @@
], ],
"properties": [], "properties": [],
"tags": [ "tags": [
"anaconda_blivet_part_inactive_boot" "anaconda_blivet_part_boot"
] ]
} }

View File

@ -10,6 +10,6 @@
], ],
"properties": [], "properties": [],
"tags": [ "tags": [
"anaconda_blivet_part_inactive_boot" "anaconda_blivet_part_boot"
] ]
} }

View File

@ -1,15 +0,0 @@
{
"area": [
{
"xpos": 342,
"ypos": 307,
"width": 112,
"height": 18,
"type": "match"
}
],
"properties": [],
"tags": [
"anaconda_blivet_part_boot_selected"
]
}

Binary file not shown.

Before

Width:  |  Height:  |  Size: 70 KiB

View File

@ -10,6 +10,6 @@
], ],
"properties": [], "properties": [],
"tags": [ "tags": [
"anaconda_blivet_part_efi_selected" "anaconda_blivet_part_efi"
] ]
} }

View File

@ -10,6 +10,6 @@
], ],
"properties": [], "properties": [],
"tags": [ "tags": [
"anaconda_blivet_part_inactive_home" "anaconda_blivet_part_home"
] ]
} }

View File

@ -1,15 +0,0 @@
{
"area": [
{
"xpos": 264,
"height": 19,
"ypos": 330,
"width": 40,
"type": "match"
}
],
"properties": [],
"tags": [
"anaconda_blivet_part_home_selected"
]
}

Binary file not shown.

Before

Width:  |  Height:  |  Size: 76 KiB

View File

@ -1,15 +0,0 @@
{
"area": [
{
"height": 17,
"width": 108,
"ypos": 308,
"type": "match",
"xpos": 340
}
],
"properties": [],
"tags": [
"anaconda_blivet_part_inactive_efi"
]
}

Binary file not shown.

Before

Width:  |  Height:  |  Size: 74 KiB

View File

@ -13,11 +13,10 @@ sub run {
if (get_var("UEFI")) { if (get_var("UEFI")) {
# if we're running on UEFI, let us reformat the UEFI first # if we're running on UEFI, let us reformat the UEFI first
# Select the UEFI partition if it is not selected by default # Select the UEFI partition if it is not selected by default;
if (not(check_screen "anaconda_blivet_part_efi_selected", 30)) { # if it is, clicking it is harmless
assert_and_click "anaconda_blivet_part_inactive_efi"; assert_and_click "anaconda_blivet_part_efi";
wait_still_screen 5; wait_still_screen 5;
}
# Go to the partition settings # Go to the partition settings
assert_and_click "anaconda_blivet_part_edit"; assert_and_click "anaconda_blivet_part_edit";
@ -35,13 +34,8 @@ sub run {
} }
# Reformat the /boot partition # Reformat the /boot partition
if (check_screen "anaconda_blivet_part_boot_selected", 30) { assert_and_click "anaconda_blivet_part_boot";
assert_and_click "anaconda_blivet_part_boot_selected";
}
else {
assert_and_click "anaconda_blivet_part_inactive_boot";
wait_still_screen 5; wait_still_screen 5;
}
# Go to the partition settings # Go to the partition settings
assert_and_click "anaconda_blivet_part_edit"; assert_and_click "anaconda_blivet_part_edit";
@ -60,9 +54,7 @@ sub run {
assert_and_click "anaconda_blivet_volumes_icon"; assert_and_click "anaconda_blivet_volumes_icon";
# Select the home partition # Select the home partition
if (not(check_screen "anaconda_blivet_part_home_selected")) { assert_and_click "anaconda_blivet_part_home";
assert_and_click "anaconda_blivet_part_inactive_home";
}
# Go to the partition settings # Go to the partition settings
assert_and_click "anaconda_blivet_part_edit"; assert_and_click "anaconda_blivet_part_edit";
# Select the Set mountpoint option # Select the Set mountpoint option