refactor(testing): simplify exclusion filtering in getTestItemsForWorkspace

Remove isTestItemExcluded and exclusion logic from getTestItemsForWorkspace
since exclusions are fully handled downstream via expandExcludeSet and
getTestCaseNodes in WorkspaceTestAdapter.executeTests.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: andravin

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

@@ -498,23 +498,6 @@ export async function updateTestItemFromRawData(
     item.busy = false;
 }
 
-/**
- * Checks if a test item or any of its ancestors is in the exclude set.
- */
-export function isTestItemExcluded(item: TestItem, excludeSet: Set<TestItem> | undefined): boolean {
-    if (!excludeSet || excludeSet.size === 0) {
-        return false;
-    }
-    let current: TestItem | undefined = item;
-    while (current) {
-        if (excludeSet.has(current)) {
-            return true;
-        }
-        current = current.parent;
-    }
-    return false;
-}
-
 /**
  * Expands an exclude set to include all descendants of excluded items.
  * After expansion, checking if a node is excluded is O(1) - just check set membership.

src/client/testing/testController/controller.ts

@@ -34,7 +34,7 @@ import { IEventNamePropertyMapping, sendTelemetryEvent } from '../../telemetry';
 import { EventName } from '../../telemetry/constants';
 import { PYTEST_PROVIDER, UNITTEST_PROVIDER } from '../common/constants';
 import { TestProvider } from '../types';
-import { createErrorTestItem, DebugTestTag, getNodeByUri, isTestItemExcluded, RunTestTag } from './common/testItemUtilities';
+import { createErrorTestItem, DebugTestTag, getNodeByUri, RunTestTag } from './common/testItemUtilities';
 import { buildErrorNodeOptions } from './common/utils';
 import { ITestController, ITestFrameworkController, TestRefreshOptions } from './common/types';
 import { WorkspaceTestAdapter } from './workspaceTestAdapter';
@@ -848,14 +848,12 @@ export class PythonTestController implements ITestController, IExtensionSingleAc
      */
     private getTestItemsForWorkspace(workspace: WorkspaceFolder, request: TestRunRequest): TestItem[] {
         const testItems: TestItem[] = [];
-        const excludeSet = request.exclude?.length ? new Set(request.exclude) : undefined;
         // If the run request includes test items then collect only items that belong to
         // `workspace`. If there are no items in the run request then just run the `workspace`
         // root test node. Include will be `undefined` in the "run all" scenario.
-        // Exclusions are applied after inclusions per VS Code API contract.
         (request.include ?? this.testController.items).forEach((i: TestItem) => {
             const w = this.workspaceService.getWorkspaceFolder(i.uri);
-            if (w?.uri.fsPath === workspace.uri.fsPath && !isTestItemExcluded(i, excludeSet)) {
+            if (w?.uri.fsPath === workspace.uri.fsPath) {
                 testItems.push(i);
             }
         });

src/test/testing/testController/testItemUtilities.unit.test.ts

@@ -1,7 +1,6 @@
 import * as assert from 'assert';
 import { TestItem } from 'vscode';
 import {
-    isTestItemExcluded,
     expandExcludeSet,
     getTestCaseNodes,
     RunTestTag,
@@ -28,47 +27,6 @@ function createMockTestItem(id: string, canResolveChildren: boolean, children: T
     return item;
 }
 
-suite('isTestItemExcluded', () => {
-    test('should return false when excludeSet is undefined', () => {
-        const item = createMockTestItem('item1', false);
-        assert.strictEqual(isTestItemExcluded(item, undefined), false);
-    });
-
-    test('should return false when excludeSet is empty', () => {
-        const item = createMockTestItem('item1', false);
-        assert.strictEqual(isTestItemExcluded(item, new Set()), false);
-    });
-
-    test('should return true when item is directly in excludeSet', () => {
-        const item = createMockTestItem('item1', false);
-        const excludeSet = new Set([item]);
-        assert.strictEqual(isTestItemExcluded(item, excludeSet), true);
-    });
-
-    test('should return true when ancestor is in excludeSet', () => {
-        const child = createMockTestItem('child', false);
-        const parent = createMockTestItem('parent', true, [child]);
-        const excludeSet = new Set([parent]);
-        assert.strictEqual(isTestItemExcluded(child, excludeSet), true);
-    });
-
-    test('should return false when unrelated item is in excludeSet', () => {
-        const child = createMockTestItem('child', false);
-        createMockTestItem('parent', true, [child]);
-        const other = createMockTestItem('other', false);
-        const excludeSet = new Set([other]);
-        assert.strictEqual(isTestItemExcluded(child, excludeSet), false);
-    });
-
-    test('should walk multiple levels of ancestor chain', () => {
-        const grandchild = createMockTestItem('grandchild', false);
-        const child = createMockTestItem('child', true, [grandchild]);
-        const root = createMockTestItem('root', true, [child]);
-        const excludeSet = new Set([root]);
-        assert.strictEqual(isTestItemExcluded(grandchild, excludeSet), true);
-    });
-});
-
 suite('expandExcludeSet', () => {
     test('should return undefined when excludeSet is undefined', () => {
         assert.strictEqual(expandExcludeSet(undefined), undefined);