mirror of
https://github.com/bitwarden/server.git
synced 2024-11-23 12:25:16 +01:00
0a01051d83
* [EC-507] SCIM CQRS Refactor - Groups/Put (#2269) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-531] Implemented CQRS for Groups Put and added unit tests * [EC-507] Created ScimServiceCollectionExtensions * [EC-507] Renamed AddScimCommands to AddScimGroupCommands * [EC-507] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Update PutGroupCommand to return Group PutGroupCommand returns Group and GroupsController creates ScimGroupResponseModel response * [EC-507] Remove Queries/Commands folders from Scim and Scim.Tests * [EC-507] Remove unneeded check on empty provided memberIds * [EC-507] SCIM CQRS Refactor - Groups/GetList (#2272) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-508] Implemented CQRS for Groups GetList and added unit tests * [EC-507] Created ScimServiceCollectionExtensions and renamed GetGroupsListCommand to GetGroupsListQuery * [EC-507] Renamed AddScimCommands to AddScimGroupQueries * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Remove 'Queries' folder from Scim and Scim.Test * [EC-507] Move ScimListResponseModel from GetGroupsListQuery to Scim.GroupsController * [EC-507] Remove asserts on IGroupRepository.GetManyByOrganizationIdAsync from unit tests * [EC-507] SCIM CQRS Refactor - Groups/Get (#2271) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-507] Implemented CQRS for Groups Get and added unit tests * [EC-507] Created ScimServiceCollectionExtensions and renamed GetGroupCommand to GetGroupQuery * [EC-507] Renamed AddScimCommands to AddScimGroupQueries * [EC-507] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Sorted order of methods * [EC-507] Removed GetGroupQuery and moved logic to controller * [EC-507] Remove 'Queries' folder from Scim and Scim.Test * [EC-507] SCIM CQRS Refactor - Groups/Patch (#2268) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-532] Implemented CQRS for Groups Patch and added unit tests * [EC-507] Created ScimServiceCollectionExtensions * [EC-507] Renamed AddScimCommands to AddScimGroupCommands * [EC-507] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Remove Queries/Commands folders from Scim and Scim.Tests * [EC-507] Assert group.Name after saving. Assert userIds saved. * [EC-508] SCIM CQRS Refactor - Users/Delete (#2261) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-539] Implemented CQRS for Users Delete and added unit tests * [EC-508] Created ScimServiceCollectionExtensions * [EC-508] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-508] Removed unneeded model from DeleteUserCommand. Removed unneeded dependencies from UsersController * [EC-508] Removed Bit.Scim.Models dependency from DeleteUserCommandTests * [EC-508] Deleted 'DeleteUserCommand' from SCIM; Created commands on Core 'DeleteOrganizationUserCommand', 'PushDeleteUserRegistrationOrganizationCommand' and 'OrganizationHasConfirmedOwnersExceptQuery' * [EC-508] Changed DeleteOrganizationUserCommand back to using IOrganizationService * [EC-508] Fixed DeleteOrganizationUserCommand unit tests * [EC-508] Remove unneeded obsolete comments. Update DeleteUserAsync Obsolete comment with ticket reference * [EC-508] Move DeleteOrganizationUserCommand to OrganizationFeatures folder * [EC-508] SCIM CQRS Refactor - Users/Post (#2264) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-536] Implemented CQRS for Users Post and added unit tests * [EC-508] Created ScimServiceCollectionExtensions * [EC-508] Renamed AddScimCommands to AddScimUserCommands * [EC-508] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-508] Catching NotFoundException on ExceptionHandlerFilter * [EC-508] Remove Queries/Commands folders from Scim and Scim.Tests * [EC-508] SCIM CQRS Refactor - Users/Patch (#2262) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-538] Implemented CQRS for Users Patch and added unit tests * [EC-508] Added ScimServiceCollectionExtensions * [EC-508] Removed HandleActiveOperationAsync method from UsersController * [EC-508] Renamed AddScimCommands to AddScimUserCommands * [EC-508] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-508] Removed unneeded dependencies from UsersController * [EC-508] Remove 'Query' folder from Scim and Scim.Test * [EC-507] SCIM CQRS Refactor - Groups/Post (#2270) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-530] Implemented CQRS for Groups Post and added unit tests * [EC-507] Created ScimServiceCollectionExtensions * [EC-507] Renamed AddScimCommands to AddScimGroupCommands * [EC-507] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Remove Queries/Commands folders from Scim and Scim.Test * [EC-507] Remove unneeded skipIfEmpty argument. Updated unit test to check provided userIds * [EC-507] Remove UpdateGroupMembersAsync from GroupsController * [EC-508] SCIM CQRS Refactor - Users/GetList (#2265) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-535] Implemented CQRS for Users GetList and added unit tests * [EC-508] Created ScimServiceCollectionExtensions and renamed GetUsersListCommand to GetUsersListQuery * [EC-508] Renamed AddScimCommands to AddScimUserQueries * [EC-508] Removed unneeded IUserRepository and IOptions<ScimSettings> from UsersController * [EC-508] Sorted UsersController properties and dependencies * [EC-508] Remove 'Queries' folder from Scim and Scim.Test * [EC-508] Move ScimListResponseModel creation to Scim.UsersController * [EC-508] Move ScimUserResponseModel creation to Scim.UsersController Co-authored-by: Thomas Rittson <trittson@bitwarden.com> * [EC-507] SCIM CQRS Refactor - Groups/Delete (#2267) * [EC-390] Added Scim.Test unit tests project * [EC-390] Added ConflictException type. Updated BadRequestException to have parameterless constructor. Updated NotFoundException to have constructor with a message parameter * [EC-533] Implemented CQRS for Groups Delete and added unit tests * [EC-507] Created ScimServiceCollectionExtensions * [EC-507] Renamed AddScimCommands to AddScimGroupCommands * [EC-507] Created ExceptionHandlerFilterAttribute on SCIM project * [EC-507] Removed unneeded dependencies from GroupsController * [EC-507] Move DeleteGroupCommand to OrganizationFeatures/OrganizationUsers * [EC-507] Remove IGetUserQuery and move logic to UsersController. Remove unused references. * [EC-507] Move IDeleteGroupCommand to Groups folder Co-authored-by: Thomas Rittson <trittson@bitwarden.com>
130 lines
4.8 KiB
C#
130 lines
4.8 KiB
C#
using Bit.Core.Entities;
|
|
using Bit.Core.Repositories;
|
|
using Bit.Scim.Groups;
|
|
using Bit.Test.Common.AutoFixture;
|
|
using Bit.Test.Common.AutoFixture.Attributes;
|
|
using Bit.Test.Common.Helpers;
|
|
using NSubstitute;
|
|
using Xunit;
|
|
|
|
namespace Bit.Scim.Test.Groups;
|
|
|
|
[SutProviderCustomize]
|
|
public class GetGroupsListCommandTests
|
|
{
|
|
[Theory]
|
|
[BitAutoData(10, 1)]
|
|
[BitAutoData(2, 1)]
|
|
[BitAutoData(1, 3)]
|
|
public async Task GetGroupsList_Success(int count, int startIndex, SutProvider<GetGroupsListQuery> sutProvider, Guid organizationId, IList<Group> groups)
|
|
{
|
|
groups = SetGroupsOrganizationId(groups, organizationId);
|
|
|
|
sutProvider.GetDependency<IGroupRepository>()
|
|
.GetManyByOrganizationIdAsync(organizationId)
|
|
.Returns(groups);
|
|
|
|
var result = await sutProvider.Sut.GetGroupsListAsync(organizationId, null, count, startIndex);
|
|
|
|
AssertHelper.AssertPropertyEqual(groups.Skip(startIndex - 1).Take(count).ToList(), result.groupList);
|
|
AssertHelper.AssertPropertyEqual(groups.Count, result.totalResults);
|
|
}
|
|
|
|
[Theory]
|
|
[BitAutoData]
|
|
public async Task GetGroupsList_FilterDisplayName_Success(SutProvider<GetGroupsListQuery> sutProvider, Guid organizationId, IList<Group> groups)
|
|
{
|
|
groups = SetGroupsOrganizationId(groups, organizationId);
|
|
string name = groups.First().Name;
|
|
string filter = $"displayName eq {name}";
|
|
|
|
var expectedGroupList = groups
|
|
.Where(g => g.Name == name)
|
|
.ToList();
|
|
var expectedTotalResults = expectedGroupList.Count;
|
|
|
|
sutProvider.GetDependency<IGroupRepository>()
|
|
.GetManyByOrganizationIdAsync(organizationId)
|
|
.Returns(groups);
|
|
|
|
var result = await sutProvider.Sut.GetGroupsListAsync(organizationId, filter, null, null);
|
|
|
|
AssertHelper.AssertPropertyEqual(expectedGroupList, result.groupList);
|
|
AssertHelper.AssertPropertyEqual(expectedTotalResults, result.totalResults);
|
|
}
|
|
|
|
[Theory]
|
|
[BitAutoData]
|
|
public async Task GetGroupsList_FilterDisplayName_Empty(string name, SutProvider<GetGroupsListQuery> sutProvider, Guid organizationId, IList<Group> groups)
|
|
{
|
|
groups = SetGroupsOrganizationId(groups, organizationId);
|
|
string filter = $"displayName eq {name}";
|
|
|
|
var expectedGroupList = new List<Group>();
|
|
var expectedTotalResults = expectedGroupList.Count;
|
|
|
|
sutProvider.GetDependency<IGroupRepository>()
|
|
.GetManyByOrganizationIdAsync(organizationId)
|
|
.Returns(groups);
|
|
|
|
var result = await sutProvider.Sut.GetGroupsListAsync(organizationId, filter, null, null);
|
|
|
|
AssertHelper.AssertPropertyEqual(expectedGroupList, result.groupList);
|
|
AssertHelper.AssertPropertyEqual(expectedTotalResults, result.totalResults);
|
|
}
|
|
|
|
[Theory]
|
|
[BitAutoData]
|
|
public async Task GetGroupsList_FilterExternalId_Success(SutProvider<GetGroupsListQuery> sutProvider, Guid organizationId, IList<Group> groups)
|
|
{
|
|
groups = SetGroupsOrganizationId(groups, organizationId);
|
|
string externalId = groups.First().ExternalId;
|
|
string filter = $"externalId eq {externalId}";
|
|
|
|
var expectedGroupList = groups
|
|
.Where(ou => ou.ExternalId == externalId)
|
|
.ToList();
|
|
var expectedTotalResults = expectedGroupList.Count;
|
|
|
|
sutProvider.GetDependency<IGroupRepository>()
|
|
.GetManyByOrganizationIdAsync(organizationId)
|
|
.Returns(groups);
|
|
|
|
var result = await sutProvider.Sut.GetGroupsListAsync(organizationId, filter, null, null);
|
|
|
|
AssertHelper.AssertPropertyEqual(expectedGroupList, result.groupList);
|
|
AssertHelper.AssertPropertyEqual(expectedTotalResults, result.totalResults);
|
|
}
|
|
|
|
[Theory]
|
|
[BitAutoData]
|
|
public async Task GetGroupsList_FilterExternalId_Empty(string externalId, SutProvider<GetGroupsListQuery> sutProvider, Guid organizationId, IList<Group> groups)
|
|
{
|
|
groups = SetGroupsOrganizationId(groups, organizationId);
|
|
string filter = $"externalId eq {externalId}";
|
|
|
|
var expectedGroupList = groups
|
|
.Where(ou => ou.ExternalId == externalId)
|
|
.ToList();
|
|
var expectedTotalResults = expectedGroupList.Count;
|
|
|
|
sutProvider.GetDependency<IGroupRepository>()
|
|
.GetManyByOrganizationIdAsync(organizationId)
|
|
.Returns(groups);
|
|
|
|
var result = await sutProvider.Sut.GetGroupsListAsync(organizationId, filter, null, null);
|
|
|
|
AssertHelper.AssertPropertyEqual(expectedGroupList, result.groupList);
|
|
AssertHelper.AssertPropertyEqual(expectedTotalResults, result.totalResults);
|
|
}
|
|
|
|
private IList<Group> SetGroupsOrganizationId(IList<Group> groups, Guid organizationId)
|
|
{
|
|
return groups.Select(g =>
|
|
{
|
|
g.OrganizationId = organizationId;
|
|
return g;
|
|
}).ToList();
|
|
}
|
|
}
|