Skip to content

Commit 2e10c49

Browse files
committed
Merge branch 'main' of github.com:vladopajic/go-test-coverage
2 parents e155870 + dd31f96 commit 2e10c49

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

pkg/testcoverage/coverage/cover.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ import (
1414

1515
"golang.org/x/tools/cover"
1616

17+
"github.com/vladopajic/go-test-coverage/v2/pkg/testcoverage/logger"
1718
"github.com/vladopajic/go-test-coverage/v2/pkg/testcoverage/path"
1819
)
1920

@@ -47,6 +48,7 @@ func GenerateCoverageStats(cfg Config) ([]Stats, error) {
4748
}
4849

4950
if ok := matches(excludeRules, fi.name); ok {
51+
logger.L.Debug().Str("file", fi.name).Msg("file excluded")
5052
continue // this file is excluded
5153
}
5254

@@ -189,8 +191,7 @@ func listAllFiles(rootDir string) []fileInfo {
189191
return name
190192
}
191193

192-
//nolint:errcheck // error ignored because there is fallback mechanism for finding files
193-
filepath.Walk(rootDir, func(file string, info os.FileInfo, err error) error {
194+
err := filepath.Walk(rootDir, func(file string, info os.FileInfo, err error) error {
194195
if err != nil { // coverage-ignore
195196
return err
196197
}
@@ -206,6 +207,9 @@ func listAllFiles(rootDir string) []fileInfo {
206207

207208
return nil
208209
})
210+
if err != nil { // coverage-ignore
211+
logger.L.Error().Err(err).Msg("listing go files")
212+
}
209213

210214
return files
211215
}

pkg/testcoverage/coverage/module.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ func findModuleDirective(rootDir string) string {
2121
logger.L.Warn().Msg("`module` directive not found")
2222
}
2323

24+
logger.L.Debug().Str("module", module).Msg("using module directive")
25+
2426
return module
2527
}
2628

0 commit comments

Comments
 (0)