Skip to content

Commit fea2585

Browse files
committed
Log exceptions
1 parent 0e512b7 commit fea2585

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

src/LinkDotNet.Blog.Web/Features/Services/UserRecordService.cs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
using LinkDotNet.Blog.Infrastructure.Persistence;
55
using Microsoft.AspNetCore.Components;
66
using Microsoft.AspNetCore.Components.Authorization;
7+
using Microsoft.Extensions.Logging;
78

89
namespace LinkDotNet.Blog.Web.Features.Services;
910

@@ -13,17 +14,20 @@ public class UserRecordService : IUserRecordService
1314
private readonly NavigationManager navigationManager;
1415
private readonly AuthenticationStateProvider authenticationStateProvider;
1516
private readonly ILocalStorageService localStorageService;
17+
private readonly ILogger<UserRecordService> logger;
1618

1719
public UserRecordService(
1820
IRepository<UserRecord> userRecordRepository,
1921
NavigationManager navigationManager,
2022
AuthenticationStateProvider authenticationStateProvider,
21-
ILocalStorageService localStorageService)
23+
ILocalStorageService localStorageService,
24+
ILogger<UserRecordService> logger)
2225
{
2326
this.userRecordRepository = userRecordRepository;
2427
this.navigationManager = navigationManager;
2528
this.authenticationStateProvider = authenticationStateProvider;
2629
this.localStorageService = localStorageService;
30+
this.logger = logger;
2731
}
2832

2933
public async ValueTask StoreUserRecordAsync()
@@ -34,7 +38,7 @@ public async ValueTask StoreUserRecordAsync()
3438
}
3539
catch (Exception e)
3640
{
37-
Console.Write($"Exception: {e}");
41+
logger.LogError("Error while storing user record service: {Exception}", e);
3842
}
3943
}
4044

@@ -62,7 +66,7 @@ private async ValueTask GetAndStoreUserRecordAsync()
6266

6367
private async ValueTask<int> GetIdentifierHashAsync()
6468
{
65-
if (await TryGetKey())
69+
if (await HasKeyAsync())
6670
{
6771
var key = await localStorageService.GetItemAsync<Guid>("user");
6872
return key.GetHashCode();
@@ -73,7 +77,7 @@ private async ValueTask<int> GetIdentifierHashAsync()
7377
return id.GetHashCode();
7478
}
7579

76-
private async ValueTask<bool> TryGetKey()
80+
private async ValueTask<bool> HasKeyAsync()
7781
{
7882
try
7983
{

tests/LinkDotNet.Blog.UnitTests/Web/Features/Services/UserRecordServiceTests.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
using LinkDotNet.Blog.Domain;
44
using LinkDotNet.Blog.Infrastructure.Persistence;
55
using LinkDotNet.Blog.Web.Features.Services;
6+
using Microsoft.Extensions.Logging;
67

78
namespace LinkDotNet.Blog.UnitTests.Web.Features.Services;
89

@@ -24,7 +25,8 @@ public UserRecordServiceTests()
2425
repositoryMock.Object,
2526
fakeNavigationManager,
2627
fakeAuthenticationStateProvider,
27-
localStorageService.Object);
28+
localStorageService.Object,
29+
Mock.Of<ILogger<UserRecordService>>());
2830
}
2931

3032
[Fact]

0 commit comments

Comments
 (0)