Don't crash when user records can't be saved

Author: linkdotnet

LinkDotNet.Blog.UnitTests/Web/Shared/UserRecordServiceTests.cs

@@ -7,6 +7,7 @@
 using LinkDotNet.Blog.Web.Shared;
 using LinkDotNet.Domain;
 using LinkDotNet.Infrastructure.Persistence;
+using Microsoft.Extensions.Logging;
 using Moq;
 using Xunit;
 
@@ -30,7 +31,8 @@ public UserRecordServiceTests()
                 repositoryMock.Object,
                 fakeNavigationManager,
                 fakeAuthenticationStateProvider,
-                localStorageService.Object);
+                localStorageService.Object,
+                new Mock<ILogger>().Object);
         }
 
         [Fact]
@@ -78,5 +80,15 @@ public async Task ShouldNotStoreForAdmin()
 
             repositoryMock.Verify(r => r.StoreAsync(It.IsAny<UserRecord>()), Times.Never);
         }
+
+        [Fact]
+        public async Task ShouldNotThrowExceptionToOutsideWorld()
+        {
+            localStorageService.Setup(l => l.ContainKeyAsync("user", default)).Throws<Exception>();
+
+            Func<Task> act = () => sut.StoreUserRecordAsync();
+
+            await act.Should().NotThrowAsync<Exception>();
+        }
     }
 }

LinkDotNet.Blog.Web/Shared/UserRecordService.cs

@@ -5,6 +5,7 @@
 using LinkDotNet.Infrastructure.Persistence;
 using Microsoft.AspNetCore.Components;
 using Microsoft.AspNetCore.Components.Authorization;
+using Microsoft.Extensions.Logging;
 
 namespace LinkDotNet.Blog.Web.Shared
 {
@@ -19,20 +20,35 @@ public class UserRecordService : IUserRecordService
         private readonly NavigationManager navigationManager;
         private readonly AuthenticationStateProvider authenticationStateProvider;
         private readonly ILocalStorageService localStorageService;
+        private readonly ILogger logger;
 
         public UserRecordService(
             IRepository<UserRecord> userRecordRepository,
             NavigationManager navigationManager,
             AuthenticationStateProvider authenticationStateProvider,
-            ILocalStorageService localStorageService)
+            ILocalStorageService localStorageService,
+            ILogger logger)
         {
             this.userRecordRepository = userRecordRepository;
             this.navigationManager = navigationManager;
             this.authenticationStateProvider = authenticationStateProvider;
             this.localStorageService = localStorageService;
+            this.logger = logger;
         }
 
         public async Task StoreUserRecordAsync()
+        {
+            try
+            {
+                await GetAndStoreUSerRecordAsync();
+            }
+            catch (Exception e)
+            {
+                logger.Log(LogLevel.Error, e, "Couldn't write user record");
+            }
+        }
+
+        private async Task GetAndStoreUSerRecordAsync()
         {
             var userIdentity = (await authenticationStateProvider.GetAuthenticationStateAsync()).User.Identity;
             if (userIdentity == null || userIdentity.IsAuthenticated)