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)]

Warning · Non-blocking recommendation

Could an unscoped pytest parameter ID contain @@vsc@@ (for example, tests/test_foo.py::test_value[value@@vsc@@suffix])? If so, parseVsId would treat it as project-scoped and copy only suffix]. Please ensure the scope prefix is validated before stripping it, and cover this parameterized-ID case.

[Bill Schnurr (bschnurr)]

Approved via Review Center.