From 237e26da1e6b45697f3f65c11c868ba142767c70 Mon Sep 17 00:00:00 2001
From: Shannon <sdeminick@gmail.com>
Date: Mon, 12 Apr 2021 18:02:43 +1000
Subject: [PATCH] fix tests

---
 .../Security/MemberSignInManagerTests.cs      | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/src/Umbraco.Tests.UnitTests/Umbraco.Web.Common/Security/MemberSignInManagerTests.cs b/src/Umbraco.Tests.UnitTests/Umbraco.Web.Common/Security/MemberSignInManagerTests.cs
index b1317acb95..c5d24e6bd9 100644
--- a/src/Umbraco.Tests.UnitTests/Umbraco.Web.Common/Security/MemberSignInManagerTests.cs
+++ b/src/Umbraco.Tests.UnitTests/Umbraco.Web.Common/Security/MemberSignInManagerTests.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Linq;
 using System.Net;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Authentication;
@@ -20,7 +21,7 @@ namespace Umbraco.Cms.Tests.UnitTests.Umbraco.Web.Common.Security
     public class MemberSignInManagerTests
     {
         private Mock<ILogger<SignInManager<MemberIdentityUser>>> _mockLogger;
-        private readonly Mock<MemberManager> _memberManager = MockUserManager();
+        private readonly Mock<MemberManager> _memberManager = MockMemberManager();
 
         public UserClaimsPrincipalFactory<MemberIdentityUser> CreateClaimsFactory(MemberManager userMgr)
             => new UserClaimsPrincipalFactory<MemberIdentityUser>(userMgr, Options.Create(new MemberIdentityOptions()));
@@ -54,12 +55,18 @@ namespace Umbraco.Cms.Tests.UnitTests.Umbraco.Web.Common.Security
                     Mock.Of<IAuthenticationSchemeProvider>(),
                     Mock.Of<IUserConfirmation<MemberIdentityUser>>());
         }
-        private static Mock<MemberManager> MockUserManager()
-        {
-            var store = new Mock<IUserStore<MemberIdentityUser>>();
-            var mgr = new Mock<MemberManager>(store.Object, null, null, null, null, null, null, null, null);
-            return mgr;
-        }
+        private static Mock<MemberManager> MockMemberManager()
+            => new Mock<MemberManager>(
+                    Mock.Of<IIpResolver>(),
+                    Mock.Of<IUserStore<MemberIdentityUser>>(),
+                    Options.Create(new MemberIdentityOptions()),
+                    Mock.Of<IPasswordHasher<MemberIdentityUser>>(),
+                    Enumerable.Empty<IUserValidator<MemberIdentityUser>>(),
+                    Enumerable.Empty<IPasswordValidator<MemberIdentityUser>>(),
+                    new BackOfficeIdentityErrorDescriber(),
+                    Mock.Of<IServiceProvider>(),
+                    Mock.Of<ILogger<UserManager<MemberIdentityUser>>>(),
+                    Options.Create(new Cms.Core.Configuration.Models.MemberPasswordConfigurationSettings()));
 
         [Test]
         public async Task WhenPasswordSignInAsyncIsCalled_AndEverythingIsSetup_ThenASignInResultSucceededShouldBeReturnedAsync()