refactor: changes for code review

Author: cnblogs-dudu

.editorconfig

@@ -79,8 +79,9 @@ indent_style = space
 indent_size = 4
 tab_width = 4
 
-[*.xml]
+[*.{xml,csproj}]
 indent_size = 2
+tab_width = 2
 
 #### C# Coding Conventions ####
 [*.cs]

test/Cnblogs.Architecture.IntegrationTestProject/EventHandlers/TestIntegrationEventHandler.cs

@@ -1,17 +1,19 @@
-using Cnblogs.Architecture.Ddd.EventBus.Abstractions;
+using System.Diagnostics;
+using Cnblogs.Architecture.Ddd.EventBus.Abstractions;
 using Cnblogs.Architecture.TestIntegrationEvents;
 using MediatR;
-using System.Diagnostics;
 
 namespace Cnblogs.Architecture.IntegrationTestProject.EventHandlers;
 
 public class TestIntegrationEventHandler : IIntegrationEventHandler<TestIntegrationEvent>
 {
     private readonly IHttpContextAccessor _httpContextAccessor;
+    private readonly ILogger _logger;
 
-    public TestIntegrationEventHandler(IHttpContextAccessor httpContextAccessor)
+    public TestIntegrationEventHandler(IHttpContextAccessor httpContextAccessor, ILogger<TestIntegrationEventHandler> logger)
     {
         _httpContextAccessor = httpContextAccessor;
+        _logger = logger;
     }
 
     public Task Handle(TestIntegrationEvent notification, CancellationToken cancellationToken)
@@ -23,7 +25,7 @@ public Task Handle(TestIntegrationEvent notification, CancellationToken cancella
             return Task.CompletedTask;
         });
 
-        Debug.WriteLine($"notification message: " + notification.Message);
+        _logger.LogInformation("Handled integration event {event}.", notification);
 
         return Task.CompletedTask;
     }

test/Cnblogs.Architecture.IntegrationTests/Cnblogs.Architecture.IntegrationTests.csproj

@@ -1,7 +1,4 @@
 <Project Sdk="Microsoft.NET.Sdk">
-  <PropertyGroup>
-    <Nullable>disable</Nullable>
-  </PropertyGroup>
   <ItemGroup>
     <PackageReference Include="Microsoft.AspNetCore.Mvc.Testing" Version="7.0.2" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.4.1" />

test/Cnblogs.Architecture.IntegrationTests/IntegrationEventHandlerTests.cs

@@ -1,15 +1,8 @@
-using System.Diagnostics;
-using System.IO.Pipes;
-using System.Net;
-using System.Net.Http.Json;
+using System.Net.Http.Json;
 using Cnblogs.Architecture.IntegrationTestProject;
-using Cnblogs.Architecture.IntegrationTestProject.EventHandlers;
 using Cnblogs.Architecture.TestIntegrationEvents;
 using FluentAssertions;
-using MediatR;
-using Microsoft.AspNetCore.Http;
-using Microsoft.AspNetCore.Mvc.Testing;
-using Microsoft.Extensions.DependencyInjection;
+using Xunit.Abstractions;
 
 namespace Cnblogs.Architecture.IntegrationTests;
 
@@ -22,7 +15,6 @@ public class IntegrationEventHandlerTests
     public IntegrationEventHandlerTests(IntegrationTestFactory factory, ITestOutputHelper testOutputHelper)
     {
         _factory = factory;
-        _factory.TestOutputHelper = testOutputHelper;
         _testOutputHelper = testOutputHelper;
     }
 
@@ -35,7 +27,7 @@ public async Task IntegrationEventHandler_TestIntegrationEvent_SuccessAsync()
 
         // Act
         var subscriptions = await client.GetFromJsonAsync<Subscription[]>("/dapr/subscribe");
-        var sub = subscriptions.First(x => x.Route.Contains(nameof(TestIntegrationEvent)));
+        var sub = subscriptions!.First(x => x.Route.Contains(nameof(TestIntegrationEvent)));
         var response = await client.PostAsJsonAsync(sub.Route, @event);
         _testOutputHelper.WriteLine("Subscription Route: " + sub.Route);