Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Disable test in System.Net.Sockets for UAPAOT #21058

Merged
merged 1 commit into from
Jun 14, 2017

Conversation

caesar-chen
Copy link
Contributor

Now all Sockets Functional tests (innerloop and outerloop) can run clean on both UAP and UAPAOT mode.

Contributes to: #20135

@caesar-chen caesar-chen added api-approved API was approved in API review, it can be implemented os-windows-uwp labels Jun 14, 2017
@caesar-chen caesar-chen added this to the UWP6.0 milestone Jun 14, 2017
@caesar-chen caesar-chen self-assigned this Jun 14, 2017
@caesar-chen caesar-chen added area-System.Net.Sockets and removed api-approved API was approved in API review, it can be implemented labels Jun 14, 2017
@@ -623,6 +623,7 @@ public void Ctor_NotConnected_ThrowsIOException()
[InlineData(4096)]
[InlineData(4095)]
[InlineData(1024*1024)]
[ActiveIssue(20135, TargetFrameworkMonikers.Uap)]
Copy link
Member

Choose a reason for hiding this comment

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

I looked at the issue, but there's not much detail in there. Is the relevant work known / being tracked somewhere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Our current mission is to make all tests run clean for UWP. There is no known issue to track the problem, because I'm still investigating it. My following work for S.N.Sockets is to solve those ActiveIssue label: submit a fix or open a new issue.

cc: @davidsh

@caesar-chen caesar-chen merged commit d9df3ed into dotnet:master Jun 14, 2017
@caesar-chen caesar-chen deleted the socket_uwp branch June 14, 2017 21:53
@karelz karelz modified the milestones: UWP6.0, 2.1.0 Jun 16, 2017
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.

5 participants