Only strip the project prefix when it is a project URI

Co-authored-by: edvilme <5952839+edvilme@users.noreply.github.com>

Author: Copilot

src/client/testing/utils.ts

@@ -5,7 +5,7 @@ import { parseVsId } from './testController/common/projectUtils';
 export async function writeTestIdToClipboard(testItem: TestItem): Promise<void> {
     if (testItem && typeof testItem.id === 'string') {
         // Strip the project scope prefix (if any) so only the test id is copied.
-        const [, testId] = parseVsId(testItem.id);
+        const testId = stripProjectId(testItem.id);
         if (testId.includes('\\') && testId.indexOf('::') === -1) {
             // Convert the id to a module.class.method format as this is a unittest
             const moduleClassMethod = idToModuleClassMethod(testId);
@@ -21,6 +21,19 @@ export async function writeTestIdToClipboard(testItem: TestItem): Promise<void>
     }
 }
 
+// Project ids are always URIs (see getProjectId), so only strip a prefix that looks
+// like one. Otherwise a pytest parameter containing the separator text, such as
+// "test_foo.py::test_value[value@@vsc@@suffix]", would be truncated.
+const PROJECT_ID_PATTERN = /^[a-zA-Z][a-zA-Z\d+.-]*:\/\//;
+
+function stripProjectId(vsId: string): string {
+    const [projectId, testId] = parseVsId(vsId);
+    if (projectId === undefined || !PROJECT_ID_PATTERN.test(projectId)) {
+        return vsId;
+    }
+    return testId;
+}
+
 export function idToModuleClassMethod(id: string): string | undefined {
     // Split by backslash
     const parts = id.split('\\');

src/test/testing/utils.unit.test.ts

@@ -56,6 +56,18 @@ suite('Testing - utils', () => {
             expect(await copiedText(id)).to.equal('tests/unit/test_foo.py::test_pipe_single[False]');
         });
 
+        test('legacy parameterized pytest id containing the separator text is copied as is', async () => {
+            const id = `tests/test_foo.py::test_value[value${PROJECT_ID_SEPARATOR}suffix]`;
+
+            expect(await copiedText(id)).to.equal(id);
+        });
+
+        test('project scoped parameterized pytest id containing the separator text keeps the parameters', async () => {
+            const id = `file:///path/to/workspace${PROJECT_ID_SEPARATOR}tests/test_foo.py::test_value[value${PROJECT_ID_SEPARATOR}suffix]`;
+
+            expect(await copiedText(id)).to.equal(`tests/test_foo.py::test_value[value${PROJECT_ID_SEPARATOR}suffix]`);
+        });
+
         test('project scoped windows pytest id drops the project prefix', async () => {
             const id = `file:///c%3A/workspace${PROJECT_ID_SEPARATOR}c:\\workspace\\tests\\test_foo.py::test_bar`;