Fix: Correct execution method in TestCommand

AnnaSasDev · AnnaSasDev · commit ec72544dee25 · 2025-05-15T11:26:27.000+02:00
Removed unnecessary `async` keyword from `ExecuteAsync` method in `TestCommand` and added a proper return with `ValueTask.CompletedTask`. Updated TUnit package reference to version 0.20.0 in the test project.
diff --git a/src/Sample.CodeOfChaos.CliArgsParser/TestCommand.cs b/src/Sample.CodeOfChaos.CliArgsParser/TestCommand.cs
@@ -13,10 +13,11 @@ namespace Sample.CodeOfChaos.CliArgsParser;
 // ---------------------------------------------------------------------------------------------------------------------
 [CliData, AutoName]
 public partial class TestCommand : ICliCommand<TestCommandParameters> {
-    public async ValueTask ExecuteAsync(TestCommandParameters parameters, CancellationToken ct = default) {
+    public ValueTask ExecuteAsync(TestCommandParameters parameters, CancellationToken ct = default) {
         Console.WriteLine(parameters.TestString);
         Console.WriteLine(parameters.TestInt);
         Console.WriteLine(parameters.TestBool);
         Console.WriteLine(parameters.TestRequiredString);
+        return ValueTask.CompletedTask;
     }
 }
diff --git a/tests/Tests.CodeOfChaos.CliArgsParser/Tests.CodeOfChaos.CliArgsParser.csproj b/tests/Tests.CodeOfChaos.CliArgsParser/Tests.CodeOfChaos.CliArgsParser.csproj
@@ -13,7 +13,7 @@
         <PackageReference Include="JetBrains.Annotations" Version="2024.3.0"/>
         <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="9.0.5" />
         <PackageReference Include="Moq" Version="4.20.72"/>
-        <PackageReference Include="TUnit" Version="0.19.148" />
+        <PackageReference Include="TUnit" Version="0.20.0" />
         <PackageReference Include="Bogus" Version="35.6.3" />
     </ItemGroup>
 

Original file line number	Diff line number	Diff line change
`@@ -13,10 +13,11 @@ namespace Sample.CodeOfChaos.CliArgsParser;`
`13`	`13`	`// ---------------------------------------------------------------------------------------------------------------------`
`14`	`14`	`[CliData, AutoName]`
`15`	`15`	`public partial class TestCommand : ICliCommand<TestCommandParameters> {`
`16`		`- public async ValueTask ExecuteAsync(TestCommandParameters parameters, CancellationToken ct = default) {`
	`16`	`+ public ValueTask ExecuteAsync(TestCommandParameters parameters, CancellationToken ct = default) {`
`17`	`17`	`Console.WriteLine(parameters.TestString);`
`18`	`18`	`Console.WriteLine(parameters.TestInt);`
`19`	`19`	`Console.WriteLine(parameters.TestBool);`
`20`	`20`	`Console.WriteLine(parameters.TestRequiredString);`
	`21`	`+ return ValueTask.CompletedTask;`
`21`	`22`	`}`
`22`	`23`	`}`