Skip to content

test(ts-plugin): add e2e test for cmkOptions.enabled: false#360

Merged
mizdra merged 1 commit into
mainfrom
test/disabled-plugin
Apr 18, 2026
Merged

test(ts-plugin): add e2e test for cmkOptions.enabled: false#360
mizdra merged 1 commit into
mainfrom
test/disabled-plugin

Conversation

@mizdra
Copy link
Copy Markdown
Owner

@mizdra mizdra commented Apr 18, 2026

Summary

  • Add e2e-test/disabled.test.ts to verify that ts-plugin provides no language features when cmkOptions.enabled is false
  • Uses sendDefinitionAndBoundSpan (Go to Definition) and asserts an empty result, confirming the plugin's early-return short-circuit in src/index.cts is working

Test plan

  • vp test packages/ts-plugin/e2e-test/disabled.test.ts passes

🤖 Generated with Claude Code

Verify that ts-plugin provides no language features (Go to Definition
returns empty) when cmkOptions.enabled is false.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 18, 2026

⚠️ No Changeset found

Latest commit: 5dcd3f7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mizdra mizdra added the Type: Testing Adding missing tests or correcting existing tests label Apr 18, 2026
@mizdra mizdra merged commit 944361e into main Apr 18, 2026
20 checks passed
@mizdra mizdra deleted the test/disabled-plugin branch April 18, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Testing Adding missing tests or correcting existing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant