draft test for issue

.github/workflows/test-ui.yaml

@@ -46,8 +46,8 @@ jobs:
         id: cache-key
         run: echo "key=$(date +%s)" >> $GITHUB_OUTPUT
 
-      - name: Save cache
-        uses: actions/cache/save@5a3ec84eff668545956fd18022155c47e93e2684
+      - name: Cache setup
+        uses: actions/cache@v3
         with:
           path: |
             ComfyUI
@@ -62,13 +62,9 @@ jobs:
       matrix:
         browser: [chromium, chromium-2x, mobile-chrome]
     steps:
-      - name: Wait for cache propagation
-        run: sleep 10
-
       - name: Restore cached setup
-        uses: actions/cache/restore@5a3ec84eff668545956fd18022155c47e93e2684
+        uses: actions/cache@v3
         with:
-          fail-on-cache-miss: true
           path: |
             ComfyUI
             ComfyUI_frontend

browser_tests/fixtures/ComfyPage.ts

@@ -1046,8 +1046,6 @@ export class ComfyPage {
   }
 }
 
-export const testComfySnapToGridGridSize = 50
-
 export const comfyPageFixture = base.extend<{
   comfyPage: ComfyPage
   comfyMouse: ComfyMouse
@@ -1075,7 +1073,8 @@ export const comfyPageFixture = base.extend<{
         'Comfy.userId': userId,
         // Set tutorial completed to true to avoid loading the tutorial workflow.
         'Comfy.TutorialCompleted': true,
-        'Comfy.SnapToGrid.GridSize': testComfySnapToGridGridSize
+        // Set installed version for consistent test behavior.
+        'Comfy.InstalledVersion': '1.22.0'
       })
     } catch (e) {
       console.error(e)
@@ -1086,7 +1085,7 @@ export const comfyPageFixture = base.extend<{
   },
   comfyMouse: async ({ comfyPage }, use) => {
     const comfyMouse = new ComfyMouse(comfyPage)
-    use(comfyMouse)
+    await use(comfyMouse)
   }
 })
 

browser_tests/tests/interaction.spec.ts

@@ -1,12 +1,6 @@
 import { expect } from '@playwright/test'
-import { Position } from '@vueuse/core'
 
-import {
-  type ComfyPage,
-  comfyPageFixture as test,
-  testComfySnapToGridGridSize
-} from '../fixtures/ComfyPage'
-import { type NodeReference } from '../fixtures/utils/litegraphUtils'
+import { type ComfyPage, comfyPageFixture as test } from '../fixtures/ComfyPage'
 
 test.describe('Item Interaction', () => {
   test('Can select/delete all items', async ({ comfyPage }) => {
@@ -63,10 +57,8 @@ test.describe('Node Interaction', () => {
       await expect(comfyPage.canvas).toHaveScreenshot('selected-node2.png')
     })
 
-    const dragSelectNodes = async (
-      comfyPage: ComfyPage,
-      clipNodes: NodeReference[]
-    ) => {
+    test('Can drag-select nodes with Meta (mac)', async ({ comfyPage }) => {
+      const clipNodes = await comfyPage.getNodeRefsByType('CLIPTextEncode')
       const clipNode1Pos = await clipNodes[0].getPosition()
       const clipNode2Pos = await clipNodes[1].getPosition()
       const offset = 64
@@ -82,67 +74,10 @@ test.describe('Node Interaction', () => {
         }
       )
       await comfyPage.page.keyboard.up('Meta')
-    }
-
-    test('Can drag-select nodes with Meta (mac)', async ({ comfyPage }) => {
-      const clipNodes = await comfyPage.getNodeRefsByType('CLIPTextEncode')
-      await dragSelectNodes(comfyPage, clipNodes)
       expect(await comfyPage.getSelectedGraphNodesCount()).toBe(
         clipNodes.length
       )
     })
-
-    test('Can move selected nodes using the Comfy.Canvas.MoveSelectedNodes.{Up|Down|Left|Right} commands', async ({
-      comfyPage
-    }) => {
-      const clipNodes = await comfyPage.getNodeRefsByType('CLIPTextEncode')
-      const getPositions = () =>
-        Promise.all(clipNodes.map((node) => node.getPosition()))
-      const testDirection = async ({
-        direction,
-        expectedPosition
-      }: {
-        direction: string
-        expectedPosition: (originalPosition: Position) => Position
-      }) => {
-        const originalPositions = await getPositions()
-        await dragSelectNodes(comfyPage, clipNodes)
-        await comfyPage.executeCommand(
-          `Comfy.Canvas.MoveSelectedNodes.${direction}`
-        )
-        await comfyPage.canvas.press(`Control+Arrow${direction}`)
-        const newPositions = await getPositions()
-        expect(newPositions).toEqual(originalPositions.map(expectedPosition))
-      }
-      await testDirection({
-        direction: 'Down',
-        expectedPosition: (originalPosition) => ({
-          ...originalPosition,
-          y: originalPosition.y + testComfySnapToGridGridSize
-        })
-      })
-      await testDirection({
-        direction: 'Right',
-        expectedPosition: (originalPosition) => ({
-          ...originalPosition,
-          x: originalPosition.x + testComfySnapToGridGridSize
-        })
-      })
-      await testDirection({
-        direction: 'Up',
-        expectedPosition: (originalPosition) => ({
-          ...originalPosition,
-          y: originalPosition.y - testComfySnapToGridGridSize
-        })
-      })
-      await testDirection({
-        direction: 'Left',
-        expectedPosition: (originalPosition) => ({
-          ...originalPosition,
-          x: originalPosition.x - testComfySnapToGridGridSize
-        })
-      })
-    })
   })
 
   test('Can drag node', async ({ comfyPage }) => {