Skip to content

test(extension): run extension tests headless - #44063

Merged
Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:extension-tests-chrome-stub
Nov 11, 2026
Merged

Pavel Feldman (pavelfeldman) merged 1 commit into
microsoft:mainfrom
pavelfeldman:extension-tests-chrome-stub

Conversation

@pavelfeldman

Copy link
Copy Markdown
Member

Summary

  • Extension tests run headless, in parallel, on ubuntu-latest.
  • The relay starts tests/extension/chrome-stub.js in place of the Chrome executable. The stub starts the browser when the profile has none, and opens the connect page in a new tab of the running browser.
  • New test for the path where the relay starts the browser.

Replaces #44048.

Headless Chrome does not hand the connect page over to a running browser
of the same profile. The relay now starts a stub of the Chrome executable
in the tests. The stub starts the browser when the profile has none, and
opens the connect page in a new tab.
@github-actions

This comment has been minimized.

strategy:
fail-fast: false
matrix:
os: [macos-latest]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

let's keep true e2e coverage on mac

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'll double the harness complexity with no clear benefit. With the CLI args assertion it is good enough.

@pavelfeldman
Pavel Feldman (pavelfeldman) merged commit 392ab0f into microsoft:main Nov 11, 2026
25 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

5229 passed, 637 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

23628 passed, 713 skipped


Merge workflow run.

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