Skip to content

Commit 271643a

Browse files
[test] Fix failing test case that uses old subgraph breadcrumb element (#4537)
1 parent 35fb141 commit 271643a

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

browser_tests/tests/subgraph.spec.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -397,7 +397,8 @@ test.describe('Subgraph Operations', () => {
397397
// Enable new menu for breadcrumb navigation
398398
await comfyPage.setSetting('Comfy.UseNewMenu', 'Top')
399399

400-
await comfyPage.loadWorkflow('subgraph-with-promoted-text-widget')
400+
const workflowName = 'subgraph-with-promoted-text-widget'
401+
await comfyPage.loadWorkflow(workflowName)
401402

402403
const textareaCount = await comfyPage.page
403404
.locator(SELECTORS.domWidget)
@@ -420,8 +421,13 @@ test.describe('Subgraph Operations', () => {
420421
})
421422

422423
// Click breadcrumb to navigate back to parent graph
423-
const breadcrumb = comfyPage.page.locator(SELECTORS.breadcrumb).first()
424-
await breadcrumb.click()
424+
const homeBreadcrumb = comfyPage.page.getByRole('link', {
425+
// In the subgraph navigation breadcrumbs, the home/top level
426+
// breadcrumb is just the workflow name
427+
name: workflowName
428+
})
429+
await homeBreadcrumb.waitFor({ state: 'visible' })
430+
await homeBreadcrumb.click()
425431
await comfyPage.nextFrame()
426432
await comfyPage.page.waitForTimeout(300)
427433

0 commit comments

Comments
 (0)