Skip to content

Tests: recover missing test elements#1296

Merged
haydenbleasel merged 1 commit into
vercel:mainfrom
plpxsk:main
Nov 1, 2025
Merged

Tests: recover missing test elements#1296
haydenbleasel merged 1 commit into
vercel:mainfrom
plpxsk:main

Conversation

@plpxsk
Copy link
Copy Markdown
Contributor

@plpxsk plpxsk commented Oct 31, 2025

Fix Playwright chat tests by recovering lost test elements

3 missing elements added, resulting in more tests that now pass

Run tests with pnpm test or pnpm exec playwright test tests/e2e/chat.test.ts --project=e2e

Example

One missing test element was data-testid for send-button in file components/multimodal-input.tsx

It is required for basic chat tests to run, in tests/pages/chat.ts:

 get sendButton() {
    return this.page.getByTestId("send-button");
  }

The similar stop-button is present and works, around line 467 in components/multimodal-input.tsx

The send-button was present in 9dd9a98 but inadvertently removed in #1155

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Oct 31, 2025

@plpxsk is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@haydenbleasel
Copy link
Copy Markdown
Contributor

Nice one, thanks @plpxsk 🙏

@haydenbleasel haydenbleasel merged commit 4e0e222 into vercel:main Nov 1, 2025
1 of 5 checks passed
shuibuzhuo pushed a commit to shuibuzhuo/ai-chatbot that referenced this pull request Dec 10, 2025
systimahq pushed a commit to systima-ai/chatbot-comply-test that referenced this pull request May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants