Skip to content

Fix enableDevContainer test output channel mock#7

Open
lambrianmsft wants to merge 3 commits into
mainfrom
lambrian/devcontainer_experience_test_failures
Open

Fix enableDevContainer test output channel mock#7
lambrianmsft wants to merge 3 commits into
mainfrom
lambrian/devcontainer_experience_test_failures

Conversation

@lambrianmsft
Copy link
Copy Markdown
Owner

Summary

  • Fix the enableDevContainerIntegration.test.ts output channel mock to include appendLog, matching the shared IAzExtOutputChannel test shape.
  • Clear ext.designTimeInstances before and after each integration test to avoid stale design-time state leaking across tests.
  • Addresses CI-only Vitest unhandled rejections from the Linux design-time validation path calling ext.outputChannel.appendLog.

Validation

  • npx biome check --write apps\vs-code-designer\src\app\commands\enableDevContainer\__test__\enableDevContainerIntegration.test.ts
  • npx vitest run src\app\commands\enableDevContainer\__test__\enableDevContainerIntegration.test.ts
  • npx vitest run src\app\utils\codeless\__test__\startDesignTimeApi.test.ts
  • npx vitest run src\app\commands\enableDevContainer\__test__\enableDevContainerIntegration.test.ts src\app\utils\codeless\__test__\startDesignTimeApi.test.ts
  • pnpm run test:extension-unit

lambrianmsft and others added 3 commits April 29, 2026 13:00
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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