updates

Author: eleanorjboyd

src/client/testing/testController/common/utils.ts

@@ -267,11 +267,11 @@ export function populateTestTree(
                 testItemMappings.runIdToVSid.set(child.runID, vsId);
                 testItemMappings.vsIdToRunId.set(vsId, child.runID);
             } else {
-                let node = testController.items.get(child.path);
+                // Use project-scoped ID for non-test nodes and look up within the current root
+                const nodeId = projectId ? `${projectId}${PROJECT_ID_SEPARATOR}${child.id_}` : child.id_;
+                let node = testRoot!.children.get(nodeId);
 
                 if (!node) {
-                    // Create project-scoped ID for non-test nodes
-                    const nodeId = projectId ? `${projectId}${PROJECT_ID_SEPARATOR}${child.id_}` : child.id_;
                     node = testController.createTestItem(nodeId, child.name, Uri.file(child.path));
 
                     node.canResolveChildren = true;

src/client/testing/testController/controller.ts

@@ -562,7 +562,7 @@ export class PythonTestController implements ITestController, IExtensionSingleAc
             await Promise.all(projects.map((project) => this.discoverTestsForProject(project, projectsCompleted)));
 
             traceInfo(
-                `[test-by-project] Discovery complete: ${projectsCompleted.size}/${projects.length} projects succeeded`,
+                `[test-by-project] Discovery complete: ${projectsCompleted.size}/${projects.length} projects completed`,
             );
         } catch (error) {
             traceError(`[test-by-project] Discovery failed for workspace ${workspaceUri.fsPath}:`, error);