テストクラスのベースクラスを作成しテストコードを更新する

samples/ConsoleAppWithDI/.gitignore

@@ -9,6 +9,7 @@
 *.user
 *.userosscache
 *.sln.docstates
+*.lutconfig
 
 # User-specific files (MonoDevelop/Xamarin Studio)
 *.userprefs

samples/ConsoleAppWithDI/solution/tests/.editorconfig

@@ -1,5 +1,7 @@
 [*.cs]
 # Wrapping preferences
+dotnet_diagnostic.IDE0039.severity=none
 dotnet_diagnostic.SA0001.severity=none
 dotnet_diagnostic.SA1123.severity=none
 dotnet_diagnostic.SA1600.severity=none
+dotnet_diagnostic.SA1602.severity=none

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/Commands/Command.cs

@@ -2,17 +2,11 @@
 
 namespace Maris.ConsoleApp.IntegrationTests.ScopeTests.Commands;
 
-internal class Command : SyncCommand<Parameter>
+internal class Command(TestObject1 obj1, TestObject2 obj2) : SyncCommand<Parameter>
 {
     internal const string CommandName = "scope-test";
-    private readonly TestObject1 obj1;
-    private readonly TestObject2 obj2;
-
-    public Command(TestObject1 obj1, TestObject2 obj2)
-    {
-        this.obj1 = obj1;
-        this.obj2 = obj2;
-    }
+    private readonly TestObject1 obj1 = obj1;
+    private readonly TestObject2 obj2 = obj2;
 
     protected override ICommandResult Execute(Parameter parameter)
     {

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/ObjectStateHistory.cs

@@ -2,7 +2,7 @@
 
 internal static class ObjectStateHistory
 {
-    private static readonly List<ObjectState> HistoryStore = new();
+    private static readonly List<ObjectState> HistoryStore = [];
 
     internal static IReadOnlyCollection<ObjectState> Histories => HistoryStore.AsReadOnly();
 

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/ScopedTest.cs

@@ -1,20 +1,14 @@
 using Maris.ConsoleApp.Hosting;
 using Maris.ConsoleApp.IntegrationTests.ScopeTests.Commands;
-using Maris.Logging.Testing.Xunit;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
 using Xunit.Abstractions;
 
 namespace Maris.ConsoleApp.IntegrationTests.ScopeTests;
 
 [Collection(nameof(ScopeTests))]
-public class ScopedTest
+public class ScopedTest(ITestOutputHelper testOutputHelper) : TestBase(testOutputHelper)
 {
-    private readonly TestLoggerManager loggerManager;
-
-    public ScopedTest(ITestOutputHelper testOutputHelper)
-        => this.loggerManager = new TestLoggerManager(testOutputHelper);
-
     [Fact]
     public async Task Scopedで登録したインスタンスはコマンド実行時に1回だけ初期化される()
     {
@@ -127,7 +121,7 @@ private IHost CreateHost()
         var builder = Host.CreateDefaultBuilder(args);
         builder.ConfigureServices((context, services) =>
         {
-            services.AddTestLogging(this.loggerManager);
+            services.AddTestLogging(this.LoggerManager);
             services.AddScoped<TestObject1>(); // Command 内で利用
             services.AddScoped<TestObject2>(); // Command, TestObject1 内で利用
             services.AddConsoleAppService(args);

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/SingletonTest.cs

@@ -1,20 +1,14 @@
 using Maris.ConsoleApp.Hosting;
 using Maris.ConsoleApp.IntegrationTests.ScopeTests.Commands;
-using Maris.Logging.Testing.Xunit;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
 using Xunit.Abstractions;
 
 namespace Maris.ConsoleApp.IntegrationTests.ScopeTests;
 
 [Collection(nameof(ScopeTests))]
-public class SingletonTest
+public class SingletonTest(ITestOutputHelper testOutputHelper) : TestBase(testOutputHelper)
 {
-    private readonly TestLoggerManager loggerManager;
-
-    public SingletonTest(ITestOutputHelper testOutputHelper)
-        => this.loggerManager = new TestLoggerManager(testOutputHelper);
-
     [Fact]
     public async Task Singletonで登録したインスタンスはコマンド実行時に1回だけ初期化される()
     {
@@ -127,7 +121,7 @@ private IHost CreateHost()
         var builder = Host.CreateDefaultBuilder(args);
         builder.ConfigureServices((context, services) =>
         {
-            services.AddTestLogging(this.loggerManager);
+            services.AddTestLogging(this.LoggerManager);
             services.AddSingleton<TestObject1>(); // Command 内で利用
             services.AddSingleton<TestObject2>(); // Command, TestObject1 内で利用
             services.AddConsoleAppService(args);

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/TestObject1.cs

@@ -1,10 +1,8 @@
 namespace Maris.ConsoleApp.IntegrationTests.ScopeTests;
 
-internal class TestObject1 : TestObjectBase
+internal class TestObject1(TestObject2 obj2) : TestObjectBase
 {
-    private readonly TestObject2 obj2;
-
-    public TestObject1(TestObject2 obj2) => this.obj2 = obj2;
+    private readonly TestObject2 obj2 = obj2;
 
     internal void DoSomething()
     {

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/TestObject2.cs

@@ -2,9 +2,5 @@
 
 internal class TestObject2 : TestObjectBase
 {
-    public TestObject2()
-    {
-    }
-
     internal void DoSomething() => this.LogHistory();
 }

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/ScopeTests/TransientTest.cs

@@ -1,20 +1,14 @@
 using Maris.ConsoleApp.Hosting;
 using Maris.ConsoleApp.IntegrationTests.ScopeTests.Commands;
-using Maris.Logging.Testing.Xunit;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Hosting;
 using Xunit.Abstractions;
 
 namespace Maris.ConsoleApp.IntegrationTests.ScopeTests;
 
 [Collection(nameof(ScopeTests))]
-public class TransientTest
+public class TransientTest(ITestOutputHelper testOutputHelper) : TestBase(testOutputHelper)
 {
-    private readonly TestLoggerManager loggerManager;
-
-    public TransientTest(ITestOutputHelper testOutputHelper)
-        => this.loggerManager = new TestLoggerManager(testOutputHelper);
-
     [Fact]
     public async Task Transientで登録したインスタンスはインジェクション時に毎回初期化される()
     {
@@ -130,7 +124,7 @@ private IHost CreateHost()
         var builder = Host.CreateDefaultBuilder(args);
         builder.ConfigureServices((context, services) =>
         {
-            services.AddTestLogging(this.loggerManager);
+            services.AddTestLogging(this.LoggerManager);
             services.AddTransient<TestObject1>(); // Command 内で利用
             services.AddTransient<TestObject2>(); // Command, TestObject1 内で利用
             services.AddConsoleAppService(args);

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.IntegrationTests/TestBase.cs

@@ -0,0 +1,15 @@
+using Maris.Logging.Testing.Xunit;
+using Xunit.Abstractions;
+
+namespace Maris.ConsoleApp.IntegrationTests;
+
+public class TestBase
+{
+    protected TestBase(ITestOutputHelper testOutputHelper)
+    {
+        ArgumentNullException.ThrowIfNull(testOutputHelper);
+        this.LoggerManager = new TestLoggerManager(testOutputHelper);
+    }
+
+    protected TestLoggerManager LoggerManager { get; }
+}

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.UnitTests/Core/AsyncCommandTest.cs

@@ -26,7 +26,7 @@ public void Parameter_ConsoleAppContextに設定したパラメーターを取
     }
 
     [Fact]
-    public void ValidateAllParameterを呼び出すとValidateParameterメソッドが指定したパラメーターを伴って呼び出される()
+    public void ValidateAllParameter_ValidateParameterメソッドがコンテキストに指定したパラメーターを伴って1回呼び出される()
     {
         // Arrange
         var parameter = new CommandParameter();
@@ -44,7 +44,7 @@ public void ValidateAllParameterを呼び出すとValidateParameterメソッド
     }
 
     [Fact]
-    public void IAsyncCommandのExecuteAsyncを呼び出すとExecuteAsyncメソッドが指定したパラメーターとキャンセルトークンを伴って呼び出される()
+    public void IAsyncCommandのExecuteAsync_ExecuteAsyncメソッドがコンテキストに指定したパラメーターとキャンセルトークンを伴って1回呼び出される()
     {
         // Arrange
         var parameter = new CommandParameter();
@@ -64,10 +64,10 @@ public void IAsyncCommandのExecuteAsyncを呼び出すとExecuteAsyncメソッ
     }
 
     [Fact]
-    public void パラメーターの型にインターフェースを使用できる()
+    public void Initialize_パラメーターの型にインターフェースを使用できる()
     {
         // Arrange
-        var parameter = new ParameterWithInterface();
+        IParameter parameter = new ParameterWithInterface();
         var commandAttribute = new CommandAttribute("dummy-command", typeof(CommandWithInterface));
         var context = new ConsoleAppContext(commandAttribute, parameter);
         var commandMock = new Mock<CommandWithInterface>();
@@ -78,7 +78,7 @@ public void パラメーターの型にインターフェースを使用でき
     }
 
     [Fact]
-    public void パラメーターの型とコンテキストのパラメーター型が一致しない場合は例外()
+    public void Initialize_パラメーターの型とコンテキストのパラメーター型が一致しない_InvalidOperationExceptionが発生する()
     {
         // Arrange
         var parameter = new ParameterWithInterface();

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.UnitTests/Core/CommandAttributeTest.cs

@@ -5,7 +5,7 @@ namespace Maris.ConsoleApp.UnitTests.Core;
 public class CommandAttributeTest
 {
     [Fact]
-    public void Constructor_コマンドの型がnullの場合は例外()
+    public void Constructor_コマンドの型がnull_ArgumentNullExceptionが発生する()
     {
         // Arrange
         string name = "command";
@@ -19,7 +19,7 @@ public void Constructor_コマンドの型がnullの場合は例外()
     }
 
     [Fact]
-    public void Constructor_コマンドの型がコマンドの定義を満たしていない場合は例外()
+    public void Constructor_コマンドの型がコマンドの定義を満たしていない_ArgumentExceptionが発生する()
     {
         // Arrange
         string name = "command";

samples/ConsoleAppWithDI/solution/tests/Maris.ConsoleApp.UnitTests/Core/CommandBaseTest.cs

@@ -37,7 +37,7 @@ public void CommandName_初期化時に指定したコマンド名を取得で
     }
 
     [Fact]
-    public void Context_初期値のまま取得すると例外()
+    public void Context_初期値のまま取得する_InvalidOperationExceptionが発生する()
     {
         // Arrange
         var command = new CommandImpl();
@@ -68,7 +68,7 @@ public void Context_初期化時に指定したコンテキストを取得でき
     }
 
     [Fact]
-    public void Initialize_nullで初期化すると例外()
+    public void Initialize_nullで初期化する_ArgumentNullExceptionが発生する()
     {
         // Arrange
         var command = new CommandImpl();
@@ -82,27 +82,29 @@ public void Initialize_nullで初期化すると例外()
     }
 
     [Fact]
-    public void ValidateAllParameter_入力パラメータのクラスにプロパティが定義されていない()
+    public void ValidateAllParameter_入力パラメータのクラスにプロパティが定義されていない_例外が発生しない()
     {
         // Arrange
         var command = new CommandImpl();
         string commandName = "dummy-command";
         var commandAttribute = new CommandAttribute(commandName, typeof(CommandImpl));
-        var context = new ConsoleAppContext(commandAttribute, new object());
+        var parameter = new object();
+        var context = new ConsoleAppContext(commandAttribute, parameter);
         command.Initialize(context);
 
         // Act and Assert(例外が発生しないこと)
         command.ValidateAllParameter();
     }
 
     [Fact]
-    public void ValidateAllParameter_入力パラメータのクラスにプロパティがあるが検証属性が定義されていない()
+    public void ValidateAllParameter_入力パラメータのクラスにプロパティがあるが検証属性が定義されていない_例外が発生しない()
     {
         // Arrange
         var command = new CommandImpl();
         string commandName = "dummy-command";
         var commandAttribute = new CommandAttribute(commandName, typeof(CommandImpl));
-        var context = new ConsoleAppContext(commandAttribute, new NoDataAnnotationParameter());
+        var parameter = new NoDataAnnotationParameter();
+        var context = new ConsoleAppContext(commandAttribute, parameter);
         command.Initialize(context);
 
         // Act and Assert(例外が発生しないこと)
@@ -112,7 +114,7 @@ public void ValidateAllParameter_入力パラメータのクラスにプロパ
     [Theory]
     [InlineData("", 0)]
     [InlineData("1234567890", 5)]
-    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり検証に成功する値が設定されている(string param1, int param2)
+    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり検証に成功する値が設定されている_例外が発生しない(string param1, int param2)
     {
         // Arrange
         var command = new CommandImpl();
@@ -131,7 +133,7 @@ public void ValidateAllParameter_入力パラメータのクラスに検証属
     }
 
     [Fact]
-    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり一部検証に失敗する値が設定されている()
+    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり一部検証に失敗する値が設定されている_検証失敗メッセージを伴うInvalidParameterExceptionが発生する()
     {
         // Arrange
         var command = new CommandImpl();
@@ -153,13 +155,11 @@ public void ValidateAllParameter_入力パラメータのクラスに検証属
 
         // Assert
         var ex = Assert.Throws<InvalidParameterException>(action);
-        Assert.Collection(
-            ex.ValidationResults,
-            result => Assert.Equal("Param2 は 0 から 5 の間で設定してください。", result.ErrorMessage));
+        Assert.Single(ex.ValidationResults, result => result.ErrorMessage == "Param2 は 0 から 5 の間で設定してください。");
     }
 
     [Fact]
-    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり複数検証に失敗する値が設定されている()
+    public void ValidateAllParameter_入力パラメータのクラスに検証属性を定義したプロパティがあり複数検証に失敗する値が設定されている_検証失敗メッセージを伴うInvalidParameterExceptionが発生する()
     {
         // Arrange
         var command = new CommandImpl();
@@ -188,7 +188,7 @@ public void ValidateAllParameter_入力パラメータのクラスに検証属
     }
 
     [Fact]
-    public void ValidateAllParameter_コマンドクラスのロジック内でパラメーターの入力値検証エラーがあった()
+    public void ValidateAllParameter_コマンドクラスのロジック内でパラメーターの入力値検証エラーがあった_InvalidParameterExceptionが発生する()
     {
         // Arrange
         var command = new ValidationErrorCommand();