Log at debug level

Signed-off-by: Adam Korczynski <[email protected]>
ossf · Dec 11, 2023 · 0bf5cad · 0bf5cad
1 parent a7970c6
commit 0bf5cad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/checks/evaluation/ci_tests.go b/checks/evaluation/ci_tests.go
@@ -40,7 +40,7 @@ func CITests(name string,
 	// Debug PRs that were merged without CI tests
 	for i := range findings {
 		f := &findings[i]
-		if f.Outcome == finding.OutcomeNegative {
+		if f.Outcome == finding.OutcomeNegative || f.Outcome == finding.OutcomePositive {
 			dl.Debug(&checker.LogMessage{
 				Text: f.Message,
 			})
@@ -59,9 +59,6 @@ func CITests(name string,
 		return checker.CreateRuntimeErrorResult(name, e)
 	}
 
-	// Log all findings
-	checker.LogFindings(nonNegativeFindings(findings), dl)
-
 	reason := fmt.Sprintf("%d out of %d merged PRs checked by a CI test", totalTested, totalMerged)
 	return checker.CreateProportionalScoreResult(CheckCITests, reason, totalTested, totalMerged)
 }

diff --git a/checks/evaluation/ci_tests_test.go b/checks/evaluation/ci_tests_test.go
@@ -40,8 +40,8 @@ func TestCITests(t *testing.T) {
 				},
 			},
 			result: scut.TestReturn{
-				Score:        10,
-				NumberOfInfo: 1,
+				Score:         10,
+				NumberOfDebug: 1,
 			},
 		},
 		{
@@ -74,8 +74,7 @@ func TestCITests(t *testing.T) {
 			},
 			result: scut.TestReturn{
 				Score:         7,
-				NumberOfInfo:  3,
-				NumberOfDebug: 1,
+				NumberOfDebug: 4,
 			},
 		},
 		{