refactor: enhance error messages in TestCaseValidator and update RunCommand output format

Author: thomasduft

src/testr.Cli/Commands/RunCommand.cs

@@ -165,7 +165,7 @@ CancellationToken cancellationToken
       .WithDomain(domain)
       .WithVariables(VariablesHelper.CreateVariables(_variables.Values));
 
-    ConsoleHelper.WriteLineYellow($"Running Test Case: {testCase.Id}");
+    ConsoleHelper.WriteLineYellow($"Running Test Case Id '{testCase.Id}': {testCase.Title}");
 
     // Validate the Test Case definition
     var testCaseValidator = new TestCaseValidator(testCase);

src/testr.Cli/Domain/TestCaseParser.cs

@@ -41,12 +41,6 @@ internal async Task<TestCase> ToTestCaseAsync(CancellationToken cancellationToke
     };
   }
 
-  private IEnumerable<TestStep> GetTestSteps(string markdownContent)
-  {
-    var parser = new MarkdownTableParser(markdownContent);
-    return parser.ParseTestSteps();
-  }
-
   private (string TestCaseId, string TestCaseTitle) GetTestCaseIdAndTitle(string[] lines)
   {
     // we are just reading the first line
@@ -72,6 +66,12 @@ private IEnumerable<TestStep> GetTestSteps(string markdownContent)
     return splittedItems[1].Trim();
   }
 
+  private IEnumerable<TestStep> GetTestSteps(string markdownContent)
+  {
+    var parser = new MarkdownTableParser(markdownContent);
+    return parser.ParseTestSteps();
+  }
+
   private string? GetLinkedFile(string file, string? link)
   {
     if (string.IsNullOrWhiteSpace(link)) return null;

src/testr.Cli/Domain/TestCaseValidator.cs

@@ -23,7 +23,7 @@ public TestCaseValidationResult Validate()
     };
     if (!allowedTypes.Contains(_testCase.Type))
     {
-      result.AddError("Type", "Type must be 'Definition' or 'Run'.");
+      result.AddError("Type", $"Type must be '{Constants.TestCaseType.Definition}' or '{Constants.TestCaseType.Run}'.");
     }
 
     // check property Status
@@ -34,7 +34,7 @@ public TestCaseValidationResult Validate()
     };
     if (!allowedStatus.Contains(_testCase.Status))
     {
-      result.AddError("Status", "Status must be either 'Passed', 'Failed' or 'Unknown'.");
+      result.AddError("Status", $"Status must be either '{Constants.TestCaseStatus.Passed}', '{Constants.TestCaseStatus.Failed}' or '{Constants.TestCaseStatus.Unknown}'.");
     }
 
     // check property Link if contains link should be a valid markdown link pointing to a file