-
Notifications
You must be signed in to change notification settings - Fork 4.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TEST] inc coverage of logger service #15870
[TEST] inc coverage of logger service #15870
Conversation
...inscheduler-api/src/test/java/org/apache/dolphinscheduler/api/service/LoggerServiceTest.java
Outdated
Show resolved
Hide resolved
5bb88c0
to
728e274
Compare
728e274
to
7ad4f37
Compare
7ad4f37
to
9a04802
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #15870 +/- ##
============================================
+ Coverage 39.28% 39.38% +0.10%
- Complexity 4951 4975 +24
============================================
Files 1348 1348
Lines 45660 45659 -1
Branches 4894 4895 +1
============================================
+ Hits 17937 17983 +46
+ Misses 25808 25764 -44
+ Partials 1915 1912 -3 ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
return user; | ||
} | ||
|
||
public static User getAdminUser() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that this part is not used in this PR
Others LGTM
Purpose of the pull request
increase coverage of logger service