From ff3a55385bd806fa7e8e4b208e7eaf6cc1ac683c Mon Sep 17 00:00:00 2001 From: jrmccannon Date: Thu, 20 Mar 2025 10:29:11 -0500 Subject: [PATCH] Had the wrong type in the SutProvider. --- .../RestoreOrganizationUserCommandTests.cs | 37 +++++++++---------- 1 file changed, 18 insertions(+), 19 deletions(-) diff --git a/test/Core.Test/AdminConsole/OrganizationFeatures/OrganizationUsers/RestoreOrganizationUserCommandTests.cs b/test/Core.Test/AdminConsole/OrganizationFeatures/OrganizationUsers/RestoreOrganizationUserCommandTests.cs index 484bd0dd91..771d763095 100644 --- a/test/Core.Test/AdminConsole/OrganizationFeatures/OrganizationUsers/RestoreOrganizationUserCommandTests.cs +++ b/test/Core.Test/AdminConsole/OrganizationFeatures/OrganizationUsers/RestoreOrganizationUserCommandTests.cs @@ -23,12 +23,11 @@ namespace Bit.Core.Test.AdminConsole.OrganizationFeatures.OrganizationUsers; [SutProviderCustomize] public class RestoreOrganizationUserCommandTests { - [Theory, BitAutoData] public async Task RestoreUser_Success(Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { RestoreUser_Setup(organization, owner, organizationUser, sutProvider); @@ -46,7 +45,7 @@ public class RestoreOrganizationUserCommandTests public async Task RestoreUser_WithPushSyncOrgKeysOnRevokeRestoreEnabled_Success(Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { RestoreUser_Setup(organization, owner, organizationUser, sutProvider); @@ -71,7 +70,7 @@ public class RestoreOrganizationUserCommandTests public async Task RestoreUser_WithEventSystemUser_Success(Organization organization, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, EventSystemUser eventSystemUser, - SutProvider sutProvider) + SutProvider sutProvider) { RestoreUser_Setup(organization, null, organizationUser, sutProvider); @@ -90,7 +89,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, EventSystemUser eventSystemUser, - SutProvider sutProvider) + SutProvider sutProvider) { RestoreUser_Setup(organization, null, organizationUser, sutProvider); @@ -115,7 +114,7 @@ public class RestoreOrganizationUserCommandTests public async Task RestoreUser_RestoreThemselves_Fails(Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.UserId = owner.Id; RestoreUser_Setup(organization, owner, organizationUser, sutProvider); @@ -144,7 +143,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed)] OrganizationUser restoringUser, [OrganizationUser(OrganizationUserStatusType.Revoked, OrganizationUserType.Owner)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { restoringUser.Type = restoringUserType; RestoreUser_Setup(organization, restoringUser, organizationUser, sutProvider); @@ -174,7 +173,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Status = userStatus; RestoreUser_Setup(organization, owner, organizationUser, sutProvider); @@ -201,7 +200,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -240,7 +239,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; @@ -289,7 +288,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -326,7 +325,7 @@ public class RestoreOrganizationUserCommandTests [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, [OrganizationUser(OrganizationUserStatusType.Accepted)] OrganizationUser secondOrganizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -377,7 +376,7 @@ public class RestoreOrganizationUserCommandTests [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, [OrganizationUser(OrganizationUserStatusType.Accepted)] OrganizationUser secondOrganizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -422,7 +421,7 @@ public class RestoreOrganizationUserCommandTests [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, [OrganizationUser(OrganizationUserStatusType.Accepted)] OrganizationUser secondOrganizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -487,7 +486,7 @@ public class RestoreOrganizationUserCommandTests OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; @@ -532,7 +531,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser organizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { organizationUser.Email = null; // this is required to mock that the user as had already been confirmed before the revoke @@ -569,7 +568,7 @@ public class RestoreOrganizationUserCommandTests [OrganizationUser(OrganizationUserStatusType.Confirmed, OrganizationUserType.Owner)] OrganizationUser owner, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser orgUser1, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser orgUser2, - SutProvider sutProvider) + SutProvider sutProvider) { // Arrange RestoreUser_Setup(organization, owner, orgUser1, sutProvider); @@ -617,7 +616,7 @@ public class RestoreOrganizationUserCommandTests [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser orgUser1, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser orgUser2, [OrganizationUser(OrganizationUserStatusType.Revoked)] OrganizationUser orgUser3, - SutProvider sutProvider) + SutProvider sutProvider) { // Arrange RestoreUser_Setup(organization, owner, orgUser1, sutProvider); @@ -681,7 +680,7 @@ public class RestoreOrganizationUserCommandTests Organization organization, OrganizationUser? requestingOrganizationUser, OrganizationUser targetOrganizationUser, - SutProvider sutProvider) + SutProvider sutProvider) { if (requestingOrganizationUser != null) {