Fix #1915: Allow result message to be truncated

src/Sarif/ExtensionMethods.cs

@@ -288,7 +288,7 @@ public static string GetMessageText(this Result result, ReportingDescriptor rule
  return GetMessageText(result, rule, concise: false);
  }
 
- public static string GetMessageText(this Result result, ReportingDescriptor rule, bool concise = false)
+ public static string GetMessageText(this Result result, ReportingDescriptor rule, bool concise = false, int maxLength = 200)
  {
  if (result == null)
  {
@@ -333,6 +333,10 @@ public static string GetMessageText(this Result result, ReportingDescriptor rule
  if (concise)
  {
  text = GetFirstSentence(text);
+ if (text.Length > maxLength)
+ {
+ text = text.Substring(0, maxLength) + "...";
+ }
  }
 
  return text;

src/Test.UnitTests.Sarif/SarifExtensionsTests.cs

@@ -113,5 +113,29 @@ public void SarifExtensions_Result_GetMessageText_Concise()
  string actual = result.GetMessageText(rule, concise: true);
  Assert.Equal(expected, actual);
  }
+
+ [Fact]
+ public void SarifExtensions_Result_GetMessageText_Concise_Truncated()
+ {
+ var result = new Result
+ {
+ Message = new Message
+ {
+ Id = "ruleStr1"
+ }
+ };
+
+ var rule = new ReportingDescriptor
+ {
+ MessageStrings = new Dictionary<string, MultiformatMessageString>
+ {
+ ["ruleStr1"] = new MultiformatMessageString { Text = "First sentence is very long. Second sentence." }
+ }
+ };
+
+ const string Expected = "First sentence is ve...";
+ string actual = result.GetMessageText(rule, concise: true, maxLength: 20);
+ Assert.Equal(Expected, actual);
+ }
  }
 }