Skip to content

Commit 5c9447d

Browse files
[tests] mark hanging test fixture [NonParallelizable] (#4621)
Hanging test prints: [TESTLOG] Test CheckClassIsReplacedWithMd5 Outcome=Passed This started happening since 835c59b, but I don't see how that commit would cause a hang. Working build (https://build.azdo.io/3669040) prints this after `CheckClassIsReplacedWithMd5`: => Xamarin.Android.Build.Tests.ResolveSdksTaskTests.LatestTFV_OldTargetSdkVersion After adding logging, etc. I couldn't figure out if this was a bug in NUnit or our tests. Adding `[NonParallelizable]` for the entire `ResolveSdksTaskTests` test fixture class seems to solve the hang, so we should just go with this for now.
1 parent d3e0fc8 commit 5c9447d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/Tasks/ResolveSdksTaskTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ namespace Xamarin.Android.Build.Tests {
1313

1414
[TestFixture]
1515
[Category ("Node-2")]
16-
[Parallelizable (ParallelScope.Self)]
16+
[NonParallelizable] // NOTE: This test was hanging without this
1717
public class ResolveSdksTaskTests : BaseTest {
1818
#pragma warning disable 414
1919

0 commit comments

Comments
 (0)