test(platform): confirm Revoke invite through its dialog and drop the withdrawn Assistant New chat check

Author: waleedlatif1

apps/sim/platform/checks/live/flows.spec.ts

@@ -149,6 +149,18 @@ test('a pending invite resends and revokes from its row', async ({ page }) => {
   await expect(page.getByRole('menuitem', { name: 'Resend invite' })).toHaveCount(0)
   await row.getByRole('button', { name: /^Actions for/ }).click()
   await page.getByRole('menuitem', { name: 'Revoke invite' }).click()
+  const confirm = page.getByRole('dialog', { name: 'Revoke invite' })
+  await expect(confirm).toContainText(email)
+  const [revoked] = await Promise.all([
+    page.waitForResponse(
+      (candidate) =>
+        candidate.request().method() === 'DELETE' &&
+        /^\/api\/invitations\/[^/]+$/.test(new URL(candidate.url()).pathname)
+    ),
+    confirm.getByRole('button', { name: 'Revoke', exact: true }).click(),
+  ])
+  expect(revoked.ok(), 'Revoke invite was refused').toBe(true)
+  await expect(confirm).toBeHidden()
   await expect(page.getByText(email)).toHaveCount(0)
   await page.reload()
   await expect(page.getByRole('heading', { name: 'Members', level: 1 })).toBeVisible()

apps/sim/scripts/org-redesign-e2e/checks/chat.ts

@@ -92,24 +92,17 @@ export async function chatChecks() {
       }
     }
   )
-  await check(
-    'assistant conversation under Search: switch, New chat, follow-up composer',
-    async () => {
-      await withSession(MEMBER, async ({ page }) => {
-        await page.goto(url(org.searchChat(MEMBER_CHATS.assistant.id)))
-        await expect(modeRadio(page, 'Assistant')).toBeChecked()
-        await expect(composer(page)).toHaveAttribute('placeholder', 'Ask a follow-up…')
-        // The Assistant takes images only: its attach button says so, and it offers no skills.
-        await expect(page.getByRole('button', { name: 'Attach image', exact: true })).toBeVisible()
-        await expect(page.getByRole('button', { name: 'Attach file', exact: true })).toHaveCount(0)
-        await expect(page.getByRole('button', { name: 'Skills', exact: true })).toHaveCount(0)
-        const newChat = page.getByRole('button', { name: 'New chat', exact: true })
-        await expect(newChat).toBeVisible()
-        await newChat.click()
-        await expect(page).toHaveURL(assistantUrl())
-      })
-    }
-  )
+  await check('assistant conversation under Search: switch, follow-up composer', async () => {
+    await withSession(MEMBER, async ({ page }) => {
+      await page.goto(url(org.searchChat(MEMBER_CHATS.assistant.id)))
+      await expect(modeRadio(page, 'Assistant')).toBeChecked()
+      await expect(composer(page)).toHaveAttribute('placeholder', 'Ask a follow-up…')
+      // The Assistant takes images only: its attach button says so, and it offers no skills.
+      await expect(page.getByRole('button', { name: 'Attach image', exact: true })).toBeVisible()
+      await expect(page.getByRole('button', { name: 'Attach file', exact: true })).toHaveCount(0)
+      await expect(page.getByRole('button', { name: 'Skills', exact: true })).toHaveCount(0)
+    })
+  })
   await check(
     'a stream survives switching to the Search view: Assistant stays mounted, inert',
     async () => {