Skip to content

Heavily simplify the test setup#164

Merged
jviotti merged 2 commits intomainfrom
simplify-tests
Feb 13, 2026
Merged

Heavily simplify the test setup#164
jviotti merged 2 commits intomainfrom
simplify-tests

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Feb 13, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 6 files

@augmentcode
Copy link

augmentcode bot commented Feb 13, 2026

🤖 Augment PR Summary

Summary: This PR simplifies the VS Code extension test setup by removing the TypeScript build step and running tests directly from source JS via @vscode/test-cli.

Changes:

  • Updated test/vscode/.vscode-test.js to use a simpler test file glob and keep VS Code test cache/user-data dirs under build/test/vscode
  • Converted the extension test entry from extension.test.ts to extension.test.js (CommonJS require)
  • Adjusted fixture path handling to be relative to the test file’s directory
  • Removed the legacy test/vscode/index.ts runner shim (now handled by @vscode/test-cli)
  • Dropped the local tsconfig.json and TypeScript/typing-related dev dependencies from the VS Code test package

Technical Notes: Test discovery is now driven by .vscode-test.js globs, and the test package no longer compiles sources into build/test/vscode before execution.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

@jviotti jviotti force-pushed the simplify-tests branch 3 times, most recently from e3d2239 to cc0db14 Compare February 13, 2026 15:46
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit da85e67 into main Feb 13, 2026
4 checks passed
@jviotti jviotti deleted the simplify-tests branch February 13, 2026 16:08
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.

1 participant