Skip to content

Commit d8c5571

Browse files
committed
fix: correct channel glossary test locators (dashboard not timeline)
1 parent 7202d25 commit d8c5571

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

tests/e2e/test_glossary.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,15 +71,11 @@ class TestGlossaryChannels:
7171
"""Verify glossary hints in channel tables."""
7272

7373
def test_ds_channel_group_has_glossary_hint(self, demo_page):
74-
demo_page.locator('a.nav-item[data-view="channels"]').click()
75-
demo_page.wait_for_timeout(500)
76-
hint = demo_page.locator('#view-channels .docsis-group-header .glossary-hint').first
74+
hint = demo_page.locator('#view-dashboard .docsis-group-header .glossary-hint').first
7775
expect(hint).to_be_visible()
7876

7977
def test_channel_glossary_popover_works(self, demo_page):
80-
demo_page.locator('a.nav-item[data-view="channels"]').click()
81-
demo_page.wait_for_timeout(500)
82-
hint = demo_page.locator('#view-channels .docsis-group-header .glossary-hint').first
78+
hint = demo_page.locator('#view-dashboard .docsis-group-header .glossary-hint').first
8379
hint.click()
8480
popover = hint.locator('.glossary-popover')
8581
expect(popover).to_be_visible()

tests/e2e/test_glossary_visual.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,8 @@ def test_screenshot_popover_open(self, demo_page):
4040
)
4141

4242
def test_screenshot_channel_popover(self, demo_page):
43-
demo_page.locator('a.nav-item[data-view="channels"]').click()
44-
demo_page.wait_for_timeout(500)
45-
hint = demo_page.locator('#view-channels .docsis-group-header .glossary-hint').first
43+
"""Screenshot channel group header with glossary popover (on dashboard)."""
44+
hint = demo_page.locator('#view-dashboard .docsis-group-header .glossary-hint').first
4645
hint.click()
4746
demo_page.wait_for_timeout(300)
4847
demo_page.screenshot(

0 commit comments

Comments
 (0)