Simplify logic flow in JobsHubTests

This commit is contained in:
Jordan Dominion
2023-11-05 09:00:44 -05:00
parent cab8ce68f5
commit 2c82da46f6
@@ -92,13 +92,12 @@ namespace Tgstation.Server.Tests.Live.Instance
public async Task Run(CancellationToken cancellationToken)
{
var neverReceiverTcs = new TaskCompletionSource();
var neverReceiver = new ShouldNeverReceiveUpdates()
{
Callback = job =>
{
if (!permlessIsPermed)
neverReceiverTcs.TrySetException(new Exception($"ShouldNeverReceiveUpdates received an update for job {job.Id}!"));
finishTcs.TrySetException(new Exception($"ShouldNeverReceiveUpdates received an update for job {job.Id}!"));
else
lock (permlessSeenJobs)
permlessSeenJobs.Add(job.Id.Value);
@@ -131,13 +130,9 @@ namespace Tgstation.Server.Tests.Live.Instance
return Task.CompletedTask;
};
var completedTask = await Task.WhenAny(finishTcs.Task, neverReceiverTcs.Task);
await completedTask;
await finishTcs.Task;
}
neverReceiverTcs.TrySetResult();
await neverReceiverTcs.Task;
var allInstances = await permedUser.Instances.List(null, cancellationToken);
async ValueTask<List<JobResponse>> CheckInstance(InstanceResponse instance)