#43 - increase coverage

Author: Stepami

.github/workflows/develop.yml

@@ -63,7 +63,7 @@ jobs:
         with:
           path: ./tests/HydraScript.IntegrationTests/coverage.cobertura.xml
           repo_token: ${{ secrets.GITHUB_TOKEN }}
-          minimum_coverage: 20
+          minimum_coverage: 80
           fail_below_threshold: true
           show_class_names: true
           show_missing: true

src/HydraScript/Program.cs

@@ -1,6 +1,7 @@
 using System.CommandLine.Builder;
 using System.CommandLine.Hosting;
 using System.CommandLine.Parsing;
+using System.Diagnostics.CodeAnalysis;
 using HydraScript;
 using HydraScript.Infrastructure;
 using Microsoft.Extensions.DependencyInjection;
@@ -10,6 +11,7 @@
 
 return GetRunner(ConfigureHost).Invoke(args);
 
+[ExcludeFromCodeCoverage]
 internal static partial class Program
 {
     internal static readonly ExecuteCommand Command = new();

src/HydraScript/SimplestConsoleFormatter.cs

@@ -1,9 +1,11 @@
+using System.Diagnostics.CodeAnalysis;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
 using Microsoft.Extensions.Logging.Console;
 
 namespace HydraScript;
 
+[ExcludeFromCodeCoverage]
 internal class SimplestConsoleFormatter() : ConsoleFormatter(nameof(SimplestConsoleFormatter))
 {
     public override void Write<TState>(

tests/HydraScript.IntegrationTests/DumpOptionTests.cs

@@ -0,0 +1,31 @@
+using System.CommandLine.Parsing;
+using System.IO.Abstractions;
+using NSubstitute;
+
+namespace HydraScript.IntegrationTests;
+
+public class DumpOptionTests(TestHostFixture fixture) : IClassFixture<TestHostFixture>
+{
+    [Fact]
+    public void Invoke_DumpOptionPassed_FilesCreated()
+    {
+        var fileSystemMock = Substitute.For<IFileSystem>();
+        var runner = fixture.GetRunner(
+            configureTestServices: services =>
+                services.SetupInMemoryScript(script: string.Empty, fileSystemMock));
+        runner.Invoke(fixture.InMemoryScriptWithDump);
+
+        fileSystemMock.File.Received(1)
+            .WriteAllText(
+                TestHostFixture.ScriptFileName + ".tokens",
+                Arg.Any<string>());
+        fileSystemMock.File.Received(1)
+            .WriteAllText(
+                "ast.dot",
+                Arg.Any<string>());
+        fileSystemMock.File.Received(1)
+            .WriteAllLines(
+                TestHostFixture.ScriptFileName + ".tac",
+                Arg.Any<IEnumerable<string>>());
+    }
+}

tests/HydraScript.IntegrationTests/ServiceCollectionTestExtensions.cs

@@ -6,9 +6,12 @@ namespace HydraScript.IntegrationTests;
 
 internal static class ServiceCollectionTestExtensions
 {
-    internal static void SetupInMemoryScript(this IServiceCollection services, string script)
+    internal static void SetupInMemoryScript(
+        this IServiceCollection services,
+        string script,
+        IFileSystem? fileSystemMock = null)
     {
-        var fileSystem = Substitute.For<IFileSystem>();
+        var fileSystem = fileSystemMock ?? Substitute.For<IFileSystem>();
         var file = Substitute.For<IFile>();
         file.ReadAllText(default!).ReturnsForAnyArgs(script);
         fileSystem.File.ReturnsForAnyArgs(file);

tests/HydraScript.IntegrationTests/TestHostFixture.cs

@@ -15,7 +15,10 @@ public class TestHostFixture(
 {
     private readonly List<string> _logMessages = [];
 
-    public readonly string[] InMemoryScript = ["file.js"];
+    public const string ScriptFileName = "file";
+    public readonly string[] InMemoryScript = [$"{ScriptFileName}.js"];
+    public readonly string[] InMemoryScriptWithDump = [$"{ScriptFileName}.js", "-d"];
+
     public IReadOnlyCollection<string> LogMessages => _logMessages;
 
     public ITestOutputHelper? OutputHelper
@@ -41,12 +44,13 @@ public Parser GetRunner(Action<IServiceCollection>? configureTestServices = null
                 .ConfigureServices((context, services) =>
                 {
                     services.Configure<InvocationLifetimeOptions>(options => options.SuppressStatusMessages = true);
-                    var fileInfo = context.GetInvocationContext().ParseResult
-                        .GetValueForArgument(Program.Command.PathArgument);
+                    var parseResult = context.GetInvocationContext().ParseResult;
+                    var fileInfo = parseResult.GetValueForArgument(Program.Command.PathArgument);
+                    var dump = parseResult.GetValueForOption(Program.Command.DumpOption);
                     services
                         .AddDomain()
                         .AddApplication()
-                        .AddInfrastructure(dump: false, fileInfo);
+                        .AddInfrastructure(dump, fileInfo);
                     configureTestServices?.Invoke(services);
                 })
                 .UseCommandHandler<ExecuteCommand, ExecuteCommandHandler>(),