Skip to content

Update VSTest docs: fix archived cross-links, add runsettings CLI override and logger catalog#11396

Closed
azat-msft wants to merge 5 commits into
MicrosoftDocs:mainfrom
azat-msft:vstest-docs-enrichment
Closed

Update VSTest docs: fix archived cross-links, add runsettings CLI override and logger catalog#11396
azat-msft wants to merge 5 commits into
MicrosoftDocs:mainfrom
azat-msft:vstest-docs-enrichment

Conversation

@azat-msft

Copy link
Copy Markdown

Enrichment of the VSTest documentation on Microsoft Learn (VS docs test area), reconciled against the maintained microsoft/vstest repository.

Files

  • docs/test/vstest-console-options.md — update cross-reference links that pointed at the archived microsoft/vstest-docs repo to the maintained microsoft/vstest repo (filter, blame data collector, analyze), and add a pointer to the full logger catalog + logger-authoring guide.
  • docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md — add an "Override run settings from the command line" section covering the -- name=value inline override syntax, TestRunParameters via the command line, and shell-specific quoting. Examples use vstest.console.exe (the page's context; the -- name=value override is a native vstest.console.exe feature via CliRunSettingsArgumentProcessor), with a note that dotnet test uses the same syntax.

Opened from a fork; happy to adjust to match the docs team's conventions.

azat-msft and others added 3 commits July 13, 2026 16:29
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 4e840b63-ad95-40c2-9ef8-5f2fcc4cfdaa
Cross-reference microsoft/vstest report.md for the full list of built-in and
community test loggers and instructions for authoring a logger.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 4e840b63-ad95-40c2-9ef8-5f2fcc4cfdaa
The command-line override section is introduced with vstest.console.exe but used
dotnet test examples. vstest.console.exe supports the same -- name=value inline
runsettings override natively (CliRunSettingsArgumentProcessor), so show the
vstest.console.exe form to match the page's context, and note dotnet test uses
the same syntax.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 4e840b63-ad95-40c2-9ef8-5f2fcc4cfdaa
@prmerger-automator

Copy link
Copy Markdown
Contributor

@azat-msft : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @Mikejo5000

@prmerger-automator

Copy link
Copy Markdown
Contributor

@azat-msft : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @Mikejo5000

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit b7ccf26:

✅ Validation status: passed

File Status Preview URL Details
docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md ✅Succeeded
docs/test/vstest-console-options.md ✅Succeeded

For more details, please refer to the build report.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Visual Studio VSTest documentation to (1) replace cross-references that pointed to the archived microsoft/vstest-docs repository with links to the maintained microsoft/vstest repository, and (2) document how to override .runsettings values from the command line.

Changes:

  • Updated GitHub cross-links for TestCase filter, blame data collector, and data collection docs to reference microsoft/vstest.
  • Added a pointer to the upstream logger catalog / logger authoring guidance.
  • Added a new section describing command-line runsettings overrides (including TestRunParameters) and shell quoting considerations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
docs/test/vstest-console-options.md Updates archived GitHub cross-links and adds a new logger-catalog pointer.
docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md Adds command-line runsettings override guidance and a reference to upstream quoting examples.

Comment thread docs/test/vstest-console-options.md Outdated
Comment thread docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md Outdated
Comment thread docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Court72

Court72 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@BillWagner

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Jul 14, 2026
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit bc282e7:

✅ Validation status: passed

File Status Preview URL Details
docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md ✅Succeeded
docs/test/vstest-console-options.md ✅Succeeded

For more details, please refer to the build report.

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a bit more work here before this is ready.

Also, tagging @RoseHJM for review

vstest.console.exe test.dll -- TestRunParameters.Parameter(name="myParam", value="value")
```

Shell quoting rules differ. In Command Prompt, escape quotation marks as needed. In PowerShell 7.3 and later, native command argument handling supports this syntax more directly; in earlier PowerShell versions, or when `$PSNativeCommandArgumentPassing` is set to `legacy`, use the `--%` stop-parsing token. In Bash, escape characters such as parentheses, quotation marks, spaces, and semicolons.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This paragraph should also include the rules for unix shells, using zsh as the example.

@azat-msft azat-msft Jul 15, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added a paragraph covering unix shell quoting with zsh

Address PR review feedback: document unix shell (zsh) escaping rules for the `TestRunParameters.Parameter(...)` override syntax, using zsh as the macOS default-shell example.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0cc38362-9c3b-46eb-8142-9c49c63dc4a0
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 04ba0c2:

✅ Validation status: passed

File Status Preview URL Details
docs/test/configure-unit-tests-by-using-a-dot-runsettings-file.md ✅Succeeded
docs/test/vstest-console-options.md ✅Succeeded

For more details, please refer to the build report.

@v-dirichards

Copy link
Copy Markdown
Contributor

@BillWagner Could you review this proposed update to your article and enter #sign-off in a comment if it's ready to merge?

Thanks!

@azat-msft

Copy link
Copy Markdown
Author

Superseded by #11401, which carries all of this PR's changes (archived-link fixes, runsettings CLI override + zsh quoting, logger catalog pointer) and expands the VSTest doc work into a single coherent change: a new exit-codes section, command-line filter/shell escaping, blame crash/hang dump coverage, and CLI option gaps in \�stest-console-options.md, plus the remaining archived \�stest-docs\ links repointed to \microsoft/vstest. Consolidating here to avoid parallel edits to the same files. Closing in favor of #11401.

@azat-msft azat-msft closed this Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants