test: harden linked widget regression coverage

Author: jaeone94

browser_tests/tests/vueNodes/widgets/int/integerWidget.spec.ts

@@ -53,7 +53,7 @@ test.describe('Vue Integer Widget', { tag: '@vue-nodes' }, () => {
     await expect(controls.input).toHaveValue(initialValue.toString())
     await expect
       .poll(async () => (await samplerNode.boundingBox())?.height)
-      .toBe(nodeBounds.height)
+      .toBeCloseTo(nodeBounds.height, 0)
   })
 
   test('displays control widgets with default state', async ({ comfyPage }) => {

browser_tests/tests/vueNodes/widgets/linked/linkedStandardWidgets.spec.ts

@@ -98,18 +98,16 @@ test.describe(
         ).toBe(false)
       }
 
-      await expect(
-        targetNode.getByText('STALE SELECT VALUE', { exact: true })
-      ).toBeHidden()
-      await expect(
-        targetNode.getByText('STALE ON', { exact: true })
-      ).toBeHidden()
-      await expect(
-        targetNode.getByText('STALE OFF', { exact: true })
-      ).toBeHidden()
-      await expect(
-        targetNode.getByText('#22c55d', { exact: true })
-      ).toBeHidden()
+      for (const text of [
+        'STALE SELECT VALUE',
+        'STALE ON',
+        'STALE OFF',
+        '#22c55d'
+      ]) {
+        const staleValue = targetNode.getByText(text, { exact: true })
+        await expect(staleValue).toBeAttached()
+        await expect(staleValue).toBeHidden()
+      }
 
       const widgetRefs = await Promise.all(
         WIDGET_NAMES.map((name) => targetNodeRef.getWidgetByName(name))
@@ -127,14 +125,7 @@ test.describe(
 
       for (let index = 0; index < WIDGET_NAMES.length + 2; index++) {
         await comfyPage.page.keyboard.press('Tab')
-        expect(
-          await targetNode.evaluate(
-            (node) =>
-              node.querySelector(
-                '[data-testid="linked-widget-content"]:focus-within'
-              ) !== null
-          )
-        ).toBe(false)
+        await expect(linkedContent.locator(':focus-within')).toHaveCount(0)
       }
 
       await expect

browser_tests/tests/vueNodes/widgets/text/multilineStringWidget.spec.ts

@@ -93,7 +93,7 @@ test.describe('Vue Multiline String Widget', { tag: '@vue-nodes' }, () => {
     await expect(restoredTextarea).toHaveValue('restored local prompt')
     await expect
       .poll(async () => (await clipNode.boundingBox())?.height)
-      .toBe(nodeBounds.height)
+      .toBeCloseTo(nodeBounds.height, 0)
   })
 
   test('should use native context menu when focused', async ({ comfyPage }) => {

src/components/rightSidePanel/parameters/WidgetItem.test.ts

@@ -101,7 +101,9 @@ const i18n = createI18n({
   }
 })
 
-function createMockNode(overrides: Record<string, unknown> = {}): LGraphNode {
+function createMockNode(
+  overrides: Partial<Record<keyof LGraphNode, unknown>> = {}
+): LGraphNode {
   return fromAny<LGraphNode, unknown>({
     id: 1,
     type: 'TestNode',
@@ -160,6 +162,7 @@ describe('WidgetItem', () => {
     mockIsAssetAPIEnabled.mockReturnValue(false)
     mockShouldUseAssetBrowser.mockReturnValue(false)
     mockFromLGraphNode.mockReturnValue(null)
+    mockGetInputSpecForWidget.mockReset()
   })
 
   describe('widget state rendering', () => {

src/renderer/extensions/vueNodes/widgets/components/form/dropdown/FormDropdown.test.ts

@@ -482,6 +482,23 @@ describe('FormDropdown', () => {
     expect(onUpdateIsOpen).toHaveBeenLastCalledWith(false)
   })
 
+  it('stays open when disabled without closeOnDisable', async () => {
+    const onUpdateIsOpen = vi.fn()
+    const items = [createItem('1', 'alpha')]
+    const { rerender, user } = mountDropdown(items, { onUpdateIsOpen })
+    await openDropdown(user)
+
+    expect(onUpdateIsOpen).toHaveBeenLastCalledWith(true)
+
+    await rerender({
+      items,
+      disabled: true,
+      'onUpdate:isOpen': onUpdateIsOpen
+    })
+
+    expect(onUpdateIsOpen).toHaveBeenLastCalledWith(true)
+  })
+
   it('stays open on a pointerdown inside the menu', async () => {
     const onUpdateIsOpen = vi.fn()
     const { user } = mountDropdown([createItem('1', 'alpha')], {