Skip to content

test: add BYOK Model Management test plan#149

Merged
ethanyhou merged 5 commits into
mainfrom
ethan/byok-plan
May 9, 2026
Merged

test: add BYOK Model Management test plan#149
ethanyhou merged 5 commits into
mainfrom
ethan/byok-plan

Conversation

@ethanyhou
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 8, 2026 04:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new SWTBot test plan document for the BYOK (Bring Your Own Key) Model Management preference page, describing manual/probe-verifiable scenarios for provider listing, API key lifecycle, custom model CRUD, reload behavior, and conditional signed-in/feature-flag states.

Changes:

  • Introduces a comprehensive BYOK Model Management test plan with 12 test cases.
  • Documents expected UI states (signed-out, org-disabled, enabled) and selection-driven button enablement.
  • Captures cross-cutting failure modes and troubleshooting signals (workspace.log, loading overlays, provider-level loading).

Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md Outdated
Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md Outdated
Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md Outdated
Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md
Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md Outdated
Comment thread com.microsoft.copilot.eclipse.swtbot.test/test-plans/byok/byok.md
ethanyhou and others added 4 commits May 9, 2026 10:13
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
jdneo
jdneo previously approved these changes May 9, 2026
Copy link
Copy Markdown
Member

@xinyi-gong xinyi-gong left a comment

Choose a reason for hiding this comment

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

LGTM

@ethanyhou ethanyhou merged commit 391f8c9 into main May 9, 2026
4 checks passed
@ethanyhou ethanyhou deleted the ethan/byok-plan branch May 9, 2026 02:59
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.

4 participants