Skip to content

Seed VS Code test user settings in .vscode-test.mjs for local parity#1566

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment-again
Draft

Seed VS Code test user settings in .vscode-test.mjs for local parity#1566
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-code-review-comment-again

Conversation

Copilot AI commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

A review comment flagged that local smoke-test / e2e-test / integration-test runs depended on CI-only setup for python.useEnvironmentsExtension. This updates test config initialization so the required user setting is provisioned directly by .vscode-test.mjs.

  • Problem addressed

    • Local runs could skip extension activation because .vscode-test/user-data/User/settings.json was not created outside CI.
  • Change

    • In .vscode-test.mjs, create User/settings.json under the configured --user-data-dir and write:
      • "python.useEnvironmentsExtension": true
  • Resulting behavior

    • Test entrypoints now initialize the same required setting in local and CI contexts without relying on external workflow steps.
const userSettingsFile = path.join(userDataDir, 'User', 'settings.json');

fs.mkdirSync(path.dirname(userSettingsFile), { recursive: true });
fs.writeFileSync(
  userSettingsFile,
  JSON.stringify({ 'python.useEnvironmentsExtension': true }),
);

Copilot AI changed the title [WIP] Fix code based on review comment Seed VS Code test user settings in .vscode-test.mjs for local parity Jun 8, 2026
Copilot AI requested a review from eleanorjboyd June 8, 2026 03:17
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