ToxicMushroom/Dalamud
1
0
Fork 0
mirror of https://github.com/goatcorp/Dalamud.git synced 2025-12-12 18:27:23 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Correctly test Framework Run vs. RunOnTick

Browse source
This commit is contained in:
goaaats 2025-04-03 00:53:28 +02:00
parent c1805bd510
commit f812bf8f09
1 changed files with 30 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

35
Dalamud/Interface/Internal/Windows/SelfTest/Steps/FrameworkTaskSchedulerSelfTestStep.cs
View file

@ -3,6 +3,10 @@ using System.Threading.Tasks;
using Dalamud.Game;
using Dalamud.Utility;
using Microsoft.VisualBasic.Logging;
using Log = Serilog.Log;
namespace Dalamud.Interface.Internal.Windows.SelfTest.Steps;
/// <summary>
@ -25,15 +29,29 @@ internal class FrameworkTaskSchedulerSelfTestStep : ISelfTestStep
{
ThreadSafety.AssertNotMainThread();
await framework.Run(async () =>
{
ThreadSafety.AssertMainThread();
await Task.Delay(100).ConfigureAwait(true);
ThreadSafety.AssertMainThread();
await Task.Delay(100).ConfigureAwait(false);
ThreadSafety.AssertNotMainThread();
}).ConfigureAwait(true);
ThreadSafety.AssertNotMainThread();
await framework.RunOnTick(async () =>
{
ThreadSafety.AssertMainThread();
await Task.Delay(100);
// TODO: Should we be scheduled back to the framework thread here?
// ThreadSafety.AssertMainThread();
await Task.Delay(100).ConfigureAwait(true);
ThreadSafety.AssertNotMainThread();
}).ConfigureAwait(false);
await Task.Delay(100).ConfigureAwait(false);
ThreadSafety.AssertNotMainThread();
}).ConfigureAwait(true);
ThreadSafety.AssertNotMainThread();
@ -45,7 +63,14 @@ internal class FrameworkTaskSchedulerSelfTestStep : ISelfTestStep
ThreadSafety.AssertMainThread();
this.passed = true;
});
}).ContinueWith(
t =>
{
if (t.IsFaulted)
{
Log.Error(t.Exception, "Framework Task scheduler test failed");
}
});
if (this.task is { IsFaulted: true } or { IsCanceled: true })
{