Skip to content

Commit ca7c99f

Browse files
fix formatting and nullable warnings in UserAccessAccessTokenManagerTests
- Fix import ordering (OpenIdConnect before OTel) to pass Verify Formatting - Add null-forgiving operator on FailedResult to suppress CS8602 - Correct misleading test comment about HttpContextUserAccessor fallback
1 parent 2978cd2 commit ca7c99f

1 file changed

Lines changed: 4 additions & 5 deletions

File tree

access-token-management/test/AccessTokenManagement.Tests/UserAccessAccessTokenManagerTests.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@
22
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.
33

44
using System.Security.Claims;
5-
using Duende.AccessTokenManagement.OTel;
65
using Duende.AccessTokenManagement.OpenIdConnect;
76
using Duende.AccessTokenManagement.OpenIdConnect.Internal;
7+
using Duende.AccessTokenManagement.OTel;
88
using Microsoft.Extensions.Logging.Abstractions;
99
using Microsoft.Extensions.Options;
1010

@@ -18,8 +18,7 @@ public class UserAccessAccessTokenManagerTests
1818
public async Task GetAccessTokenAsync_with_null_identity_should_return_failure()
1919
{
2020
// A ClaimsPrincipal created with no arguments has Identity == null.
21-
// This happens when HttpContextUserAccessor falls back because
22-
// there is no HttpContext (e.g., background service).
21+
// This can happen if a custom IUserAccessor returns a default principal.
2322
var user = new ClaimsPrincipal();
2423

2524
var sut = CreateSut();
@@ -28,7 +27,7 @@ public async Task GetAccessTokenAsync_with_null_identity_should_return_failure()
2827

2928
result.Succeeded.ShouldBeFalse();
3029
result.FailedResult.ShouldNotBeNull();
31-
result.FailedResult.Error.ShouldBe("No active user");
30+
result.FailedResult!.Error.ShouldBe("No active user");
3231
}
3332

3433
[Fact]
@@ -43,7 +42,7 @@ public async Task GetAccessTokenAsync_with_unauthenticated_identity_should_retur
4342

4443
result.Succeeded.ShouldBeFalse();
4544
result.FailedResult.ShouldNotBeNull();
46-
result.FailedResult.Error.ShouldBe("No active user");
45+
result.FailedResult!.Error.ShouldBe("No active user");
4746
}
4847

4948
private static UserAccessAccessTokenManager CreateSut()

0 commit comments

Comments
 (0)