fix: update success indicator logic in MarkdownTableParser and corresponding tests

Author: thomasduft

src/testr.Cli/Domain/MarkdownTableParser.cs

@@ -36,7 +36,6 @@ public IEnumerable<TestStep> ParseTestSteps()
 
     // Parse the table rows
     var tableRows = ExtractTableRows(stepsSection);
-
     foreach (var row in tableRows)
     {
       var testStep = ParseTableRow(row);
@@ -173,8 +172,7 @@ private static bool IsSeparatorRow(string line)
       {
         var actualResult = cells[4].Trim();
         // Check for success/failure indicators
-        testStep.IsSuccess = actualResult.Contains("✅") ||
-                           actualResult.Equals("-", StringComparison.OrdinalIgnoreCase);
+        testStep.IsSuccess = actualResult.Contains("✅");
       }
 
       return testStep;

src/testr.Tests/MarkdownTableParserTests.cs

@@ -85,7 +85,7 @@ public void ParseTestSteps_WithSuccessFailureIndicators_ShouldSetIsSuccess()
     Assert.Equal(3, testSteps.Count);
     Assert.True(testSteps[0].IsSuccess);   // ✅ indicates success
     Assert.False(testSteps[1].IsSuccess);  // ❌ indicates failure
-    Assert.True(testSteps[2].IsSuccess);   // - indicates not run yet (considered success)
+    Assert.False(testSteps[2].IsSuccess);  // - indicates not run yet (considered failure)
   }
 
   [Fact]