Skip to content

Commit 5f10e9c

Browse files
committed
test: Fix TestFirewall.testAddServices
1 parent 706cf1c commit 5f10e9c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/verify/check-networkmanager-firewall

+3-3
Original file line numberDiff line numberDiff line change
@@ -279,7 +279,7 @@ class TestFirewall(netlib.NetworkCase):
279279
b.assert_pixels(".pf-v6-c-modal-box", "firewall-add-services-to-zone-modal", wait_delay=3)
280280

281281
# don't select anything in the dialog
282-
b.wait_visible(f"#add-services-dialog .{self.btn_primary}.pf-m-disabled")
282+
b.wait_visible(f"#add-services-dialog .{self.btn_primary}:disabled")
283283
b.click("#add-services-dialog footer .btn-cancel")
284284
b.wait_not_present(".pf-v6-c-modal-box")
285285

@@ -323,7 +323,7 @@ class TestFirewall(netlib.NetworkCase):
323323
# belonging either libvirt or public
324324
for service in services:
325325
b.click(f".zone-section[data-id='public'] tr[data-row-id='{service}'] button.pf-v6-c-menu-toggle")
326-
b.click(".zone-section[data-id='public'] .pf-v6-c-menu__list-item.pf-m-danger button")
326+
b.click(".pf-v6-c-menu__list-item.pf-m-danger button")
327327
# removing cockpit services requires confirmation
328328
if service == "cockpit":
329329
b.click(f"#delete-confirmation-dialog button.{self.btn_danger}")
@@ -340,7 +340,7 @@ class TestFirewall(netlib.NetworkCase):
340340
b.wait_visible(".zone-section[data-id='public'] td:contains('Additional ports') + td:contains('9998')")
341341
# remove service in cockpit
342342
b.click(".zone-section[data-id='public'] tr[data-row-id='pop3'] button.pf-v6-c-menu-toggle")
343-
b.click(".zone-section[data-id='public'] .pf-v6-c-menu__list-item.pf-m-danger button")
343+
b.click(".pf-v6-c-menu__list-item.pf-m-danger button")
344344
b.wait_not_present(".zone-section[data-id='public'] tr[data-row-id='pop3']")
345345

346346
def testAddCustomServices(self):

0 commit comments

Comments
 (0)