Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

optimize arklet modules #122

Closed
wants to merge 27 commits into from

Conversation

Lunarscave
Copy link
Contributor

  • fix actuator pom
  • add unit test for healthhandler
  • optimize code

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (77b7075) 46.19% compared to head (3c8ec4b) 41.31%.
Report is 125 commits behind head on master.

❗ Current head 3c8ec4b differs from pull request most recent head 417e7d7. Consider uploading reports for the commit 417e7d7 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #122      +/-   ##
============================================
- Coverage     46.19%   41.31%   -4.88%     
+ Complexity      160      137      -23     
============================================
  Files            45       45              
  Lines          1156     1152       -4     
  Branches         56       56              
============================================
- Hits            534      476      -58     
- Misses          597      648      +51     
- Partials         25       28       +3     
Files Coverage Δ
.../arklet/core/command/builtin/model/PluginInfo.java 31.57% <100.00%> (ø)
...ess/arklet/core/health/indicator/CpuIndicator.java 95.55% <ø> (ø)
...erless/arklet/core/health/indicator/Indicator.java 100.00% <100.00%> (ø)
...ess/arklet/core/health/indicator/JvmIndicator.java 100.00% <100.00%> (ø)
...sofa/serverless/arklet/core/util/ConvertUtils.java 66.66% <ø> (+26.66%) ⬆️
...rverless/arklet/core/health/HealthServiceImpl.java 56.66% <90.90%> (+18.62%) ⬆️

... and 24 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Lunarscave Lunarscave closed this Sep 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants