From 1ae7b69b43bafede7bae98ed1a8598e08cc96eba Mon Sep 17 00:00:00 2001 From: Denis Date: Sun, 9 Nov 2025 00:16:38 +0300 Subject: [PATCH 1/5] fix: update and add new test case --- pkg/testcoverage/report_test.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pkg/testcoverage/report_test.go b/pkg/testcoverage/report_test.go index af90792..b2774de 100644 --- a/pkg/testcoverage/report_test.go +++ b/pkg/testcoverage/report_test.go @@ -413,11 +413,13 @@ func Test_ReportUncoveredLines(t *testing.T) { FilesWithUncoveredLines: []coverage.Stats{ {Name: "a.go", UncoveredLines: []int{1, 2, 3}}, {Name: "b.go", UncoveredLines: []int{3, 5, 7}}, + {Name: "c.go", UncoveredLines: []int{1, 4, 10}, Covered: 20, Total: 100}, }, }) assertHasUncoveredLinesInfo(t, buf.String(), []string{ - "a.go\t\t1-3\n", - "b.go\t\t3 5 7\n", + "a.go\t\t0%\t\t1-3\n", + "b.go\t\t0%\t\t3 5 7\n", + "c.go\t\t20%\t\t1 4 10\n", }) // when result passes, there should be no output From a5e62ef6d395a992a7c4e6722d919ff57d8a9ae9 Mon Sep 17 00:00:00 2001 From: Denis Date: Sun, 9 Nov 2025 00:17:37 +0300 Subject: [PATCH 2/5] feat: add file coverage column in report table --- pkg/testcoverage/report.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/testcoverage/report.go b/pkg/testcoverage/report.go index 44979f4..b78f8dd 100644 --- a/pkg/testcoverage/report.go +++ b/pkg/testcoverage/report.go @@ -77,11 +77,13 @@ func reportUncoveredLines(w io.Writer, result AnalyzeResult) { defer tabber.Flush() fmt.Fprintf(tabber, "\nFiles with uncovered lines:") - fmt.Fprintf(tabber, "\n file:\tuncovered lines:") + fmt.Fprintf(tabber, "\n file:\tfile coverage:\tuncovered lines:") for _, stats := range result.FilesWithUncoveredLines { + if len(stats.UncoveredLines) > 0 { fmt.Fprintf(tabber, "\n %s\t", stats.Name) + fmt.Fprintf(tabber, "%v%%\t", stats.CoveredPercentageF()) compressUncoveredLines(tabber, stats.UncoveredLines) } } From a532cec6b5c93a1f3a5385f273735b486b5a68b6 Mon Sep 17 00:00:00 2001 From: Denis Date: Sun, 9 Nov 2025 00:19:00 +0300 Subject: [PATCH 3/5] fix: code fix --- pkg/testcoverage/report.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/testcoverage/report.go b/pkg/testcoverage/report.go index b78f8dd..7a98f48 100644 --- a/pkg/testcoverage/report.go +++ b/pkg/testcoverage/report.go @@ -80,7 +80,6 @@ func reportUncoveredLines(w io.Writer, result AnalyzeResult) { fmt.Fprintf(tabber, "\n file:\tfile coverage:\tuncovered lines:") for _, stats := range result.FilesWithUncoveredLines { - if len(stats.UncoveredLines) > 0 { fmt.Fprintf(tabber, "\n %s\t", stats.Name) fmt.Fprintf(tabber, "%v%%\t", stats.CoveredPercentageF()) From 39cdb7d898c8e8741c71d22a9a9745e9212dd761 Mon Sep 17 00:00:00 2001 From: Denis Date: Sun, 9 Nov 2025 02:56:12 +0300 Subject: [PATCH 4/5] fix: update name column --- pkg/testcoverage/report.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkg/testcoverage/report.go b/pkg/testcoverage/report.go index 7a98f48..c9cddff 100644 --- a/pkg/testcoverage/report.go +++ b/pkg/testcoverage/report.go @@ -77,7 +77,7 @@ func reportUncoveredLines(w io.Writer, result AnalyzeResult) { defer tabber.Flush() fmt.Fprintf(tabber, "\nFiles with uncovered lines:") - fmt.Fprintf(tabber, "\n file:\tfile coverage:\tuncovered lines:") + fmt.Fprintf(tabber, "\n file:\tcoverage:\tuncovered lines:") for _, stats := range result.FilesWithUncoveredLines { if len(stats.UncoveredLines) > 0 { From 714f63efd141f7c549b81169b5e6cfb457653170 Mon Sep 17 00:00:00 2001 From: Denis Date: Sun, 9 Nov 2025 02:56:50 +0300 Subject: [PATCH 5/5] feat: add new test case with float result --- pkg/testcoverage/report_test.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/pkg/testcoverage/report_test.go b/pkg/testcoverage/report_test.go index b2774de..e9173fe 100644 --- a/pkg/testcoverage/report_test.go +++ b/pkg/testcoverage/report_test.go @@ -414,12 +414,14 @@ func Test_ReportUncoveredLines(t *testing.T) { {Name: "a.go", UncoveredLines: []int{1, 2, 3}}, {Name: "b.go", UncoveredLines: []int{3, 5, 7}}, {Name: "c.go", UncoveredLines: []int{1, 4, 10}, Covered: 20, Total: 100}, + {Name: "d.go", UncoveredLines: []int{7, 8, 9}, Covered: 18, Total: 88}, }, }) assertHasUncoveredLinesInfo(t, buf.String(), []string{ "a.go\t\t0%\t\t1-3\n", "b.go\t\t0%\t\t3 5 7\n", "c.go\t\t20%\t\t1 4 10\n", + "d.go\t\t20.5%\t\t7-9\n", }) // when result passes, there should be no output