Update dotnet run tests for launch settings message on stderr by MichaelSimons · Pull Request #53905 · dotnet/sdk

MichaelSimons · 2026-04-15T23:29:52Z

PR #53797 moved the 'Using launch settings from...' message from stdout to stderr but only updated dotnet test assertions. This updates the dotnet run test assertions to match:

StdErr.Should().BeEmpty() → StdErr.Should().Contain('Using launch settings from') for tests that use launch profiles
HaveStdOutContaining(UsingLaunchSettingsFromMessage) → HaveStdErrContaining
NotHaveStdOutContaining(UsingLaunchSettingsFromMessage) → NotHaveStdErrContaining

Fixes ~11 test failures across Windows, Linux, and macOS CI legs.

These test failures weren't surfaced in #53797 because we had a test outage last week.

PR #53797 moved the 'Using launch settings from...' message from stdout to stderr but only updated dotnet test assertions. This updates the dotnet run test assertions to match: - StdErr.Should().BeEmpty() → StdErr.Should().Contain('Using launch settings from') for tests that use launch profiles - HaveStdOutContaining(UsingLaunchSettingsFromMessage) → HaveStdErrContaining - NotHaveStdOutContaining(UsingLaunchSettingsFromMessage) → NotHaveStdErrContaining Fixes ~11 test failures across Windows, Linux, and macOS CI legs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

MichaelSimons · 2026-04-15T23:31:17Z

@Christian-Sidak, Can you please review? TIA

Copilot

Pull request overview

Updates dotnet run tests to reflect the behavioral change from PR #53797 where the “Using launch settings from …” diagnostic is written to stderr instead of stdout, fixing CI test failures across platforms.

Changes:

Adjusts assertions in run-related tests to expect the launch settings message on stderr.
Replaces Have/NotHaveStdOutContaining(UsingLaunchSettingsFromMessage) with the corresponding stderr variants for launch-profile scenarios.
Updates tests that previously asserted empty stderr to instead assert presence/absence of the launch settings diagnostic.

Reviewed changes

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

File	Description
test/dotnet.Tests/CommandTests/Run/RunCommandTests.cs	Updates stderr expectations for launch settings diagnostics in a `dotnet run` integration test.
test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsVbProj.cs	Moves launch settings message assertions to stderr for VB `dotnet run` tests.
test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs	Moves launch settings message assertions to stderr for C# `dotnet run` tests and updates quiet/non-quiet scenarios.

marcpopMSFT · 2026-04-15T23:40:20Z

Overlap with #53906. We should check tomorrow to see if there's anything either of us missed between the two.

- CS ItDefaultsToTheFirstUsableLaunchProfile: restore StdErr.BeEmpty() since --verbosity quiet suppresses the launch settings message - VB ItDefaultsToTheFirstUsableLaunchProfile: fix path from Properties to My Project and assert message IS present on stderr Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

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

Comments suppressed due to low confidence (1)

test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs:424

In ItDefaultsToTheFirstUsableLaunchProfile, launchSettingsPath is still being computed but is no longer used after removing the assertion that referenced it. With TreatWarningsAsErrors enabled, this will fail the build; remove the unused local or use it in an assertion (e.g., against stderr) as intended.

            cmd.Should().Pass()
                .And.HaveStdOutContaining("First");

            cmd.StdErr.Should().BeEmpty();
        }

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

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Replace all hardcoded 'Using launch settings from' checks with string.Format(CliCommandStrings.UsingLaunchSettingsFromMessage, launchSettingsPath) for proper localization support and stronger assertions. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

Copilot

Pull request overview

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

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

MichaelSimons · 2026-04-16T13:47:36Z

/ba-g The failing container tests are tracked with #53918. The failing RunFileTests_CscOnlyAndApi.CscOnly_SpacesInPath test is tracked with #53869

Copilot AI review requested due to automatic review settings April 15, 2026 23:29

Copilot started reviewing on behalf of MichaelSimons April 15, 2026 23:30 View session

MichaelSimons requested a review from baronfel April 15, 2026 23:30

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsVbProj.cs

Comment thread test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs Outdated

MichaelSimons mentioned this pull request Apr 15, 2026

Fix launch settings stderr test assertions and container digest validation #53906

Open

MichaelSimons requested a review from Copilot April 15, 2026 23:46

Copilot started reviewing on behalf of MichaelSimons April 15, 2026 23:47 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs Outdated

Comment thread test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs Outdated

Remove unused launchSettingsPath variables to fix warning-as-error

7ffe863

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

MichaelSimons requested a review from Copilot April 15, 2026 23:56

Copilot started reviewing on behalf of MichaelSimons April 15, 2026 23:56 View session

Copilot AI reviewed Apr 15, 2026

View reviewed changes

Comment thread test/dotnet.Tests/CommandTests/Run/GivenDotnetRunBuildsCsProj.cs

Copilot and others added 2 commits April 16, 2026 00:05

Remove unused launchSettingsPath in quiet-mode test

7ed3b6e

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

MichaelSimons requested a review from Copilot April 16, 2026 00:08

Copilot started reviewing on behalf of MichaelSimons April 16, 2026 00:08 View session

Copilot AI reviewed Apr 16, 2026

View reviewed changes

MiYanni mentioned this pull request Apr 16, 2026

Re-enable C++/CLI, SourceLink C++, and compilation context tests now that VS 2026 Helix images include VCTools #53820

Merged

baronfel mentioned this pull request Apr 16, 2026

Fix dotnet run tests for launch settings stderr change #53908

Closed

Add missing using directive for CliCommandStrings in RunCommandTests

6570229

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

build-analysis Bot mentioned this pull request Apr 16, 2026

[Known Build Error] RunFileTests csc-only mode fails expecting binary log warning that is not emitted #53869

Closed

MichaelSimons mentioned this pull request Apr 16, 2026

[Known Build Error] Invalid digest failures in Containers.UnitTests #53918

Closed

marcpopMSFT approved these changes Apr 16, 2026

View reviewed changes

MichaelSimons merged commit 3dd3634 into main Apr 16, 2026
18 of 24 checks passed

MichaelSimons deleted the fix/update-run-tests-for-stderr-launch-settings branch April 16, 2026 16:27

dotnet-maestro Bot mentioned this pull request Apr 17, 2026

[main] Source code updates from dotnet/sdk dotnet/dotnet#6096

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update dotnet run tests for launch settings message on stderr#53905

Update dotnet run tests for launch settings message on stderr#53905
MichaelSimons merged 6 commits intomainfrom
fix/update-run-tests-for-stderr-launch-settings

MichaelSimons commented Apr 15, 2026 •

edited

Loading

Uh oh!

MichaelSimons commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

marcpopMSFT commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

MichaelSimons commented Apr 16, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

MichaelSimons commented Apr 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MichaelSimons commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

marcpopMSFT commented Apr 15, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

MichaelSimons commented Apr 16, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

MichaelSimons commented Apr 15, 2026 •

edited

Loading