Add exclusion pattern support to --test-modules glob expressions

[Copilot]

New Feature

What does this feature do?

--test-modules now supports !-prefixed exclusion patterns in the semicolon-separated glob expression, consistent with VSTest behavior. Patterns with and without ! can be freely mixed:

dotnet test --test-modules "*.mstest.exe"
dotnet test --test-modules "!*.toexclude.exe"
dotnet test --test-modules "*.mstest.exe; !*.toexclude.mstest.exe"

Why is this feature needed?

Previously all patterns were treated as includes only, making it impossible to exclude specific assemblies from a broad glob match without listing every desired assembly explicitly.

Implementation details

• TestModulesFilterHandler.GetMatchedModulePaths: Each ;-separated token is now trimmed of surrounding whitespace, then routed to either Matcher.AddExclude (if prefixed with !, stripping the prefix and any trailing whitespace) or Matcher.AddInclude.
• Implicit include: When the expression contains only exclusion patterns, ** is implicitly added as an include so the result is "all files minus excluded."
• Test: Added RunTestProjectsWithIncludeAndExcludeFilterOfDll_ShouldReturnExitCodeSuccess — builds two test projects, runs with a combined include+exclude filter that drops the failing project, and asserts only the passing project executed.

[Copilot]

Pull request overview

Adds support for !-prefixed exclusion patterns in dotnet test --test-modules glob expressions (MTP path filtering), aligning behavior with common globbing semantics and enabling “include broad, then exclude specific” scenarios.

Changes:

• Update TestModulesFilterHandler.GetMatchedModulePaths to parse ;-separated patterns as includes vs excludes (leading !) and trim whitespace.
• Add implicit include ("**") when only exclusion patterns are provided.
• Add an integration test validating combined include+exclude module filtering.

Show a summary per file

File	Description
test/dotnet.Tests/CommandTests/Test/GivenDotnetTestBuildsAndRunsTestBasedOnGlobbingFilter.cs	Adds a test that validates include+exclude behavior for --test-modules.
src/Cli/dotnet/Commands/Test/MTP/TestModulesFilterHandler.cs	Implements parsing and matching logic for exclusion patterns and exclusion-only expressions.
src/Microsoft.CodeAnalysis.NetAnalyzers/src/Microsoft.CodeAnalysis.NetAnalyzers.sarif	Appears to change only file encoding/format (BOM/whitespace), unrelated to the feature.

Copilot's findings

• Files reviewed: 3/3 changed files
• Comments generated: 2

[Evangelink]

@copilot address review comments

[Copilot]

@copilot address review comments

Addressed both review comments:

1. SARIF encoding change — was already reverted in a prior session (commit 437dc17); the file is back to its original encoding.

2. Exclusion-only test coverage — GetMatchedModulePaths is now internal (dotnet.Tests already has InternalsVisibleTo access), and two focused unit tests were added:

   • ExclusionOnlyFilterPattern_MatchesAllFilesExceptExcluded — creates a temp dir with three known files, uses !ExcludeMe.dll (exclusion-only), and asserts exactly the two non-excluded files are returned.
   • ExclusionOnlyFilterPattern_WithSpaceAfterBang_MatchesCorrectly — validates the ! ExcludeMe.dll (space after !) trimming edge case as well.

All 6 tests pass.