Cinka/OldThink
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Increase test time limit (#12911)

Browse Source
This commit is contained in:
Leon Friedrich
2022-12-07 19:34:58 +13:00
committed by GitHub
parent 3818063b9f
commit 8ad010ea9b
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
Content.IntegrationTests/PoolManagerTestEventHandler.cs
View File

@@ -9,8 +9,8 @@ namespace Content.IntegrationTests;
[SetUpFixture] [SetUpFixture]
public sealed class PoolManagerTestEventHandler public sealed class PoolManagerTestEventHandler
{ {
// This value is double the usual time for Content Integration tests // This value is completely arbitrary.
private static TimeSpan MaximumTotalTestingTimeLimit => TimeSpan.FromMinutes(7); private static TimeSpan MaximumTotalTestingTimeLimit => TimeSpan.FromMinutes(15);
private static TimeSpan HardStopTimeLimit => MaximumTotalTestingTimeLimit.Add(TimeSpan.FromMinutes(1)); private static TimeSpan HardStopTimeLimit => MaximumTotalTestingTimeLimit.Add(TimeSpan.FromMinutes(1));
[OneTimeSetUp] [OneTimeSetUp]
public void Setup() public void Setup()
@@ -18,6 +18,8 @@ public sealed class PoolManagerTestEventHandler
// If the tests seem to be stuck, we try to end it semi-nicely // If the tests seem to be stuck, we try to end it semi-nicely
_ = Task.Delay(MaximumTotalTestingTimeLimit).ContinueWith(_ => _ = Task.Delay(MaximumTotalTestingTimeLimit).ContinueWith(_ =>
{ {
// This can and probably will cause server/client pairs to shut down MID test, and will lead to really confusing test failures.
TestContext.Error.WriteLine($"\n\n{nameof(PoolManagerTestEventHandler)}: ERROR: Tests are taking too long. Shutting down all tests. This may lead to weird failures/exceptions.\n\n");
PoolManager.Shutdown(); PoolManager.Shutdown();
}); });