Skip to content

Add ScriptImporter URL unit tests#2744

Merged
ConnorQi01 merged 3 commits into
masterfrom
dev/v-peq/scriptImporter_urlTests
Jun 26, 2026
Merged

Add ScriptImporter URL unit tests#2744
ConnorQi01 merged 3 commits into
masterfrom
dev/v-peq/scriptImporter_urlTests

Conversation

@ConnorQi01

Copy link
Copy Markdown
Collaborator

Summary

  • Add focused unit tests for ScriptImporter debugger worker URL construction across legacy, debugger UI, canary, and explicit path cases.
  • Include the new ScriptImporter test in the VS Code test runner without enabling the full debugger test directory.
  • Cover packager port replacement while preserving URL query strings.

Test Plan

  • npx gulp format
  • npx tsc --noEmit
  • npm test (ScriptImporter tests passed; existing unrelated settings/android tests failed with timeout/channel-closed errors)

Closes #2716

@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 144ee9c into master Jun 26, 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 ScriptImporter URL construction and debugger worker URL logic

2 participants