hertz icon indicating copy to clipboard operation
hertz copied to clipboard

test: add ut for pkg/common/hlog

Open L2ncE opened this issue 3 years ago • 2 comments

What type of PR is this?

test

Check the PR title.

  • [x] This PR title match the format: <type>(optional scope): <description>
  • [x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) More detail description for this PR(en: English/zh: Chinese).

en: add ut for pkg/common/hlog, coverage rate reached 81.4%

Which issue(s) this PR fixes:

#293

L2ncE avatar Oct 15 '22 07:10 L2ncE

Codecov Report

Base: 64.46% // Head: 64.88% // Increases project coverage by +0.41% :tada:

Coverage data is based on head (565fd16) compared to base (b4ba993). Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #298      +/-   ##
===========================================
+ Coverage    64.46%   64.88%   +0.41%     
===========================================
  Files           84       88       +4     
  Lines         8288     8463     +175     
===========================================
+ Hits          5343     5491     +148     
- Misses        2583     2608      +25     
- Partials       362      364       +2     
Impacted Files Coverage Δ
pkg/common/hlog/hlog.go 80.00% <0.00%> (ø)
pkg/common/hlog/default.go 83.83% <0.00%> (ø)
pkg/common/hlog/log.go 100.00% <0.00%> (ø)
pkg/common/hlog/system.go 85.96% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Oct 15 '22 08:10 codecov[bot]

Good job

li-jin-gou avatar Oct 15 '22 08:10 li-jin-gou