Skip to content

[rel/3.9] Add missing overload for Assert.Throws - #5620

Merged
Youssef Fahmy (Youssef1313) merged 3 commits into
microsoft:rel/3.9from
youssef-backport-bot:backport/5619/to/rel/3.9
May 26, 2025
Merged

Youssef Fahmy (Youssef1313) merged 3 commits into
microsoft:rel/3.9from
youssef-backport-bot:backport/5619/to/rel/3.9

Conversation

@youssef-backport-bot

Copy link
Copy Markdown
Contributor

Backport of #5619 to rel/3.9

/cc Youssef Fahmy (@Youssef1313)

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

Backport of support for interpolated string overloads in Assert.Throws to the rel/3.9 branch

  • Introduces a new AssertNonStrictThrowsInterpolatedStringHandler constructor that accepts a Func<object?> action
  • Adds a Throws<TException>(Func<object?>, handler) overload
  • Adds a unit test Throws_WithInterpolation to verify overload resolution with interpolated strings

Reviewed Changes

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

File Description
test/UnitTests/TestFramework.UnitTests/Assertions/AssertTests.ThrowsExceptionTests.cs Adds Throws_WithInterpolation test exercising the new overloads
src/TestFramework/TestFramework/Assertions/Assert.ThrowsException.cs Adds handler constructor and Throws(Func<object?>, handler) overload

@Youssef1313

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@youssef-backport-bot
youssef-backport-bot force-pushed the backport/5619/to/rel/3.9 branch from cb778d2 to bfca717 Compare May 21, 2025 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants