Strip project scope prefix from IDs copied by "Copy test ID"

[Copilot]

Fixes #26092

"Copy test ID" copied the raw TestItem.id, which since project scoping is {projectId}@@vsc@@{testId}, so users pasted file:///path/to/workspace@@vsc@@/path/to/workspace/tests/unit/test_foo.py::test_bar instead of the test ID.

Changes

• src/client/testing/utils.ts: writeTestIdToClipboard now normalizes the ID with the existing parseVsId helper before unittest detection/conversion, the pytest clipboard write, and logging.
  • TestItem.id is not mutated and discovery/execution scoping is untouched — only the clipboard-facing value is normalized.
  • parseVsId returns [undefined, vsId] for unscoped IDs, so legacy behavior and the module.class.method unittest formatting are preserved.

-        if (testItem.id.includes('\\') && testItem.id.indexOf('::') === -1) {
+        const [, testId] = parseVsId(testItem.id);
+        if (testId.includes('\\') && testId.indexOf('::') === -1) {

• src/test/testing/utils.unit.test.ts: this file was a top-level IIFE with try/catch-swallowed assertions that mocha never registered as tests (and its idToModuleClassMethod inputs had lost their backslash separators). Converted to a suite/test file asserting the real clipboard contents via the MockClipboard from src/test/vscode-mock.ts, with cases for project-scoped pytest IDs, parameterized IDs (test_pipe_single[False] preserved verbatim), Windows-style paths, legacy unscoped IDs, and project-scoped unittest IDs.

[Bill Schnurr (bschnurr)]

🔒 Automated review in progress — Bill Schnurr (@bschnurr) is auto-reviewing this PR.

[Bill Schnurr (bschnurr)]

Approved via Review Center.

[Bill Schnurr (bschnurr)]

Result: ⚠️ partially-verified

Verification details

Verification: Isolated verification observed failures that were not classified as caused by this PR: Dependency and test discovery. The relevant tests could not be fully run in the isolated environment; this review is not fully verified.

Summary: The targeted utils unit suite compiled and passed all 13 tests, including scoped pytest, parameterized, Windows, legacy, and unittest IDs. TypeScript compilation and ESLint also passed. An initial combined run and git-based discovery were blocked by sandbox infrastructure, so the verdict is partially verified despite the successful targeted run.

Test runs: 3 passed, 1 failed, 1 not run

• ⚠️ Not run | Combined compile and utils unit tests | node ./node_modules/typescript/bin/tsc -p ./ && node ./node_modules/mocha/bin/mocha.js --require out/test/unittests.js --ui tdd --timeout 180000 --reporter spec out/test/testing/utils.unit.test.js
• ❌ Failed | unrelated to this PR | Dependency and test discovery | printf '%s\n' '=== sandbox profile ==='; printf '%s\n' "${AUTOMATION_SANDBOX_PROFILE:-<unset>}"; printf '%s\n' '=== changed files ==='; git diff --name-status HEAD^...HEAD; printf '%s\n' '=== relevant diff ==='; git diff --unified=20 HEAD^...HEAD -- src/client/testing/utils.ts src/test/testing/utils.unit.test.ts; printf '%s\n' '=== package scripts ==='; node -e "const p=require('./package.json'); console.log(JSON.stringify({scripts:p.scripts,packageManager:p.packageManager},null,2))"; printf '%s\n' '=== dependency state ==='; test -d node_modules && echo node_modules-present || echo node_modules-missing; test -x node_modules/.bin/mocha && echo mocha-present || echo mocha-missing; test -f package-lock.json && echo package-lock-present; test -f pnpm-lock.yaml && echo pnpm-lock-present
• ✅ Passed | Testing - utils unit tests | node ./node_modules/mocha/bin/mocha.js --require out/test/unittests.js --ui tdd --timeout 180000 --reporter spec out/test/testing/utils.unit.test.js
• ✅ Passed | Changed-file ESLint check | node ./node_modules/eslint/bin/eslint.js src/client/testing/utils.ts src/test/testing/utils.unit.test.ts
• ✅ Passed | TypeScript compilation | node ./node_modules/typescript/bin/tsc -p ./

⚠️ Combined compile and utils unit tests diagnostic output

PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\Users\bschnurr\AppData\Local\Temp\automation-job-ready-c9wl6bgb'

❌ Dependency and test discovery diagnostic output

AUTOMATION_SANDBOX_PROFILE=typescript
git: Not a git repository
node_modules-missing
mocha-missing
package-lock-present