Skip to content

Add status indicator tests#2564

Open
ConnorQi01 wants to merge 1 commit intomasterfrom
dev/v-peq/statusBar_indicatorTests
Open

Add status indicator tests#2564
ConnorQi01 wants to merge 1 commit intomasterfrom
dev/v-peq/statusBar_indicatorTests

Conversation

@ConnorQi01
Copy link
Collaborator

Summary

This PR adds test coverage for status bar indicator behaviors in extension unit tests.

Background

Status bar behavior for active device name and packager port can regress without targeted tests. This PR focuses on locking expected behavior with deterministic unit coverage.

Test-Only Changes

  1. Added test/extension/deviceStatusIndicator.test.ts
  • validates text format and tooltip content on show()
  • validates show() and hide() invocation behavior
  • validates singleton lifecycle (reuse and dispose() cleanup)
  1. Added test/extension/packagerStatusIndicator.test.ts
  • validates port rendering with PACKAGER_STARTED in full/short display modes
  • validates no-port behavior when port is omitted
  • validates port reset after PACKAGER_STOPPED
  • validates port retention across status transitions
  • validates port replacement when updated

Non-Goals

  • no production feature logic changes in this PR
  • no smoke/e2e scope expansion in this PR

Validation

  • test assertions rely on stubs for VS Code status bar APIs
  • coverage targets status text, tooltip, visibility calls, and port transition correctness

@ConnorQi01 ConnorQi01 linked an issue Mar 20, 2026 that may be closed by this pull request
@ConnorQi01
Copy link
Collaborator Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

[Feature] Add Unit Coverage for Status Bar Indicators

1 participant