Skip to content

chore: allow debugging vscode tests#4731

Merged
benfdking merged 1 commit intomainfrom
debug_vscode_e2e
Jun 13, 2025
Merged

chore: allow debugging vscode tests#4731
benfdking merged 1 commit intomainfrom
debug_vscode_e2e

Conversation

@benfdking
Copy link
Copy Markdown
Contributor

@benfdking benfdking commented Jun 13, 2025

sets up the playwright extension correctly in vscode so that playwright tests called from the vscode extension work correctly

To set up

make vscode_settings

@benfdking benfdking requested a review from Copilot June 13, 2025 10:30
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

This PR sets up VS Code configuration and test utilities to enable debugging of Playwright tests within the VS Code extension.

  • Use path.join with __dirname in utils.ts to locate the VS Code executable path file.
  • Extend VS Code settings.json with Python interpreter and terminal environment settings for Playwright.

Reviewed Changes

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

File Description
vscode/extension/tests/utils.ts Switch from relative to path.join-based lookup of paths.json
tooling/vscode/settings.json Add Python venv interpreter path and terminal env overrides

Comment thread vscode/extension/tests/utils.ts
Comment thread tooling/vscode/settings.json
@benfdking benfdking merged commit 6afa17c into main Jun 13, 2025
25 checks passed
@benfdking benfdking deleted the debug_vscode_e2e branch June 13, 2025 10:53
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.

3 participants