Comfy-Org/ComfyUI_frontend
1
0
Fork 0
mirror of https://github.com/Comfy-Org/ComfyUI_frontend.git synced 2026-04-20 06:20:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: integrate AssetHelper cleanup into Playwright fixture teardown

Browse Source 
Adds automatic clearMocks() in the comfyPageFixture teardown phase so
test cleanup is handled by the fixture lifecycle instead of requiring
manual calls inside test cases.

Addresses review feedback:
https://github.com/Comfy-Org/ComfyUI_frontend/pull/10545#discussion_r3003581474
This commit is contained in:
bymyself
2026-03-28 14:22:35 -07:00
parent 802c52ae1f
commit 9424b89893
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
browser_tests/fixtures/ComfyPage.ts
View File

@@ -26,7 +26,8 @@ import {
import { Topbar } from './components/Topbar'
import { AssetsHelper } from './helpers/AssetsHelper'
import { CanvasHelper } from './helpers/CanvasHelper'
import { AssetHelper } from './helpers/AssetHelper'
import type { AssetHelper } from './helpers/AssetHelper'
import { createAssetHelper } from './helpers/AssetHelper'
import { PerformanceHelper } from './helpers/PerformanceHelper'
import { QueueHelper } from './helpers/QueueHelper'
import { ClipboardHelper } from './helpers/ClipboardHelper'
@@ -202,8 +203,8 @@ export class ComfyPage {
public readonly bottomPanel: BottomPanel
public readonly perf: PerformanceHelper
public readonly assets: AssetsHelper
public readonly assetApi: AssetHelper
public readonly queue: QueueHelper
public readonly assets: AssetHelper
/** Worker index to test user ID */
public readonly userIds: string[] = []
@@ -250,8 +251,8 @@ export class ComfyPage {
this.bottomPanel = new BottomPanel(page)
this.perf = new PerformanceHelper(page)
this.assets = new AssetsHelper(page)
this.assetApi = createAssetHelper(page)
this.queue = new QueueHelper(page)
this.assets = new AssetHelper(page)
}
get visibleToasts() {
@@ -471,6 +472,7 @@ export const comfyPageFixture = base.extend<{
await use(comfyPage)
await comfyPage.assetApi.clearMocks()
if (needsPerf) await comfyPage.perf.dispose()
},
comfyMouse: async ({ comfyPage }, use) => {