fixed test to expect a result after the LoadCredentials graceful failure change

ibetitsmike · ibetitsmike · commit ab7cea5d6bbd · 2025-07-28T17:04:12.000+02:00
diff --git a/Tests.App/Services/CredentialManagerTest.cs b/Tests.App/Services/CredentialManagerTest.cs
@@ -317,7 +317,10 @@ public async Task SetDuringLoad(CancellationToken ct)
         var loadTask = manager.LoadCredentials(ct);
         // Then fully perform a set.
         await manager.SetCredentials(TestServerUrl, TestApiToken, ct).WaitAsync(ct);
-        // The load should have been cancelled.
-        Assert.ThrowsAsync<TaskCanceledException>(() => loadTask);
+
+        // The load should complete with the new valid credentials
+        var result = await loadTask;
+        Assert.That(result.State, Is.EqualTo(CredentialState.Valid));
+        Assert.That(result.CoderUrl?.ToString(), Is.EqualTo(TestServerUrl));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -317,7 +317,10 @@ public async Task SetDuringLoad(CancellationToken ct)`
`317`	`317`	`var loadTask = manager.LoadCredentials(ct);`
`318`	`318`	`// Then fully perform a set.`
`319`	`319`	`await manager.SetCredentials(TestServerUrl, TestApiToken, ct).WaitAsync(ct);`
`320`		`- // The load should have been cancelled.`
`321`		`- Assert.ThrowsAsync<TaskCanceledException>(() => loadTask);`
	`320`	`+`
	`321`	`+ // The load should complete with the new valid credentials`
	`322`	`+ var result = await loadTask;`
	`323`	`+ Assert.That(result.State, Is.EqualTo(CredentialState.Valid));`
	`324`	`+ Assert.That(result.CoderUrl?.ToString(), Is.EqualTo(TestServerUrl));`
`322`	`325`	`}`
`323`	`326`	`}`