intelligent-machine-learning / dlrover

DLRover: An Automatic Distributed Deep Learning System
Other
1.22k stars 153 forks source link

Add ut for log collecting. #1270

Closed BalaBalaYi closed 2 weeks ago

BalaBalaYi commented 2 weeks ago

What changes were proposed in this pull request?

  1. Add ut.
  2. Add logging.

Why are the changes needed?

Add ut.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

UT.

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.57%. Comparing base (27d5292) to head (c7f4b82). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1270 +/- ## ========================================== + Coverage 80.55% 80.57% +0.02% ========================================== Files 219 219 Lines 20031 20042 +11 ========================================== + Hits 16136 16149 +13 + Misses 3895 3893 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.