-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Diable System.Net.Http.FunctionalTests parallel execution on UAP run #35004
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
// Licensed to the .NET Foundation under one or more agreements. | ||
// The .NET Foundation licenses this file to you under the MIT license. | ||
// See the LICENSE file in the project root for more information. | ||
|
||
using Xunit; | ||
|
||
// [ActiveIssue(35002)]: Disabling parallel execution of System.Net.Http.FunctionalTests | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: you should modify the comment like this: "Disable parallel execution of System.Net.Http.FunctionalTests on UAP test runs" This way it is obvious if someone looks at this file they will know that this behavior is only for UAP test runs. Otherwise, they will have to look at the CSPROJ file. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will fix. |
||
[assembly: CollectionBehavior(CollectionBehavior.CollectionPerAssembly, DisableTestParallelization = true, MaxParallelThreads = 1)] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. If "DisableTestParallelization = true" then does it matter whether or not you change "MaxParallelThread" at all, or even have it mentioned in this file? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I did some search online, having both is useful in some scenarios, for example: when some components in the test project are not compatible with the TPL, and xUnit uses TPL extensively, even when tests aren't being run in parallel. I think it's fine to only keep |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you just want to import this for uap?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. So I put it under
<ItemGroup Condition="'$(TargetGroup)' == 'uap'">
line 146.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah didn't see that on my phone. Thx