Fix code smells and update dependencies

src/Turnierplan.App.Test.Functional/Scenarios.cs

@@ -14,7 +14,7 @@
 
 namespace Turnierplan.App.Test.Functional;
 
-public sealed class Scenarios
+public sealed class Scenarios : IDisposable
 {
     private readonly TestServer _testServer = new();
 
@@ -98,6 +98,11 @@ await userClient.Organizations.PostAsync(
         }
     }
 
+    public void Dispose()
+    {
+        _testServer.Dispose();
+    }
+
     private static async Task ExpectApiErrorAsync(Func<Task> func, HttpStatusCode code)
     {
         ApiException? exception = null;

src/Turnierplan.App.Test.Functional/TestServer.cs

@@ -14,9 +14,10 @@
 
 namespace Turnierplan.App.Test.Functional;
 
-internal sealed class TestServer
+internal sealed class TestServer : IDisposable
 {
     private readonly WebApplicationFactory<Program> _application;
+    private readonly List<HttpClientRequestAdapter> _httpClientRequestAdapters = [];
 
     public TestServer()
     {
@@ -59,6 +60,8 @@ public async Task<ApiRequestBuilder> CreateClientForUserAsync(string username, s
         var httpClientRequestAdapter = new HttpClientRequestAdapter(authenticationProvider, httpClient: httpClient);
         var client = new TurnierplanClient(httpClientRequestAdapter);
 
+        _httpClientRequestAdapters.Add(httpClientRequestAdapter);
+
         var loginResponse = await client.Api.Identity.Login.PostAsync(new LoginEndpointRequest
         {
             UserName = username,
@@ -81,4 +84,14 @@ public T ExecuteContextAction<T>(Func<TurnierplanContext, T> action)
         using var scope = _application.Services.CreateScope();
         return action(scope.ServiceProvider.GetRequiredService<TurnierplanContext>());
     }
+
+    public void Dispose()
+    {
+        foreach (var httpClientRequestAdapter in _httpClientRequestAdapters)
+        {
+            httpClientRequestAdapter.Dispose();
+        }
+
+        _application.Dispose();
+    }
 }

src/Turnierplan.App.Test.Unit/Helpers/RbacScopeHelperTest.cs

@@ -0,0 +1,56 @@
+using FluentAssertions;
+using Turnierplan.App.Helpers;
+using Turnierplan.Core.ApiKey;
+using Turnierplan.Core.Folder;
+using Turnierplan.Core.Image;
+using Turnierplan.Core.Organization;
+using Turnierplan.Core.PlanningRealm;
+using Turnierplan.Core.PublicId;
+using Turnierplan.Core.Tournament;
+using Turnierplan.Core.Venue;
+using Xunit;
+
+namespace Turnierplan.App.Test.Unit.Helpers;
+
+public sealed class RbacScopeHelperTest
+{
+    [Fact]
+    public void RbacScopeHelper___GetScopeId___Returns_Expected_Value()
+    {
+        var organization = new Organization("Test");
+        var tournament = new Tournament(organization, "Test", Visibility.Public);
+
+        var organizationId = organization.PublicId.ToString();
+        var tournamentId = tournament.PublicId.ToString();
+
+        organization.GetScopeId().Should().Be($"Organization:{organizationId}");
+        tournament.GetScopeId().Should().Be($"Tournament:{tournamentId}");
+    }
+
+    [Theory]
+    [InlineData("ApiKey:NobBmcA2jNc", nameof(ApiKey))]
+    [InlineData("Folder:NobBmcA2jNc", nameof(Folder))]
+    [InlineData("Image:NobBmcA2jNc", nameof(Image))]
+    [InlineData("Organization:NobBmcA2jNc", nameof(Organization))]
+    [InlineData("PlanningRealm:NobBmcA2jNc", nameof(PlanningRealm))]
+    [InlineData("Tournament:NobBmcA2jNc", nameof(Tournament))]
+    [InlineData("Venue:NobBmcA2jNc", nameof(Venue))]
+    public void RbacScopeHelper___TryParseScopeId___Works_As_Expected(string scopeId, string expectedTypeName)
+    {
+        RbacScopeHelper.TryParseScopeId(scopeId, out var actualTypeName, out var objectId).Should().BeTrue();
+
+        actualTypeName.Should().Be(expectedTypeName);
+        objectId.Should().Be(new PublicId("NobBmcA2jNc"));
+    }
+
+    [Theory]
+    [InlineData("InvitationLink:NobBmcA2jNc")]
+    [InlineData("ApiKey:NobBmcA2jN")]
+    public void RbacScopeHelper___TryParseScopeId_With_Invalid_Scope_Id___Works_As_Expected(string scopeId)
+    {
+        RbacScopeHelper.TryParseScopeId(scopeId, out var actualTypeName, out var objectId).Should().BeFalse();
+
+        actualTypeName.Should().BeNull();
+        objectId.Should().Be(PublicId.Empty);
+    }
+}