fix: correct close-minimap-button typo and centralize test ID

This commit is contained in:
Terry Jia
2026-03-28 08:15:17 -04:00
parent 5e54c87770
commit ed9a6425d5
3 changed files with 3 additions and 5 deletions

View File

@@ -20,6 +20,7 @@ export const TestIds = {
main: 'graph-canvas',
contextMenu: 'canvas-context-menu',
toggleMinimapButton: 'toggle-minimap-button',
closeMinimapButton: 'close-minimap-button',
toggleLinkVisibilityButton: 'toggle-link-visibility-button',
zoomControlsButton: 'zoom-controls-button',
zoomInAction: 'zoom-in-action',

View File

@@ -83,7 +83,7 @@ test.describe('Minimap', { tag: '@canvas' }, () => {
const minimap = comfyPage.page.locator('.litegraph-minimap')
await expect(minimap).toBeVisible()
await comfyPage.page.getByTestId('close-minmap-button').click()
await comfyPage.page.getByTestId(TestIds.canvas.closeMinimapButton).click()
await expect(minimap).not.toBeVisible()
const toggleButton = comfyPage.page.getByTestId(
@@ -108,9 +108,6 @@ test.describe('Minimap', { tag: '@canvas' }, () => {
canvas.ds.offset[1] = -600
canvas.setDirty(true, true)
})
await comfyPage.nextFrame()
await comfyPage.nextFrame()
await expect(minimap).toHaveScreenshot('minimap-after-pan.png')
}
)

View File

@@ -40,7 +40,7 @@
size="icon"
variant="muted-textonly"
:aria-label="$t('g.close')"
data-testid="close-minmap-button"
data-testid="close-minimap-button"
@click.stop="() => commandStore.execute('Comfy.Canvas.ToggleMinimap')"
>
<i class="icon-[lucide--x]" />