From a1237be4863e23f9a6dcacb89ad64e7943e942eb Mon Sep 17 00:00:00 2001 From: DrJKL Date: Mon, 8 Dec 2025 14:33:50 -0800 Subject: [PATCH 1/4] hotfix: Templates spec --- browser_tests/tests/templates.spec.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/browser_tests/tests/templates.spec.ts b/browser_tests/tests/templates.spec.ts index 6252332135..d283f098f2 100644 --- a/browser_tests/tests/templates.spec.ts +++ b/browser_tests/tests/templates.spec.ts @@ -171,7 +171,7 @@ test.describe('Templates', () => { // Verify English titles are shown as fallback await expect( comfyPage.templates.content.getByRole('heading', { - name: 'Image Generation' + name: 'All Templates' }) ).toBeVisible() }) From 79e8ca82738e20fb11f00c573edd666bb1a66e46 Mon Sep 17 00:00:00 2001 From: DrJKL Date: Mon, 8 Dec 2025 14:49:36 -0800 Subject: [PATCH 2/4] test: Simpler assertion, getByText --- browser_tests/tests/templates.spec.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/browser_tests/tests/templates.spec.ts b/browser_tests/tests/templates.spec.ts index d283f098f2..a953c53795 100644 --- a/browser_tests/tests/templates.spec.ts +++ b/browser_tests/tests/templates.spec.ts @@ -170,9 +170,7 @@ test.describe('Templates', () => { // Verify English titles are shown as fallback await expect( - comfyPage.templates.content.getByRole('heading', { - name: 'All Templates' - }) + comfyPage.templates.content.getByText('All Templates') ).toBeVisible() }) From 80120d015e1126724df580cab7102d09a1ea8aa5 Mon Sep 17 00:00:00 2001 From: DrJKL Date: Mon, 8 Dec 2025 14:58:17 -0800 Subject: [PATCH 3/4] test: Except that's not under the content :-| --- browser_tests/tests/templates.spec.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/browser_tests/tests/templates.spec.ts b/browser_tests/tests/templates.spec.ts index a953c53795..d168cc93e7 100644 --- a/browser_tests/tests/templates.spec.ts +++ b/browser_tests/tests/templates.spec.ts @@ -169,9 +169,7 @@ test.describe('Templates', () => { expect(englishRequest.url()).toContain('templates/index.json') // Verify English titles are shown as fallback - await expect( - comfyPage.templates.content.getByText('All Templates') - ).toBeVisible() + await expect(comfyPage.page.getByText('All Templates')).toBeVisible() }) test('template cards are dynamically sized and responsive', async ({ From 1735ca8f7dc448ddf8ab84f2ad9e0859f5dc6bc0 Mon Sep 17 00:00:00 2001 From: DrJKL Date: Mon, 8 Dec 2025 15:00:30 -0800 Subject: [PATCH 4/4] test: There are two of them --- browser_tests/tests/templates.spec.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/browser_tests/tests/templates.spec.ts b/browser_tests/tests/templates.spec.ts index d168cc93e7..1e0d24dd68 100644 --- a/browser_tests/tests/templates.spec.ts +++ b/browser_tests/tests/templates.spec.ts @@ -169,7 +169,9 @@ test.describe('Templates', () => { expect(englishRequest.url()).toContain('templates/index.json') // Verify English titles are shown as fallback - await expect(comfyPage.page.getByText('All Templates')).toBeVisible() + await expect( + comfyPage.page.getByRole('main').getByText('All Templates') + ).toBeVisible() }) test('template cards are dynamically sized and responsive', async ({