Merge pull request #6 from stevebeattie/fix-diff-changed-behavior-strings

egibs · web-flow · commit 9574aa36eb40 · 2024-12-30T12:25:58.000-06:00
terminal.go: when diffiing, compared based on scores, not string values
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" {`