Skip to content

Commit 876349f

Browse files
mvollmermartinpitt
authored andcommitted
test: Take TestSystemInfo-testBasic-overview pixel test without CPU warning
The size of the progress bars changes too much depending on whether the icon is there or not. Fixes #21669
1 parent cdcef6b commit 876349f

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

test/verify/check-system-info

+3-1
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,12 @@ class TestSystemInfo(testlib.MachineCase):
102102
# need to wait until CPU usage settles down, to avoid a layout-shifting error/warning icon
103103
b.wait_not_present("#system-usage-cpu-progress + td .pf-v5-c-progress__status-icon")
104104
testlib.wait(lambda: b.get_pf_progress_value("#system-usage-cpu-progress + td") < 30)
105-
# Wait for tuned text to appear, even thought we ignore it we need to button to have some text.
106105
if b.pixels_label:
106+
# Wait for tuned text to appear, even thought we ignore it we need to button to have some text.
107107
recommended_profile = m.execute("tuned-adm recommend").strip()
108108
b.wait_text("#tuned-status-button", recommended_profile)
109+
# Wait for there to be no warning icons in the usage.
110+
b.wait_not_present('#system-usage .pf-v5-c-progress__status-icon')
109111
b.assert_pixels("#overview", "overview", ignore=[
110112
".system-health .pf-v5-c-card__body",
111113
"#system_uptime",

0 commit comments

Comments
 (0)