Skip to content

Commit e378d5e

Browse files
christiannagelchristiannagel
authored
Merge pull request #332 from CNinnovation/copilot/fix-3
Library for users to create anonymous accounts
2 parents 6dab836 + 20086d3 commit e378d5e

14 files changed

Lines changed: 984 additions & 0 deletions

‎src/Codebreaker.Identity.sln‎

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
2+
Microsoft Visual Studio Solution File, Format Version 12.00
3+
# Visual Studio Version 17
4+
VisualStudioVersion = 17.14.36121.58
5+
MinimumVisualStudioVersion = 10.0.40219.1
6+
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Codebreaker.Identity", "services\identity\Codebreaker.Identity\Codebreaker.Identity.csproj", "{0F0F9423-3793-5157-B688-D616FC508021}"
7+
EndProject
8+
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Codebreaker.Identity.Tests", "services\identity\Codebreaker.Identity.Tests\Codebreaker.Identity.Tests.csproj", "{908DAF3F-1A04-E26E-CE74-87B1C27E47AC}"
9+
EndProject
10+
Global
11+
GlobalSection(SolutionConfigurationPlatforms) = preSolution
12+
Debug|Any CPU = Debug|Any CPU
13+
Release|Any CPU = Release|Any CPU
14+
EndGlobalSection
15+
GlobalSection(ProjectConfigurationPlatforms) = postSolution
16+
{0F0F9423-3793-5157-B688-D616FC508021}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
17+
{0F0F9423-3793-5157-B688-D616FC508021}.Debug|Any CPU.Build.0 = Debug|Any CPU
18+
{0F0F9423-3793-5157-B688-D616FC508021}.Release|Any CPU.ActiveCfg = Release|Any CPU
19+
{0F0F9423-3793-5157-B688-D616FC508021}.Release|Any CPU.Build.0 = Release|Any CPU
20+
{908DAF3F-1A04-E26E-CE74-87B1C27E47AC}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
21+
{908DAF3F-1A04-E26E-CE74-87B1C27E47AC}.Debug|Any CPU.Build.0 = Debug|Any CPU
22+
{908DAF3F-1A04-E26E-CE74-87B1C27E47AC}.Release|Any CPU.ActiveCfg = Release|Any CPU
23+
{908DAF3F-1A04-E26E-CE74-87B1C27E47AC}.Release|Any CPU.Build.0 = Release|Any CPU
24+
EndGlobalSection
25+
GlobalSection(SolutionProperties) = preSolution
26+
HideSolutionNode = FALSE
27+
EndGlobalSection
28+
GlobalSection(ExtensibilityGlobals) = postSolution
29+
SolutionGuid = {5AABC12D-CB55-4156-BFE7-95FDD686A192}
30+
EndGlobalSection
31+
EndGlobal

‎src/services/identity/Codebreaker.Identity.Tests/AnonymousUserServiceTests.cs‎

Lines changed: 104 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,104 @@
1+
using Codebreaker.Identity.Configuration;
2+
using Codebreaker.Identity.Models;
3+
using Codebreaker.Identity.Services;
4+
using Microsoft.Extensions.Logging;
5+
using Microsoft.Extensions.Options;
6+
using Microsoft.Graph;
7+
using Microsoft.Graph.Models;
8+
using Moq;
9+
10+
namespace Codebreaker.Identity.Tests;
11+
12+
public class AnonymousUserServiceTests
13+
{
14+
private readonly Mock<IOptions<AnonymousUserOptions>> _mockOptions;
15+
private readonly Mock<ILogger<GraphAnonymousUserService>> _mockLogger;
16+
17+
public AnonymousUserServiceTests()
18+
{
19+
_mockOptions = new Mock<IOptions<AnonymousUserOptions>>();
20+
_mockOptions.Setup(o => o.Value).Returns(new AnonymousUserOptions
21+
{
22+
TenantId = "test-tenant",
23+
ClientId = "test-client",
24+
ClientSecret = "test-secret",
25+
Domain = "example.com",
26+
PasswordLength = 12,
27+
UserNamePrefix = "anon"
28+
});
29+
30+
_mockLogger = new Mock<ILogger<GraphAnonymousUserService>>();
31+
}
32+
33+
//[Fact(Skip = "Requires mocking of GraphServiceClient which is challenging")]
34+
//public async Task CreateAnonUser_ShouldCreateAnonymousUser()
35+
//{
36+
// // This test would require extensive mocking of GraphServiceClient
37+
// // In a real implementation, we would use a test double or wrapper for GraphServiceClient
38+
//}
39+
40+
//[Fact(Skip = "Requires mocking of GraphServiceClient which is challenging")]
41+
//public async Task DeleteAnonUsers_ShouldDeleteStaleAnonymousUsers()
42+
//{
43+
// // This test would require extensive mocking of GraphServiceClient
44+
// // In a real implementation, we would use a test double or wrapper for GraphServiceClient
45+
//}
46+
47+
//[Fact(Skip = "Requires mocking of GraphServiceClient which is challenging")]
48+
//public async Task PromoteAnonUser_ShouldPromoteAnonymousUser()
49+
//{
50+
// // This test would require extensive mocking of GraphServiceClient
51+
// // In a real implementation, we would use a test double or wrapper for GraphServiceClient
52+
//}
53+
54+
[Fact]
55+
public void ServiceInitialization_WithValidOptions_ShouldNotThrow()
56+
{
57+
// Arrange & Act & Assert
58+
var exception = Record.Exception(() => new GraphAnonymousUserServiceWrapper(
59+
_mockOptions.Object,
60+
_mockLogger.Object));
61+
62+
Assert.Null(exception);
63+
}
64+
65+
[Fact]
66+
public void ServiceInitialization_WithNullOptions_ShouldThrow()
67+
{
68+
// Arrange
69+
IOptions<AnonymousUserOptions>? nullOptions = null;
70+
71+
// Act & Assert
72+
Assert.Throws<ArgumentNullException>(() => new GraphAnonymousUserServiceWrapper(
73+
nullOptions!,
74+
_mockLogger.Object));
75+
}
76+
77+
[Fact]
78+
public void ServiceInitialization_WithNullLogger_ShouldThrow()
79+
{
80+
// Arrange
81+
ILogger<GraphAnonymousUserService>? nullLogger = null;
82+
83+
// Act & Assert
84+
Assert.Throws<ArgumentNullException>(() => new GraphAnonymousUserServiceWrapper(
85+
_mockOptions.Object,
86+
nullLogger!));
87+
}
88+
89+
// A test wrapper for GraphAnonymousUserService that allows testing without actually using Graph API
90+
private class GraphAnonymousUserServiceWrapper(
91+
IOptions<AnonymousUserOptions> options,
92+
ILogger<GraphAnonymousUserService> logger) : GraphAnonymousUserService(options, logger)
93+
{
94+
// We don't test the actual Graph API methods here
95+
public new Task<AnonymousUser> CreateAnonUser(string userName) =>
96+
Task.FromResult(new AnonymousUser());
97+
98+
public new Task<int> DeleteAnonUsers() =>
99+
Task.FromResult(0);
100+
101+
public new Task<AnonymousUser> PromoteAnonUser(string anonymousUserId, string email, string displayName, string password) =>
102+
Task.FromResult(new AnonymousUser());
103+
}
104+
}

‎src/services/identity/Codebreaker.Identity.Tests/Codebreaker.Identity.Tests.csproj‎

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
<Project Sdk="Microsoft.NET.Sdk">
2+
3+
<PropertyGroup>
4+
<TargetFramework>net8.0</TargetFramework>
5+
<ImplicitUsings>enable</ImplicitUsings>
6+
<Nullable>enable</Nullable>
7+
8+
<IsPackable>false</IsPackable>
9+
<IsTestProject>true</IsTestProject>
10+
</PropertyGroup>
11+
12+
<ItemGroup>
13+
<PackageReference Include="Microsoft.NET.Test.Sdk" />
14+
<PackageReference Include="xunit" />
15+
<PackageReference Include="xunit.runner.visualstudio">
16+
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
17+
<PrivateAssets>all</PrivateAssets>
18+
</PackageReference>
19+
<PackageReference Include="coverlet.collector">
20+
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
21+
<PrivateAssets>all</PrivateAssets>
22+
</PackageReference>
23+
<PackageReference Include="Moq" />
24+
</ItemGroup>
25+
26+
<ItemGroup>
27+
<ProjectReference Include="..\Codebreaker.Identity\Codebreaker.Identity.csproj" />
28+
</ItemGroup>
29+
30+
</Project>

