Skip to content

fix exception handler unit test #48459

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
May 26, 2023
Merged

fix exception handler unit test #48459

merged 1 commit into from
May 26, 2023

Conversation

bjornen77
Copy link
Contributor

The unit test now uses the correct exception handler.

@bjornen77
fix unit test
cb17f0f 
The unit test now uses the correct exception handler.
@ghost ghost added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates area-runtime community-contribution Indicates that the PR has been added by a community member labels May 26, 2023
@ghost
Copy link

ghost commented May 26, 2023

Thanks for your PR, @bjornen77. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

Tratcher
Tratcher approved these changes May 26, 2023
View reviewed changes
Copy link
Member

@Tratcher Tratcher left a comment

Choose a reason for hiding this comment

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

Glad they both work 😆

@Tratcher Tratcher enabled auto-merge (squash) May 26, 2023 20:29
@Tratcher Tratcher self-assigned this May 26, 2023
@Tratcher Tratcher added feature-diagnostics Diagnostic middleware and pages (except EF diagnostics) and removed area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates labels May 26, 2023
@Tratcher Tratcher merged commit 2a70935 into dotnet:main May 26, 2023
@ghost ghost added this to the 8.0-preview6 milestone May 26, 2023
@bjornen77 bjornen77 deleted the fix-unit-test branch May 26, 2023 21:58
@amcasey amcasey added area-middleware Includes: URL rewrite, redirect, response cache/compression, session, and other general middlewares and removed area-runtime labels Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tratcher Tratcher Tratcher approved these changes

@BrennanConroy BrennanConroy Awaiting requested review from BrennanConroy BrennanConroy is a code owner

Assignees

@Tratcher Tratcher

Labels
area-middleware Includes: URL rewrite, redirect, response cache/compression, session, and other general middlewares community-contribution Indicates that the PR has been added by a community member feature-diagnostics Diagnostic middleware and pages (except EF diagnostics)
Projects
None yet
Milestone
8.0-preview6
Development

Successfully merging this pull request may close these issues.
3 participants
@bjornen77 @Tratcher @amcasey