Skip to content

[release/6.0-preview6] Handle more cases with the new entry point pattern #33633

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

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 18, 2021

Backport of #33500 to release/6.0-preview6

/cc @davidfowl

Customer Impact

This change fixes a couple of edge cases that would lead customers' unit tests to hanging when trying to use the new minimal hosting APIs with our existing testing framework (like if their application code threw an exception).

Testing

Unit tests (the scenario here enables). Unit testing similar to what code has been written in the change itself.

Risk

Low

…ion being thrown from main before start is called and make sure it propagates to the WebApplicationFactory. - Don't hang if the application doesn't call Start before it completes.
This reverts commit 2973167.
Copy link
Member

@javiercn javiercn left a comment

Choose a reason for hiding this comment

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

Looks great!

@Pilchie Pilchie merged commit 11de303 into release/6.0-preview6 Jun 18, 2021
@Pilchie Pilchie deleted the backport/pr-33500-to-release/6.0-preview6 branch June 18, 2021 18:51
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants