project labeling

Author: eleanorjboyd

.github/instructions/testing_feature_area.instructions.md

@@ -201,6 +201,10 @@ This approach was chosen because:
 2. Implementing custom exclusion would add significant complexity with minimal benefit
 3. The existing approach is transparent and predictable - each project shows what it finds
 
+### Empty projects and hidden root nodes
+
+**Important:** If a project discovers zero tests, its root node will **not appear** in the Test Explorer. This is by design - the test tree only shows projects that have actual tests.
+
 ### Logging prefix
 
 All project-based testing logs use the `[test-by-project]` prefix for easy filtering in the output channel.

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

@@ -33,15 +33,23 @@ export class PythonResultResolver implements ITestResultResolver {
      */
     private projectId?: string;
 
+    /**
+     * Optional project display name for labeling the test tree root.
+     * When set, the root node label will be "project: {projectName}" instead of the folder name.
+     */
+    private projectName?: string;
+
     constructor(
         testController: TestController,
         testProvider: TestProvider,
         private workspaceUri: Uri,
         projectId?: string,
+        projectName?: string,
     ) {
         this.testController = testController;
         this.testProvider = testProvider;
         this.projectId = projectId;
+        this.projectName = projectName;
         // Initialize a new TestItemIndex which will be used to track test items in this workspace/project
         this.testItemIndex = new TestItemIndex();
     }
@@ -76,6 +84,7 @@ export class PythonResultResolver implements ITestResultResolver {
             this.testProvider,
             token,
             this.projectId,
+            this.projectName,
         );
         sendTelemetryEvent(EventName.UNITTEST_DISCOVERY_DONE, undefined, {
             tool: this.testProvider,

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

@@ -29,6 +29,7 @@ export class TestDiscoveryHandler {
         testProvider: TestProvider,
         token?: CancellationToken,
         projectId?: string,
+        projectName?: string,
     ): void {
         if (!payload) {
             // No test data is available
@@ -70,6 +71,7 @@ export class TestDiscoveryHandler {
                 },
                 token,
                 projectId,
+                projectName,
             );
         }
     }

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

@@ -192,18 +192,23 @@ export class TestProjectRegistry {
 
         // Create test infrastructure
         const testProvider = this.getTestProvider(workspaceUri);
-        const resultResolver = new PythonResultResolver(this.testController, testProvider, workspaceUri, projectId);
+        const projectDisplayName = createProjectDisplayName(pythonProject.name, pythonEnvironment.version);
+        const resultResolver = new PythonResultResolver(
+            this.testController,
+            testProvider,
+            workspaceUri,
+            projectId,
+            pythonProject.name, // Use simple project name for test tree label (without version)
+        );
         const { discoveryAdapter, executionAdapter } = createTestAdapters(
             testProvider,
             resultResolver,
             this.configSettings,
             this.envVarsService,
         );
 
-        const projectName = createProjectDisplayName(pythonProject.name, pythonEnvironment.version);
-
         return {
-            projectName,
+            projectName: projectDisplayName,
             projectUri: pythonProject.uri,
             workspaceUri,
             pythonProject,

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

@@ -213,12 +213,15 @@ export function populateTestTree(
     testItemMappings: ITestItemMappings,
     token?: CancellationToken,
     projectId?: string,
+    projectName?: string,
 ): void {
     // If testRoot is undefined, use the info of the root item of testTreeData to create a test item, and append it to the test controller.
     if (!testRoot) {
         // Create project-scoped ID if projectId is provided
         const rootId = projectId ? `${projectId}${PROJECT_ID_SEPARATOR}${testTreeData.path}` : testTreeData.path;
-        testRoot = testController.createTestItem(rootId, testTreeData.name, Uri.file(testTreeData.path));
+        // Use "Project: {name}" label for project-based testing, otherwise use folder name
+        const rootLabel = projectName ? `Project: ${projectName}` : testTreeData.name;
+        testRoot = testController.createTestItem(rootId, rootLabel, Uri.file(testTreeData.path));
 
         testRoot.canResolveChildren = true;
         testRoot.tags = [RunTestTag, DebugTestTag];
@@ -282,7 +285,7 @@ export function populateTestTree(
 
                     testRoot!.children.add(node);
                 }
-                populateTestTree(testController, child, node, testItemMappings, token, projectId);
+                populateTestTree(testController, child, node, testItemMappings, token, projectId, projectName);
             }
         }
     });