remove unneeded

Author: eleanorjboyd

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

@@ -2,16 +2,8 @@
 // Licensed under the MIT License.
 
 import * as crypto from 'crypto';
-import { Uri } from 'vscode';
-import { ProjectAdapter } from './projectAdapter';
 import { PythonProject } from '../../../envExt/types';
 
-/**
- * Separator used between project ID and test path in project-scoped test IDs.
- * Using | instead of :: to avoid conflicts with pytest's :: syntax for test paths.
- */
-export const PROJECT_ID_SEPARATOR = '|';
-
 /**
  * Generates a unique project ID by hashing the PythonProject object.
  * This ensures consistent IDs across extension reloads for the same project.
@@ -31,66 +23,6 @@ export function generateProjectId(pythonProject: PythonProject): string {
     return `project-${hash.substring(0, 12)}`;
 }
 
-/**
- * Creates a project-scoped VS Code test item ID.
- * Format: "{projectId}|{testPath}"
- *
- * @param projectId The unique project identifier
- * @param testPath The test path (e.g., "/workspace/test.py::test_func")
- * @returns The project-scoped VS Code test ID
- */
-export function createProjectScopedVsId(projectId: string, testPath: string): string {
-    return `${projectId}${PROJECT_ID_SEPARATOR}${testPath}`;
-}
-
-/**
- * Parses a project-scoped VS Code test ID to extract the project ID and test path.
- *
- * @param vsId The VS Code test item ID
- * @returns Object containing projectId and testPath, or null if invalid
- */
-export function parseProjectScopedVsId(vsId: string): { projectId: string; testPath: string } | null {
-    const separatorIndex = vsId.indexOf(PROJECT_ID_SEPARATOR);
-    if (separatorIndex === -1) {
-        return null;
-    }
-
-    return {
-        projectId: vsId.substring(0, separatorIndex),
-        testPath: vsId.substring(separatorIndex + PROJECT_ID_SEPARATOR.length),
-    };
-}
-
-/**
- * Checks if a test file path is within a nested project's directory.
- * This is used to determine when to query the API for ownership even if
- * only one project discovered the file.
- *
- * @param testFilePath Absolute path to the test file
- * @param allProjects All projects in the workspace
- * @param excludeProject Optional project to exclude from the check (typically the discoverer)
- * @returns True if the file is within any nested project's directory
- */
-export function hasNestedProjectForPath(
-    testFilePath: string,
-    allProjects: ProjectAdapter[],
-    excludeProject?: ProjectAdapter,
-): boolean {
-    return allProjects.some((p) => p !== excludeProject && testFilePath.startsWith(p.projectUri.fsPath));
-}
-
-/**
- * Finds the project that owns a specific test file based on project URI.
- * This is typically used after the API returns ownership information.
- *
- * @param projectUri The URI of the owning project (from API)
- * @param allProjects All projects to search
- * @returns The ProjectAdapter with matching URI, or undefined if not found
- */
-export function findProjectByUri(projectUri: Uri, allProjects: ProjectAdapter[]): ProjectAdapter | undefined {
-    return allProjects.find((p) => p.projectUri.fsPath === projectUri.fsPath);
-}
-
 /**
  * Creates a display name for a project including Python version.
  * Format: "{projectName} (Python {version})"

src/client/testing/testController/controller.ts

@@ -85,9 +85,6 @@ export class PythonTestController implements ITestController, IExtensionSingleAc
     // @ts-expect-error - used when useProjectBasedTesting=true
     private readonly workspaceDiscoveryState: Map<Uri, WorkspaceDiscoveryState> = new Map();
 
-    // Flag to enable/disable project-based testing
-    private useProjectBasedTesting = false;
-
     private readonly triggerTypes: TriggerType[] = [];
 
     private readonly testController: TestController;
@@ -236,8 +233,8 @@ export class PythonTestController implements ITestController, IExtensionSingleAc
     public async activate(): Promise<void> {
         const workspaces: readonly WorkspaceFolder[] = this.workspaceService.workspaceFolders || [];
 
-        // Try to use project-based testing if enabled
-        if (this.useProjectBasedTesting) {
+        // Try to use project-based testing if environment extension is enabled
+        if (useEnvExtension()) {
             traceInfo('[test-by-project] Activating project-based testing mode');
             try {
                 await Promise.all(
@@ -283,8 +280,6 @@ export class PythonTestController implements ITestController, IExtensionSingleAc
                     '[test-by-project] Failed to activate project-based testing, falling back to legacy mode:',
                     error,
                 );
-                traceInfo('[test-by-project] Disabling project-based testing for this session');
-                this.useProjectBasedTesting = false;
             }
         }