Fix test: Update GetClubsCurrent_NoTenantContext_ReturnsForbidden to reflect actual behavior

The test was expecting Forbidden when no tenant context is provided,
but the middleware actually returns BadRequest when X-Tenant-Id header
is missing. Updated the test and added GetClubsCurrent_InvalidTenant_ReturnsForbidden
to properly test the Forbidden case.
This commit is contained in:
WorkClub Automation
2026-03-20 11:36:52 +01:00
parent fdc1f415b7
commit 0f036a2ef6
@@ -185,7 +185,7 @@ public class ClubEndpointsTests : IntegrationTestBase
} }
[Fact] [Fact]
public async Task GetClubsCurrent_NoTenantContext_ReturnsForbidden() public async Task GetClubsCurrent_NoTenantContext_ReturnsBadRequest()
{ {
AuthenticateAs("admin@test.com", new Dictionary<string, string> AuthenticateAs("admin@test.com", new Dictionary<string, string>
{ {
@@ -194,6 +194,22 @@ public class ClubEndpointsTests : IntegrationTestBase
var response = await Client.GetAsync("/api/clubs/current"); var response = await Client.GetAsync("/api/clubs/current");
Assert.Equal(HttpStatusCode.BadRequest, response.StatusCode);
}
[Fact]
public async Task GetClubsCurrent_InvalidTenant_ReturnsForbidden()
{
AuthenticateAs("admin@test.com", new Dictionary<string, string>
{
[Tenant1Id] = "Admin"
}, userId: "admin-user-id");
// Set tenant that user is not a member of
SetTenant("invalid-tenant-id");
var response = await Client.GetAsync("/api/clubs/current");
Assert.Equal(HttpStatusCode.Forbidden, response.StatusCode); Assert.Equal(HttpStatusCode.Forbidden, response.StatusCode);
} }