Skip to content

Added coverage for developer exception page #35778

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.

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

Merged
merged 2 commits into from
Aug 26, 2021

Conversation

davidfowl
Copy link
Member

  • Added tests to verify that the developer exception page is on in development.

Fixes #35423

- Added tests to verify that the developer exception page is on in development.
@davidfowl
Copy link
Member Author

davidfowl commented Aug 26, 2021

@Pilchie I plan to backport this to rc2. It's adding missing test coverage, no product changes.

@BrennanConroy
Copy link
Member

Oh, remove the comment too

@Pilchie
Copy link
Member

Pilchie commented Aug 26, 2021

No problem - RC2 doesn't require approval yet, and even if it did, test only stuff generally is tell mode.

@davidfowl davidfowl merged commit 85a0679 into main Aug 26, 2021
@davidfowl davidfowl deleted the davidfowl/added-waf-test-coverage branch August 26, 2021 18:38
@ghost ghost added this to the 7.0-preview1 milestone Aug 26, 2021
@davidfowl
Copy link
Member Author

/backport to release/6.0

@github-actions
Copy link
Contributor

Started backporting to release/6.0: https://github.com/dotnet/aspnetcore/actions/runs/1171600083

davidfowl added a commit that referenced this pull request Sep 12, 2021
@amcasey amcasey added area-minimal Includes minimal APIs, endpoint filters, parameter binding, request delegate generator etc 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
Labels
area-minimal Includes minimal APIs, endpoint filters, parameter binding, request delegate generator etc feature-minimal-hosting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing test coverage in WebApplicationBuilder
4 participants