dotnet / dotnet/arcade

Add property to disable the default set of command-line options for testing

Open
#16,008 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
729
Forks
397
Avg merge
3d 15m
Merged PRs (30d)
149

Description

At least for MTP, we need that in microsoft/testfx.

See https://github.com/microsoft/testfx/pull/6140 for context.

Basically, we run official builds with Arcade, but PRs/public builds with `dotnet test`, and we want to unify everything to use `TestingPlatformCommandLineArguments`.

However, for `dotnet test` we will need to add `--results-directory` ourselves to `TestingPlatformCommandLineArguments`, but that then will be picked up by Arcade resulting in duplicate `--results-directory` which is an error.

The ask is to modify (at least) Microsoft.Testing.Platform.targets to conditionally include `--results-directory`, `--report-trx`, `--report-trx-filename`. These will be added by default. Repos that wish to opt-out from that behavior can specify `true`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.