Skip to content

chore(vscode): enabling more tests#5073

Merged
benfdking merged 1 commit into
mainfrom
reenabling_tests
Jul 30, 2025
Merged

chore(vscode): enabling more tests#5073
benfdking merged 1 commit into
mainfrom
reenabling_tests

Conversation

@benfdking

Copy link
Copy Markdown
Contributor
  • working on enabling configuration tests
  • wanting them working before we play with multi project

@benfdking
benfdking requested a review from Copilot July 30, 2025 20:48

This comment was marked as outdated.

@benfdking
benfdking force-pushed the reenabling_tests branch 2 times, most recently from 7f2b2e3 to 4a47773 Compare July 30, 2025 20:57
@benfdking
benfdking requested a review from Copilot July 30, 2025 20:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 enables previously skipped configuration tests for the VSCode extension lineage functionality. The changes refactor test setup to use shared fixtures and remove redundant code server initialization.

  • Removes skipped status from four lineage panel tests with different project path configurations
  • Refactors tests to use shared tempDir and sharedCodeServer fixtures instead of creating individual temporary directories and code server contexts
  • Simplifies test setup by removing redundant startCodeServer and stopCodeServer calls

Comment thread vscode/extension/tests/lineage.spec.ts Outdated
Comment thread vscode/extension/tests/lineage.spec.ts
@benfdking
benfdking merged commit 5ea134f into main Jul 30, 2025
27 checks passed
@benfdking
benfdking deleted the reenabling_tests branch July 30, 2025 21:14
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