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

[QUIC] Fix TestConnect #49660

Merged
merged 2 commits into from
Mar 17, 2021
Merged

[QUIC] Fix TestConnect #49660

merged 2 commits into from
Mar 17, 2021

Conversation

CarnaViire
Copy link
Member

Contributes to #49157

@ghost
Copy link

ghost commented Mar 15, 2021

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

Issue Details

Contributes to #49157

Author: CarnaViire
Assignees: -
Labels:

area-System.Net.Quic

Milestone: -

Copy link
Member

@ManickaP ManickaP left a comment

Choose a reason for hiding this comment

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

Just a question about the ActiveIssue, otherwise LGTM, thanks.

@@ -82,6 +82,7 @@ public async Task SetListenerTimeoutWorksWithSmallTimeout()
await Assert.ThrowsAsync<QuicOperationAbortedException>(async () => await serverConnection.AcceptStreamAsync().TimeoutAfter(100000));
}

[ActiveIssue("https://github.com/dotnet/runtime/issues/49157")]
Copy link
Member

Choose a reason for hiding this comment

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

We're not running these in CI AFAIK so the failures are not blocking any PRs. This will disable them for our local runs, is that what you're goal here is?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, it is for us to have a clean local test run, since we are to start adding new changes. So if some new failures appear, we will know it was because of our changes

@ManickaP
Copy link
Member

Failures are unrelated.

@ManickaP ManickaP merged commit 49e5d17 into dotnet:main Mar 17, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
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.

3 participants