Skip to content

Commit eecd8fb

Browse files
feat(report): show file coverage with uncovered lines (#237)
1 parent 3db4a28 commit eecd8fb

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

pkg/testcoverage/report.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,12 @@ func reportUncoveredLines(w io.Writer, result AnalyzeResult) {
7777
defer tabber.Flush()
7878

7979
fmt.Fprintf(tabber, "\nFiles with uncovered lines:")
80-
fmt.Fprintf(tabber, "\n file:\tuncovered lines:")
80+
fmt.Fprintf(tabber, "\n file:\tcoverage:\tuncovered lines:")
8181

8282
for _, stats := range result.FilesWithUncoveredLines {
8383
if len(stats.UncoveredLines) > 0 {
8484
fmt.Fprintf(tabber, "\n %s\t", stats.Name)
85+
fmt.Fprintf(tabber, "%v%%\t", stats.CoveredPercentageF())
8586
compressUncoveredLines(tabber, stats.UncoveredLines)
8687
}
8788
}

pkg/testcoverage/report_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -413,11 +413,15 @@ func Test_ReportUncoveredLines(t *testing.T) {
413413
FilesWithUncoveredLines: []coverage.Stats{
414414
{Name: "a.go", UncoveredLines: []int{1, 2, 3}},
415415
{Name: "b.go", UncoveredLines: []int{3, 5, 7}},
416+
{Name: "c.go", UncoveredLines: []int{1, 4, 10}, Covered: 20, Total: 100},
417+
{Name: "d.go", UncoveredLines: []int{7, 8, 9}, Covered: 18, Total: 88},
416418
},
417419
})
418420
assertHasUncoveredLinesInfo(t, buf.String(), []string{
419-
"a.go\t\t1-3\n",
420-
"b.go\t\t3 5 7\n",
421+
"a.go\t\t0%\t\t1-3\n",
422+
"b.go\t\t0%\t\t3 5 7\n",
423+
"c.go\t\t20%\t\t1 4 10\n",
424+
"d.go\t\t20.5%\t\t7-9\n",
421425
})
422426

423427
// when result passes, there should be no output

0 commit comments

Comments
 (0)