Skip to content

Add Network Inspector manager lifecycle tests#2637

Merged
ConnorQi01 merged 2 commits into
masterfrom
connor/add-network-inspector-manager-tests
May 19, 2026
Merged

Add Network Inspector manager lifecycle tests#2637
ConnorQi01 merged 2 commits into
masterfrom
connor/add-network-inspector-manager-tests

Conversation

@ConnorQi01
Copy link
Copy Markdown
Collaborator

Summary

  • Add unit tests for NetworkInspectorManager start/stop lifecycle behavior
  • Cover running state, duplicate start rejection, stop no-op, and cleanup on start failure

Validation

  • npx tsc --noEmit
  • node .\node_modules\mocha\bin\_mocha test\extension\networkInspector\networkInspectorManager.test.js --ui tdd --timeout 10000

Fixes #2633

@ConnorQi01
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ConnorQi01
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@ConnorQi01 ConnorQi01 merged commit 41068de into master May 19, 2026
8 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.

[Feature] Add unit tests for NetworkInspectorManager lifecycle

2 participants