Skip to content

Add Network Inspector server unit tests#2704

Open
ConnorQi01 wants to merge 1 commit into
masterfrom
dev/v-peq/add-network-inspector-server-tests
Open

Add Network Inspector server unit tests#2704
ConnorQi01 wants to merge 1 commit into
masterfrom
dev/v-peq/add-network-inspector-server-tests

Conversation

@ConnorQi01

Copy link
Copy Markdown
Collaborator

Summary

  • Add direct Network Inspector server unit tests.
  • Cover certificate exchange medium conversion, secure config load failures, server start/stop lifecycle, and async connection registration/removal.

Validation

  • node .\node_modules\mocha\bin\_mocha test\extension\networkInspector\networkInspectorServer.test.js --ui tdd --timeout 10000
  • npm run build
  • npm test downloaded the VS Code test archive, then the local Windows test host failed to start with Error mutex already exists and EPIPE.

Closes #2694

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 NetworkInspectorServer internal logic

1 participant