Skip to content

Commit 6d60b2d

Browse files
committed
fix: division by zero leads to NaN for coverage
1 parent a2fcec9 commit 6d60b2d

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

internal/vulnerability/queries.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -324,6 +324,7 @@ func GetVulnerabilitySummary(ctx context.Context, s slug.Slug, filter *TeamVulne
324324
ret := &TeamVulnerabilitySummary{
325325
TeamSlug: s,
326326
}
327+
327328
for _, ws := range wSums {
328329
ret.Critical += ws.Summary.Critical
329330
ret.High += ws.Summary.High
@@ -335,7 +336,9 @@ func GetVulnerabilitySummary(ctx context.Context, s slug.Slug, filter *TeamVulne
335336
ret.BomCount++
336337
}
337338
}
338-
ret.Coverage = float64(ret.BomCount) / float64(len(wSums)) * 100
339+
if ret.BomCount > 0 {
340+
ret.Coverage = float64(ret.BomCount) / float64(len(wSums)) * 100
341+
}
339342

340343
return ret, nil
341344
}

0 commit comments

Comments
 (0)