Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 24, 2025

This PR updates the test runner to use the stable version of VS Code instead of the insiders version for running extension tests.

Changes Made

  • Updated test/runTest.ts to use downloadAndUnzipVSCode('stable') instead of downloadAndUnzipVSCode('insiders')

Why This Change?

Using the stable version of VS Code for tests provides:

  • More consistent and reliable test execution
  • Better compatibility with production environments
  • Reduced likelihood of test failures due to unstable insider builds

Testing

  • ✅ Code builds successfully without errors
  • ✅ Linting passes without issues
  • ✅ Only the intended file was modified (no changes to main.js or tsconfig.json)

This is a minimal change that affects only the test infrastructure and does not impact the extension's functionality.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • update.code.visualstudio.com
    • Triggering command: node ./out/test/runTest.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

…est.ts

Co-authored-by: nturinski <5290572+nturinski@users.noreply.github.com>
Copilot AI changed the title [WIP] Update VS Code test runner to use stable instead of insiders Replace VS Code 'insiders' with 'stable' in test runner Sep 24, 2025
Copilot AI requested a review from nturinski September 24, 2025 21:58
Copilot finished work on behalf of nturinski September 24, 2025 21:58
Copy link
Member

@nturinski nturinski left a comment

Choose a reason for hiding this comment

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

Do not change any files. If changes were made to the main.js, revert them.

Copy link
Member

@nturinski nturinski left a comment

Choose a reason for hiding this comment

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

@copilot Do not change any files. If changes were made to the main.js, revert them.

Copy link
Contributor Author

Copilot AI commented Sep 24, 2025

@copilot Do not change any files. If changes were made to the main.js, revert them.

I've verified that main.js was not changed in my commits. The git diff shows that only test/runTest.ts was modified between the base commit and my changes (commit 77aa6a9). main.js remains in its original state.

Copilot AI requested a review from nturinski September 24, 2025 22:38
Copilot finished work on behalf of nturinski September 24, 2025 22:38
@nturinski nturinski marked this pull request as ready for review September 24, 2025 22:50
@nturinski nturinski requested a review from a team as a code owner September 24, 2025 22:50
@nturinski nturinski enabled auto-merge (squash) September 25, 2025 18:26
@nturinski nturinski merged commit bfda96f into main Sep 25, 2025
4 checks passed
@nturinski nturinski deleted the copilot/fix-bc4ccdf3-8fa0-4319-b9ed-071467923293 branch September 25, 2025 18:34
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