‎src/services/identity/Codebreaker.Identity.Tests/GlobalUsings.cs‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
global using Xunit;

‎src/services/identity/Codebreaker.Identity.Tests/MockAnonymousUserService.cs‎

Lines changed: 88 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,88 @@
1+
using Codebreaker.Identity.Models;
2+
using Codebreaker.Identity.Services;
3+
using Microsoft.Extensions.Logging;
4+
5+
namespace Codebreaker.Identity.Tests;
6+
7+
/// <summary>
8+
/// A mock implementation of <see cref="IAnonymousUserService"/> for testing
9+
/// </summary>
10+
public class MockAnonymousUserService : IAnonymousUserService
11+
{
12+
private readonly ILogger<MockAnonymousUserService> _logger;
13+
private readonly List<AnonymousUser> _users = [];
14+
15+
/// <summary>
16+
/// Initializes a new instance of the <see cref="MockAnonymousUserService"/> class
17+
/// </summary>
18+
/// <param name="logger">The logger</param>
19+
public MockAnonymousUserService(ILogger<MockAnonymousUserService> logger)
20+
{
21+
_logger = logger ?? throw new ArgumentNullException(nameof(logger));
22+
}
23+
24+
/// <inheritdoc />
25+
public Task<AnonymousUser> CreateAnonUser(string userName)
26+
{
27+
_logger.LogInformation("Creating mock anonymous user: {UserName}", userName);
28+
29+
var user = new AnonymousUser
30+
{
31+
Id = Guid.NewGuid().ToString(),
32+
UserName = userName,
33+
DisplayName = string.IsNullOrWhiteSpace(userName) ? "Anonymous User" : userName,
34+
Email = $"anon-{Guid.NewGuid()}@example.com",
35+
Password = $"Password_{Guid.NewGuid().ToString().Substring(0, 8)}",
36+
CreatedAt = DateTimeOffset.UtcNow
37+
};
38+
39+
_users.Add(user);
40+
return Task.FromResult(user);
41+
}
42+
43+
/// <inheritdoc />
44+
public Task<int> DeleteAnonUsers()
45+
{
46+
_logger.LogInformation("Deleting stale anonymous users");
47+
48+
// Delete users older than 3 months
49+
DateTimeOffset cutoffDate = DateTimeOffset.UtcNow.AddMonths(-3);
50+
int count = _users.RemoveAll(u =>
51+
(u.LastLoginAt == null && u.CreatedAt < cutoffDate) ||
52+
(u.LastLoginAt != null && u.LastLoginAt < cutoffDate));
53+
54+
_logger.LogInformation("Deleted {Count} stale anonymous users", count);
55+
return Task.FromResult(count);
56+
}
57+
58+
/// <inheritdoc />
59+
public Task<AnonymousUser> PromoteAnonUser(string anonymousUserId, string email, string displayName, string password)
60+
{
61+
_logger.LogInformation("Promoting mock anonymous user: {UserId}", anonymousUserId);
62+
63+
var user = _users.FirstOrDefault(u => u.Id == anonymousUserId);
64+
if (user == null)
65+
{
66+
throw new InvalidOperationException($"User with ID {anonymousUserId} not found");
67+
}
68+
69+
// Update the user properties
70+
user.Email = email;
71+
user.DisplayName = displayName;
72+
user.Password = password;
73+
user.LastLoginAt = DateTimeOffset.UtcNow;
74+
75+
return Task.FromResult(user);
76+
}
77+
78+
/// <summary>
79+
/// Gets the current anonymous users
80+
/// </summary>
81+
/// <returns>The list of anonymous users</returns>
82+
public IReadOnlyList<AnonymousUser> GetUsers() => _users.AsReadOnly();
83+
84+
/// <summary>
85+
/// Clears all users
86+
/// </summary>
87+
public void ClearUsers() => _users.Clear();
88+
}

0 commit comments

Comments
 (0)