Collect additional report field in an ExtensionData property (#3079)

* Collect additional report field in an ExtensionData property * typo fix
microsoft · May 4, 2023 · d237563 · d237563
1 parent f3ec0e5
commit d237563
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/ApiService/ApiService/OneFuzzTypes/Model.cs b/src/ApiService/ApiService/OneFuzzTypes/Model.cs
@@ -461,6 +461,8 @@ public record Report(
     string? OnefuzzVersion,
     Uri? ReportUrl
 ) : IReport, ITruncatable<Report> {
+
+    [JsonExtensionData] public Dictionary<string, JsonElement>? ExtensionData { get; set; }
     public Report Truncate(int maxLength) {
         return this with {
             Executable = Executable[..maxLength],

diff --git a/src/ApiService/Tests/ReportTests.cs b/src/ApiService/Tests/ReportTests.cs
@@ -36,7 +36,9 @@ void TestParseReport() {
     },
     "tool_name": "libfuzzer",
     "tool_version": "1.2.3",
-    "onefuzz_version": "1.2.3"
+    "onefuzz_version": "1.2.3",
+    "extra_property1": "test",
+    "extra_property2": 5
 }
 """;
 
@@ -78,7 +80,11 @@ void TestParseReport() {
 """;
 
         var report = Reports.ParseReportOrRegression(testReport, new Uri("http://test"));
-        _ = Assert.IsType<Report>(report);
+        var reportInstance = Assert.IsType<Report>(report);
+
+        Assert.Equal("test", reportInstance?.ExtensionData?["extra_property1"].GetString());
+        Assert.Equal(5, reportInstance?.ExtensionData?["extra_property2"].GetInt32());
+
 
         var regression = Reports.ParseReportOrRegression(testRegresion, new Uri("http://test"));
         _ = Assert.IsType<RegressionReport>(regression);