Skip to content
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

PR: Add test code coverage for Host.StopAsync #40875

Merged
2 commits merged into from
Dec 15, 2020

Conversation

WernerMairl
Copy link
Contributor

Verifies StopAsync() doesn't throw CanceledException given a canceled token

Fix #40700

Verifies StopAsync() doesn't throw CanceledException given a canceled token

Fix dotnet#40700
@ghost
Copy link

ghost commented Aug 15, 2020

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@ghost
Copy link

ghost commented Aug 15, 2020

Tagging subscribers to this area: @eerhardt, @maryamariyan
See info in area-owners.md if you want to be subscribed.

@eerhardt
Copy link
Member

All the Microsoft.Extensions.Hosting.Unit.Tests legs appear to be failing. In a few I see log entries like:

[EXECUTION TIMED OUT]
Exit Code:-3Executor timed out after 900 seconds and was killed

@maryamariyan
Copy link
Member

/azp run runtime

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@maryamariyan
Copy link
Member

/azp run runtime

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ghost
Copy link

ghost commented Dec 14, 2020

Hello @danmosemsft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@danmoseley
Copy link
Member

Going through old PR's. Let's see if we can get this one merged

@ghost ghost merged commit 5a3d44e into dotnet:master Dec 15, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Jan 14, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add test code coverage for Host.StopAsync
6 participants