Remove trailing pipe in Hit.ToString() if captured data is empty

Closes #646

Author: openbullet

RuriLib.Tests/Models/Hits/HitModelsTests.cs

@@ -41,6 +41,32 @@ public void Hit_CapturedDataString_UsesNullFallback()
         Assert.Equal("TOKEN = null", hit.CapturedDataString);
     }
 
+    [Fact]
+    public void Hit_ToString_OmitsSeparatorWhenCapturedDataIsEmpty()
+    {
+        var hit = new Hit
+        {
+            Data = new DataLine("user:pass", new WordlistType())
+        };
+
+        Assert.Equal("user:pass", hit.ToString());
+    }
+
+    [Fact]
+    public void Hit_ToString_JoinsDataAndCapturedDataWhenBothArePresent()
+    {
+        var hit = new Hit
+        {
+            Data = new DataLine("user:pass", new WordlistType()),
+            CapturedData = new Dictionary<string, object>
+            {
+                ["TOKEN"] = "abc"
+            }
+        };
+
+        Assert.Equal("user:pass | TOKEN = abc", hit.ToString());
+    }
+
     [Fact]
     public async Task FileSystemHitOutput_Store_WritesUsingEnvironmentNewLine()
     {

RuriLib/Models/Hits/Hit.cs

@@ -46,7 +46,8 @@ public class Hit
     /// Returns a compact string representation of the hit.
     /// </summary>
     /// <returns>The formatted hit string.</returns>
-    public override string ToString() => $"{DataString} | {CapturedDataString}";
+    public override string ToString()
+        => string.Join(" | ", new[] { DataString, CapturedDataString }.Where(s => !string.IsNullOrEmpty(s)));
 
     private string ConvertCapturedData()
     {