Log at debug level

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

Hide file tree

Showing 2 changed files with 3 additions and 7 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
@@ -41,7 +41,7 @@ func TestCITests(t *testing.T) {
 			},
 			result: scut.TestReturn{
 				Score:        10,
-				NumberOfInfo: 1,
+				NumberOfDebug: 1,
 			},
 		},
 		{
@@ -74,8 +74,7 @@ func TestCITests(t *testing.T) {
 			},
 			result: scut.TestReturn{
 				Score:         7,
-				NumberOfInfo:  3,
-				NumberOfDebug: 1,
+				NumberOfDebug: 4,
 			},
 		},
 		{