terminal.go: when diffiing, compared based on scores, not string values

stevebeattie · stevebeattie · commit 2347243757c1 · 2024-12-30T09:21:45.000-08:00
This meant NONE -&gt; MEDIUM would get a different indicator then
LOW -&gt; MEDIUM.

Signed-off-by: Steve Beattie &lt;steve.beattie@chainguard.dev&gt;
diff --git a/pkg/render/terminal.go b/pkg/render/terminal.go
@@ -291,10 +291,10 @@ func renderFileSummary(_ context.Context, fr *malcontent.FileReport, w io.Writer
 				diff = color.HiGreenString("+")
 			}
 
-			if riskLevel > previousRiskLevel {
+			if riskScore > previousNsRiskScore[ns]{
 				nsIcon = color.HiYellowString("▲")
 			}
-			if riskLevel < previousRiskLevel {
+			if riskScore < previousNsRiskScore[ns]{
 				nsIcon = color.HiGreenString("▼")
 			}
 			if riskLevel == "NONE" {

Original file line number	Diff line number	Diff line change
`@@ -291,10 +291,10 @@ func renderFileSummary(_ context.Context, fr *malcontent.FileReport, w io.Writer`
`291`	`291`	`diff = color.HiGreenString("+")`
`292`	`292`	`}`
`293`	`293`
`294`		`- if riskLevel > previousRiskLevel {`
	`294`	`+ if riskScore > previousNsRiskScore[ns]{`
`295`	`295`	`nsIcon = color.HiYellowString("▲")`
`296`	`296`	`}`
`297`		`- if riskLevel < previousRiskLevel {`
	`297`	`+ if riskScore < previousNsRiskScore[ns]{`
`298`	`298`	`nsIcon = color.HiGreenString("▼")`
`299`	`299`	`}`
`300`	`300`	`if riskLevel == "NONE" {`