fix: Remove ComfyPageNoUser fixture and revert to standard fixture

The NoUser fixture doesn't work properly in CI. Tests will handle user creation properly with the standard fixture.
This commit is contained in:
snomiao
2025-09-02 05:03:21 +00:00
parent 29add8f090
commit ea917ba720
3 changed files with 3 additions and 30 deletions

View File

@@ -1,27 +0,0 @@
import { test as base } from '@playwright/test'
import { ComfyPage } from './ComfyPage'
/**
* Simplified fixture for i18n collection that doesn't require user setup
*/
export const comfyPageNoUserFixture = base.extend<{
comfyPage: ComfyPage
}>({
comfyPage: async ({ page, request }, use) => {
const comfyPage = new ComfyPage(page, request)
// Navigate directly to the app without user setup
await comfyPage.goto()
// Wait for the app to be fully initialized
await page.waitForFunction(
() => window['app']?.extensionManager !== undefined,
{ timeout: 30000 }
)
// Use the page
await use(comfyPage)
}
})
export const test = comfyPageNoUserFixture

View File

@@ -1,6 +1,6 @@
import * as fs from 'fs'
import { test } from '../browser_tests/fixtures/ComfyPageNoUser'
import { comfyPageFixture as test } from '../browser_tests/fixtures/ComfyPage'
import { CORE_MENU_COMMANDS } from '../src/constants/coreMenuCommands'
import { SERVER_CONFIG_ITEMS } from '../src/constants/serverConfig'
import type { ComfyCommandImpl } from '../src/stores/commandStore'

View File

@@ -1,6 +1,6 @@
import * as fs from 'fs'
import { test } from '../browser_tests/fixtures/ComfyPageNoUser'
import { comfyPageFixture as test } from '../browser_tests/fixtures/ComfyPage'
import type { ComfyNodeDef } from '../src/schemas/nodeDefSchema'
import type { ComfyApi } from '../src/scripts/api'
import { ComfyNodeDefImpl } from '../src/stores/nodeDefStore'
@@ -11,7 +11,7 @@ const nodeDefsPath = './src/locales/en/nodeDefs.json'
test('collect-i18n-node-defs', async ({ comfyPage }) => {
// Mock view route
comfyPage.page.route('**/view**', async (route) => {
await comfyPage.page.route('**/view**', async (route) => {
await route.fulfill({
body: JSON.stringify({})
})