Consolidate helper method into extensions class

src/Api/AdminConsole/Authorization/HttpContextExtensions.cs

@@ -8,6 +8,9 @@ namespace Bit.Api.AdminConsole.Authorization;
 
 public static class HttpContextExtensions
 {
+    public const string NoOrgIdError =
+        "A route decorated with with '[Authorize<Requirement>]' should include a route value named 'orgId' either through the [Controller] attribute or through a '[Http*]' attribute.";
+
     /// <summary>
     /// Returns the result of the callback, caching it in HttpContext.Features for the lifetime of the request.
     /// Subsequent calls will retrieve the cached value.
@@ -53,8 +56,24 @@ public static class HttpContextExtensions
         this HttpContext httpContext,
         IProviderUserRepository providerUserRepository,
         Guid userId)
-        => await httpContext.WithFeaturesCacheAsync(async () =>
-            (await providerUserRepository.GetManyOrganizationDetailsByUserAsync(
-                userId, ProviderUserStatusType.Confirmed)).ToList());
+        => await httpContext.WithFeaturesCacheAsync(() =>
+            providerUserRepository.GetManyOrganizationDetailsByUserAsync(userId, ProviderUserStatusType.Confirmed));
 
+
+    /// <summary>
+    /// Parses the {orgId} route parameter into a Guid, or throws if the {orgId} is not present or not a valid guid.
+    /// </summary>
+    /// <param name="httpContext"></param>
+    /// <returns></returns>
+    /// <exception cref="InvalidOperationException"></exception>
+    public static Guid GetOrganizationId(this HttpContext httpContext)
+    {
+        httpContext.GetRouteData().Values.TryGetValue("orgId", out var orgIdParam);
+        if (orgIdParam == null || !Guid.TryParse(orgIdParam.ToString(), out var orgId))
+        {
+            throw new InvalidOperationException(NoOrgIdError);
+        }
+
+        return orgId;
+    }
 }

src/Api/AdminConsole/Authorization/OrganizationRequirementHelpers.cs

@@ -1,20 +0,0 @@
-#nullable enable
-
-namespace Bit.Api.AdminConsole.Authorization;
-
-public static class OrganizationRequirementHelpers
-{
-    public const string NoOrgIdError =
-        "A route decorated with with '[Authorize<Requirement>]' should include a route value named 'orgId' either through the [Controller] attribute or through a '[Http*]' attribute.";
-
-    public static Guid GetOrganizationId(this HttpContext httpContext)
-    {
-        httpContext.GetRouteData().Values.TryGetValue("orgId", out var orgIdParam);
-        if (orgIdParam == null || !Guid.TryParse(orgIdParam.ToString(), out var orgId))
-        {
-            throw new InvalidOperationException(NoOrgIdError);
-        }
-
-        return orgId;
-    }
-}

test/Api.Test/AdminConsole/Authorization/OrganizationRequirementHandlerTests.cs

@@ -17,13 +17,13 @@ public class OrganizationRequirementHandlerTests
     public async Task IfNoOrganizationId_Throws(SutProvider<OrganizationRequirementHandler> sutProvider)
     {
         // Arrange
-        ArrangeRouteAndUser(sutProvider, null); // no orgId in route
+        ArrangeRouteAndUser(sutProvider, null!); // no orgId in route
         var testRequirement = Substitute.For<IOrganizationRequirement>();
         var authContext = new AuthorizationHandlerContext([testRequirement], new ClaimsPrincipal(), null);
 
         // Act
         var exception = await Assert.ThrowsAsync<InvalidOperationException>(() => sutProvider.Sut.HandleAsync(authContext));
-        Assert.Equal(OrganizationRequirementHelpers.NoOrgIdError, exception.Message);
+        Assert.Equal(HttpContextExtensions.NoOrgIdError, exception.Message);
         Assert.False(authContext.HasSucceeded);
     }
 
@@ -37,7 +37,7 @@ public class OrganizationRequirementHandlerTests
 
         // Act
         var exception = await Assert.ThrowsAsync<InvalidOperationException>(() => sutProvider.Sut.HandleAsync(authContext));
-        Assert.Contains(OrganizationRequirementHelpers.NoOrgIdError, exception.Message);
+        Assert.Contains(HttpContextExtensions.NoOrgIdError, exception.Message);
         Assert.False(authContext.HasSucceeded);
     }