Skip to content

test: add command validation and vs code integration tests#14

Merged
Arthurvroum merged 4 commits into
mainfrom
13-add-test-to-ci
Apr 8, 2026
Merged

test: add command validation and vs code integration tests#14
Arthurvroum merged 4 commits into
mainfrom
13-add-test-to-ci

Conversation

@Arthurvroum
Copy link
Copy Markdown
Collaborator

No description provided.

@Arthurvroum Arthurvroum linked an issue Apr 8, 2026 that may be closed by this pull request
@Arthurvroum Arthurvroum requested a review from Copilot April 8, 2026 04:41
Copy link
Copy Markdown

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 replaces the placeholder extension test with basic VS Code extension activation/command-registration checks and adds unit tests around CLI parameter validation, helping ensure the extension wires up correctly and rejects unsafe command-line inputs.

Changes:

  • Add VS Code integration tests to verify the extension is present, activates, and registers expected commands.
  • Add unit tests for parseAndValidateParams to validate safe flags and reject shell-injection payloads.
  • Ignore .vscode-test/ artifacts in .gitignore.

Reviewed changes

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

File Description
src/test/extension.test.ts Replaces the sample test with extension presence/activation and command registration checks.
src/test/command.test.ts Adds unit coverage for CLI param parsing/validation against safe and malicious inputs.
.gitignore Adds .vscode-test/ to keep VS Code test artifacts out of the repo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/test/extension.test.ts Outdated
Comment thread src/test/extension.test.ts Outdated
Comment thread src/test/extension.test.ts Outdated
@Arthurvroum Arthurvroum merged commit 6d25656 into main Apr 8, 2026
6 checks passed
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.

add: test to ci

2 participants