Improved tests

Author: NikolayPianikov

CSharpInteractive.Tests/UsageScenarios/CommandLineAsyncScenario.cs

@@ -35,6 +35,6 @@ await GetService<ICommandLineRunner>()
             .EnsureSuccess();
         // }
 
-        result.ExitCode.HasValue.ShouldBeTrue();
+        result.ExitCode.HasValue.ShouldBeTrue(result.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/CommandLineInParallelScenario.cs

@@ -36,7 +36,7 @@ public async Task Run()
         await task;
         // }
 
-        task.Result.ExitCode.HasValue.ShouldBeTrue();
-        result.ExitCode.HasValue.ShouldBeTrue();
+        task.Result.ExitCode.HasValue.ShouldBeTrue(result.ToString());
+        result.ExitCode.HasValue.ShouldBeTrue(result.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/CommandLineOutputScenario.cs

@@ -33,6 +33,6 @@ public void Run()
         lines.ShouldContain("MyEnv=MyVal");
         // }
 
-        result.ExitCode.HasValue.ShouldBeTrue();
+        result.ExitCode.HasValue.ShouldBeTrue(result.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/CommandLineWithTimeoutScenario.cs

@@ -30,8 +30,8 @@ public void Run()
             .Run(default, TimeSpan.FromMilliseconds(1))
             .EnsureSuccess()
             .ExitCode;
-
-        exitCode.HasValue.ShouldBeFalse();
         // }
+
+        exitCode.HasValue.ShouldBeFalse(exitCode.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/DockerDotNetBuildScenario.cs

@@ -49,12 +49,12 @@ public void Run()
             .Build()
             .EnsureSuccess();
 
-        // The "result" variable provides details about a build
-        result.Errors.Any(message => message.State == BuildMessageState.StdError).ShouldBeFalse();
-        result.ExitCode.ShouldBe(0);
-
         string ToAbsoluteLinuxPath(string path) =>
             "/" + path.Replace(":", "").Replace('\\', '/');
         // }
+        
+        // The "result" variable provides details about a build
+        result.Errors.Any(message => message.State == BuildMessageState.StdError).ShouldBeFalse(result.ToString());
+        result.ExitCode.ShouldBe(0, result.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/DotNetBuildScenario.cs

@@ -30,11 +30,13 @@ public void Run()
             .Build().EnsureSuccess();
 
         // Builds the library project, running a command like: "dotnet build" from the directory "MyLib"
+        var messages = new List<BuildMessage>();
         var result = new DotNetBuild()
             .WithWorkingDirectory("MyLib")
-            .Build().EnsureSuccess();
+            .Build(message => messages.Add(message)).EnsureSuccess();
 
         // The "result" variable provides details about a build
+        messages.Count.ShouldBeGreaterThan(0, result.ToString());
         result.Errors.Any(message => message.State == BuildMessageState.StdError).ShouldBeFalse();
         result.ExitCode.ShouldBe(0);
 
@@ -43,10 +45,10 @@ public void Run()
             .WithWorkingDirectory("MyLib")
             .Build()
             .EnsureSuccess();
-
+        // }
+        
         result.ExitCode.ShouldBe(0);
         result.Summary.Tests.ShouldBe(1);
         result.Tests.Count(test => test.State == TestState.Finished).ShouldBe(1);
-        // }
     }
 }

CSharpInteractive.Tests/UsageScenarios/DotNetCsiScenario.cs

@@ -0,0 +1,42 @@
+// ReSharper disable StringLiteralTypo
+// ReSharper disable ObjectCreationAsStatement
+// ReSharper disable ReturnValueOfPureMethodIsNotUsed
+
+namespace CSharpInteractive.Tests.UsageScenarios;
+
+using System.Diagnostics.CodeAnalysis;
+using HostApi;
+
+[CollectionDefinition("Integration", DisableParallelization = true)]
+[Trait("Integration", "True")]
+[SuppressMessage("Performance", "CA1861:Avoid constant arrays as arguments")]
+public class DotNetCsiScenario : BaseScenario
+{
+    [Fact]
+    public void Run()
+    {
+        // $visible=true
+        // $tag=07 .NET CLI
+        // $priority=02
+        // $description=Run C# script
+        // {
+        // Adds the namespace "HostApi" to use .NET build API
+        // ## using HostApi;
+
+        var script = Path.GetTempFileName();
+        File.WriteAllText(script, "Console.WriteLine($\"Hello, {Args[0]}!\");");
+
+        var stdOut = new List<string>();
+        var result = new DotNetCsi()
+            .WithScript(script)
+            .AddArgs("World")
+            .Build(message => stdOut.Add(message.Text))
+            .EnsureSuccess();
+
+        result.ExitCode.ShouldBe(0);
+
+        // Checks StdOut
+        stdOut.Contains("Hello, World!").ShouldBeTrue(result.ToString());
+        // }
+    }
+}

CSharpInteractive.Tests/UsageScenarios/DotNetPackScenario.cs

@@ -36,8 +36,8 @@ public void Run()
             .WithWorkingDirectory("MyLib")
             .AddProps(("version", "1.2.3"))
             .Build().EnsureSuccess();
-
-        result.ExitCode.ShouldBe(0);
         // }
+        
+        result.ExitCode.ShouldBe(0, result.ToString());
     }
 }

CSharpInteractive.Tests/UsageScenarios/DotNetPublishScenario.cs

@@ -37,8 +37,8 @@ public void Run()
             .WithWorkingDirectory("MyLib")
             .WithFramework("net8.0")
             .Build().EnsureSuccess();
-
-        result.ExitCode.ShouldBe(0);
         // }
+        
+        result.ExitCode.ShouldBe(0);
     }
 }

CSharpInteractive.Tests/UsageScenarios/DotNetRestoreScenario.cs

@@ -35,8 +35,8 @@ public void Run()
         result = new DotNetRestore()
             .WithWorkingDirectory("MyLib")
             .Build().EnsureSuccess();
-
-        result.ExitCode.ShouldBe(0);
         // }
+        
+        result.ExitCode.ShouldBe(0);
     }
 }