Skip to content

chore(playwright): Setup LLM Provider#8362

Merged
justin-tahara merged 4 commits intomainfrom
jtahara/playwright-for-llm
Feb 12, 2026
Merged

chore(playwright): Setup LLM Provider#8362
justin-tahara merged 4 commits intomainfrom
jtahara/playwright-for-llm

Conversation

@justin-tahara
Copy link
Copy Markdown
Contributor

@justin-tahara justin-tahara commented Feb 11, 2026

Description

Adding a simple Playwright test for the setup of LLMs.

How Has This Been Tested?

Ran playwright locally

Additional Options

  • [Required] I have considered whether this PR needs to be cherry-picked to the latest beta branch.
  • [Optional] Override Linear Check

Summary by cubic

Add Playwright e2e for the admin LLM setup page: create an OpenAI provider via the UI, toggle auto mode, switch default, and delete. Use env-backed API keys for test providers (LLM/web search/image), stub /api/admin/llm/test, seed/cleanup providers, and stabilize file uploads by awaiting the upload API response.

Written for commit ef9d532. Summary will update on new commits.

@justin-tahara justin-tahara requested a review from a team as a code owner February 11, 2026 23:57
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Feb 11, 2026

Greptile Overview

Greptile Summary

Added comprehensive Playwright test coverage for LLM provider setup functionality. The test suite covers the complete lifecycle: creating providers via UI, editing configuration (including auto-mode toggle), deleting providers, and switching default providers. The tests properly mock external API calls to keep tests deterministic and include proper cleanup in afterEach hooks.

Confidence Score: 5/5

  • Safe to merge - well-structured test file with no production code changes
  • This is a test-only PR that adds comprehensive E2E coverage for LLM provider setup. The code follows Playwright best practices: proper setup/teardown, deterministic mocking, unique test data generation, and polling for async operations. No production code is modified.
  • No files require special attention

Important Files Changed

Filename Overview
web/tests/e2e/admin/llm_provider_setup.spec.ts New Playwright test for LLM provider setup workflow (create, edit, delete, and default switching). Well-structured with proper cleanup and mocking.

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 11, 2026

Preview Deployment

Status Preview Commit Updated
https://onyx-preview-k6equrj6w-danswer.vercel.app ef9d532 2026-02-12 01:53:53 UTC

@justin-tahara justin-tahara force-pushed the jtahara/playwright-for-llm branch from cae9f84 to 9d13a3f Compare February 12, 2026 00:12
@justin-tahara justin-tahara added this pull request to the merge queue Feb 12, 2026
Merged via the queue into main with commit 9bf8400 Feb 12, 2026
83 checks passed
@justin-tahara justin-tahara deleted the jtahara/playwright-for-llm branch February 12, 2026 02:09
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