update comments

microsoft · Dec 1, 2021 · fdf350f · fdf350f
1 parent d1928a1
commit fdf350f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/Sarif.WorkItems/SarifWorkItemFiler.cs b/src/Sarif.WorkItems/SarifWorkItemFiler.cs
@@ -292,8 +292,7 @@ public SarifWorkItemModel FileWorkItemInternal(SarifLog sarifLog, SarifWorkItemC
 
                 if (sarifLog.Runs?.Any(run => run.Results?.Any(result => result.ShouldBeFiled()) == true) == false)
                 {
-                    LogMetricsForProcessedModel(sarifLog, null, FilingResult.None);
-                    // if sarifLog does not contains any result should be filed as work item return null
+                    // If the sarifLog does not contains result which should be filed as a work item, return null SarifWorkItemModel.
                     return null;
                 }
 

diff --git a/src/Test.UnitTests.Sarif.WorkItems/SarifWorkItemFilerTests.cs b/src/Test.UnitTests.Sarif.WorkItems/SarifWorkItemFilerTests.cs
@@ -111,7 +111,7 @@ public void WorkItemFiler_SarifLogResultsShouldNotBeFiled()
         {
             SarifLog sarifLog = TestData.CreateSimpleLog();
 
-            // set all results baseline status to unchanged so no file item should be filed
+            // Set all results' baseline state to 'Unchanged' so no work item should be filed
             sarifLog.Runs.SelectMany(run => run.Results).ForEach(result => result.BaselineState = BaselineState.Unchanged);
 
             SarifWorkItemContext context = CreateAzureDevOpsTestContext();