Skip to content
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

improve StringUtilTest #526

Merged
merged 1 commit into from
May 27, 2019
Merged

improve StringUtilTest #526

merged 1 commit into from
May 27, 2019

Conversation

javeme
Copy link
Contributor

@javeme javeme commented May 24, 2019

Change-Id: I7a5aa7fd763efbef7fd8da3daab759224395ddec

Change-Id: I7a5aa7fd763efbef7fd8da3daab759224395ddec
@codecov
Copy link

codecov bot commented May 24, 2019

Codecov Report

Merging #526 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #526      +/-   ##
============================================
+ Coverage     69.93%   69.97%   +0.04%     
- Complexity     3388     3392       +4     
============================================
  Files           216      216              
  Lines         16503    16503              
  Branches       2359     2359              
============================================
+ Hits          11541    11548       +7     
+ Misses         3700     3695       -5     
+ Partials       1262     1260       -2
Impacted Files Coverage Δ Complexity Δ
...n/java/com/baidu/hugegraph/task/TaskScheduler.java 52.69% <0%> (+1.24%) 27% <0%> (+1%) ⬆️
...main/java/com/baidu/hugegraph/util/StringUtil.java 80% <0%> (+13.33%) 11% <0%> (+2%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f86a716...30f9bea. Read the comment docs.

@zhoney zhoney merged commit 9188fb2 into master May 27, 2019
@zhoney zhoney deleted the StringUtilTest-improve branch May 27, 2019 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants