Deduplicate command-line provider boilerplate via CommandLineOptionsProviderBase

[Copilot]

Multiple ICommandLineOptionsProvider implementations were repeating the same extension metadata and default validation/enablement boilerplate despite an existing shared base type. This change consolidates those providers on CommandLineOptionsProviderBase and adds a base constructor path for extension-delegated metadata.

• Problem addressed

  • Removes repeated Uid/Version/DisplayName/Description/default validation boilerplate across Platform and extension command-line providers.
  • Aligns providers on a single default behavior surface to reduce drift risk as command-line provider contracts evolve.

• Base abstraction updates

  • Added overload to CommandLineOptionsProviderBase:
    • CommandLineOptionsProviderBase(IExtension extension, IReadOnlyCollection<CommandLineOption> commandLineOptions)
  • Enables providers that delegate metadata to an injected extension to use the same base path as static-metadata providers.

• Provider migrations

  • Switched direct implementations to inherit from CommandLineOptionsProviderBase:
    • VSTestBridge: TestCaseFilter*, TestRunParameters*, RunSettings*
    • Extensions: Retry, Azure DevOps report, HTML report, TRX report
    • Platform: PlatformCommandLineProvider, MaxFailedTests*, TreeNodeFilter*, TerminalTestReporter*
  • Removed duplicated base-equivalent members from migrated providers.
  • Kept provider-specific validation logic intact (moved/retained as overrides where applicable).

• Unit test coverage

  • Extended CommandLineOptionsProviderBase tests for:
    • metadata propagation through the new IExtension constructor overload
    • null extension guard behavior

protected CommandLineOptionsProviderBase(
    IExtension extension,
    IReadOnlyCollection<CommandLineOption> commandLineOptions)
    : this(
        (extension ?? throw new ArgumentNullException(nameof(extension))).Uid,
        extension.Version,
        extension.DisplayName,
        extension.Description,
        commandLineOptions)
{
}

[Copilot]

Pull request overview

This PR consolidates repeated command-line provider boilerplate by moving migrated providers onto CommandLineOptionsProviderBase and adding an overload that initializes provider metadata from an IExtension.

Changes:

• Adds an IExtension-backed constructor overload to CommandLineOptionsProviderBase.
• Migrates Platform, VSTestBridge, Retry, Azure DevOps, HTML, TRX, and terminal command-line providers to inherit from the shared base.
• Adds unit tests for extension metadata propagation and null extension validation.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Platform/CommandLine/CommandLineOptionsProviderBase.cs	Adds the extension-backed base constructor.
src/Platform/Microsoft.Testing.Platform/CommandLine/PlatformCommandLineProvider.cs	Moves platform CLI provider metadata/options to the base class.
src/Platform/Microsoft.Testing.Platform/CommandLine/MaxFailedTestsCommandLineOptionsProvider.cs	Uses the new extension-backed base path while keeping validation.
src/Platform/Microsoft.Testing.Platform/CommandLine/TreeNodeFilterCommandLineOptionsProvider.cs	Simplifies tree-node filter provider via the base class.
src/Platform/Microsoft.Testing.Platform/OutputDevice/Terminal/TerminalTestReporterCommandLineOptionsProvider.cs	Moves terminal reporter metadata/options to the base class.
src/Platform/Microsoft.Testing.Extensions.VSTestBridge/CommandLine/TestCaseFilterCommandLineOptionsProvider.cs	Uses the base class for VSTest filter option metadata.
src/Platform/Microsoft.Testing.Extensions.VSTestBridge/CommandLine/TestRunParametersCommandLineOptionsProvider.cs	Uses the base class while preserving argument validation.
src/Platform/Microsoft.Testing.Extensions.VSTestBridge/CommandLine/RunSettingsCommandLineOptionsProvider.cs	Uses the base class while preserving runsettings validation.
src/Platform/Microsoft.Testing.Extensions.Retry/RetryCommandLineOptionsProvider.cs	Moves retry provider metadata/options to the base class.
src/Platform/Microsoft.Testing.Extensions.AzureDevOpsReport/AzureDevOpsCommandLineProvider.cs	Moves Azure DevOps provider metadata/options to the base class.
src/Platform/Microsoft.Testing.Extensions.HtmlReport/HtmlReportGeneratorCommandLine.cs	Moves HTML report provider metadata/options to the base class.
src/Platform/Microsoft.Testing.Extensions.TrxReport/TrxCommandLine.cs	Moves TRX report provider metadata/options to the base class.
test/UnitTests/Microsoft.Testing.Platform.UnitTests/CommandLine/CommandLineOptionsProviderBaseTests.cs	Adds tests for the new extension-backed constructor path.

Copilot's findings

• Files reviewed: 13/13 changed files
• Comments generated: 1

[Copilot]

Copilot's findings

• Files reviewed: 13/13 changed files
• Comments generated: 4

[Copilot]

Copilot's findings

• Files reviewed: 13/13 changed files
• Comments generated: 0 new

[Evangelink]

Fixed the failing acceptance tests (TestMaxFailedTestsShouldCallStopTestExecutionAsync, WhenCapabilityIsMissingShouldFail) in 22cbf2d.

Root cause: the new CommandLineOptionsProviderBase used Ensure.NotNullOrWhiteSpace on uid and version, but the original per-extension providers (e.g. TreeNodeFilterCommandLineOptionsProvider, MaxFailedTestsCommandLineOptionsProvider, RunSettingsCommandLineOptionsProvider) did no validation and just forwarded extension.Uid / extension.Version. The DummyTestFramework test asset in MaxFailedTestsExtensionTests legitimately returns string.Empty for Version/DisplayName/Description, so the new validation threw ArgumentException at construction time → exit code 57005 instead of 13/5.

Fix: Relaxed the base ctor to null-check only (matching how displayName/description were already handled). The whitespace unit tests were updated to assert no-throw, while the null tests are kept.

Validated locally with the CommandLineOptionsProviderBaseTests (13/13 pass) and a clean Debug build of Microsoft.Testing.Platform. CI should now pick up the acceptance tests.