diff --git a/src/Api/Models/Request/GroupRequestModel.cs b/src/Api/Models/Request/GroupRequestModel.cs index 6fcb34825a..aa52a08af2 100644 --- a/src/Api/Models/Request/GroupRequestModel.cs +++ b/src/Api/Models/Request/GroupRequestModel.cs @@ -10,8 +10,6 @@ public class GroupRequestModel public string Name { get; set; } [Required] public bool? AccessAll { get; set; } - [StringLength(300)] - public string ExternalId { get; set; } public IEnumerable Collections { get; set; } public IEnumerable Users { get; set; } @@ -27,7 +25,6 @@ public class GroupRequestModel { existingGroup.Name = Name; existingGroup.AccessAll = AccessAll.Value; - existingGroup.ExternalId = ExternalId; return existingGroup; } } diff --git a/test/Api.Test/Controllers/GroupsControllerTests.cs b/test/Api.Test/Controllers/GroupsControllerTests.cs index 5d700b4e36..b7f3b0cd36 100644 --- a/test/Api.Test/Controllers/GroupsControllerTests.cs +++ b/test/Api.Test/Controllers/GroupsControllerTests.cs @@ -29,7 +29,7 @@ public class GroupsControllerTests await sutProvider.GetDependency().Received(1).CreateGroupAsync( Arg.Is(g => g.OrganizationId == organization.Id && g.Name == groupRequestModel.Name && - g.AccessAll == groupRequestModel.AccessAll && g.ExternalId == groupRequestModel.ExternalId), + g.AccessAll == groupRequestModel.AccessAll), organization, Arg.Any>(), Arg.Any>()); @@ -38,7 +38,6 @@ public class GroupsControllerTests Assert.Equal(groupRequestModel.Name, response.Name); Assert.Equal(organization.Id.ToString(), response.OrganizationId); Assert.Equal(groupRequestModel.AccessAll, response.AccessAll); - Assert.Equal(groupRequestModel.ExternalId, response.ExternalId); } [Theory] @@ -57,7 +56,7 @@ public class GroupsControllerTests await sutProvider.GetDependency().Received(1).UpdateGroupAsync( Arg.Is(g => g.OrganizationId == organization.Id && g.Name == groupRequestModel.Name && - g.AccessAll == groupRequestModel.AccessAll && g.ExternalId == groupRequestModel.ExternalId), + g.AccessAll == groupRequestModel.AccessAll), Arg.Is(o => o.Id == organization.Id), Arg.Any>(), Arg.Any>()); @@ -66,6 +65,5 @@ public class GroupsControllerTests Assert.Equal(groupRequestModel.Name, response.Name); Assert.Equal(organization.Id.ToString(), response.OrganizationId); Assert.Equal(groupRequestModel.AccessAll, response.AccessAll); - Assert.Equal(groupRequestModel.ExternalId, response.ExternalId); } }