Skip to content

Handle Welcome Dialog in DevTunnel Test#309752

Merged
dmitrivMS merged 1 commit intomainfrom
dev/dmitriv/fix-sanity-tests
Apr 14, 2026
Merged

Handle Welcome Dialog in DevTunnel Test#309752
dmitrivMS merged 1 commit intomainfrom
dev/dmitriv/fix-sanity-tests

Conversation

@dmitrivMS
Copy link
Copy Markdown
Contributor

No description provided.

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 14, 2026 06:51
@dmitrivMS dmitrivMS added the engineering VS Code - Build / issue tracking / etc. label Apr 14, 2026
@dmitrivMS dmitrivMS self-assigned this Apr 14, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 182bcf49 Current: d2739cf8

Changed (4)

editor/inlineCompletions/other/JumpToHint/Dark
Before After
before after
agentSessionsViewer/WithMarkdownBadge/Light
Before After
before after
agentSessionsViewer/ApprovalRowLongLabel/Light
Before After
before after
agentSessionsViewer/WithMarkdownBadge/Dark
Before After
before after

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

Updates the Dev Tunnel sanity test flow to reliably handle the initial “Welcome” UI so the test can proceed through GitHub authentication and subsequent workbench actions.

Changes:

  • Dismiss the welcome dialog immediately after the workbench loads during tunnel connection.
  • Skip the welcome-dismiss step inside UITest.run since it has already been handled earlier in the Dev Tunnel flow.
Show a summary per file
File Description
test/sanity/src/devTunnel.test.ts Dismisses the welcome dialog before GitHub auth and avoids re-dismissing it during the subsequent UI test run.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@dmitrivMS dmitrivMS merged commit 1995777 into main Apr 14, 2026
59 of 62 checks passed
@dmitrivMS dmitrivMS deleted the dev/dmitriv/fix-sanity-tests branch April 14, 2026 07:27
@vs-code-engineering vs-code-engineering bot added this to the 1.117.0 milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engineering VS Code - Build / issue tracking / etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants