sofastack / sofa-rpc

SOFARPC is a high-performance, high-extensibility, production-level Java RPC framework.
https://www.sofastack.tech/sofa-rpc/docs/Home
Apache License 2.0
3.82k stars 1.17k forks source link

fix unit test #1291

Closed EvenLjj closed 1 year ago

EvenLjj commented 1 year ago

fix unit test

codecov[bot] commented 1 year ago

Codecov Report

Merging #1291 (c82038f) into master (81dc1ee) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1291      +/-   ##
============================================
+ Coverage     71.96%   71.99%   +0.02%     
- Complexity      780      783       +3     
============================================
  Files           413      414       +1     
  Lines         17555    17565      +10     
  Branches       2739     2740       +1     
============================================
+ Hits          12634    12646      +12     
+ Misses         3532     3527       -5     
- Partials       1389     1392       +3     
Impacted Files Coverage Δ
...rpc/bootstrap/triple/TripleClientProxyInvoker.java 100.00% <100.00%> (ø)
.../rpc/bootstrap/triple/TripleConsumerBootstrap.java 100.00% <100.00%> (ø)
...pay/sofa/rpc/config/ConfigUniqueNameGenerator.java 94.44% <100.00%> (+0.69%) :arrow_up:
...ofa/rpc/tracer/sofatracer/TripleTracerAdapter.java 61.31% <100.00%> (+0.57%) :arrow_up:
...n/java/com/alipay/sofa/rpc/log/TimeWaitLogger.java 80.00% <0.00%> (-4.00%) :arrow_down:
.../java/com/alipay/sofa/rpc/ext/ExtensionLoader.java 77.77% <0.00%> (-0.45%) :arrow_down:
...pay/sofa/rpc/message/bolt/BoltInvokerCallback.java 84.48% <0.00%> (ø)
...va/com/alipay/sofa/rpc/client/AbstractCluster.java 71.27% <0.00%> (+0.26%) :arrow_up:
...n/java/com/alipay/sofa/rpc/common/SofaConfigs.java 86.79% <0.00%> (+1.88%) :arrow_up:
...ofa/rpc/registry/consul/HealthServiceInformer.java 89.13% <0.00%> (+4.34%) :arrow_up:

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

EvenLjj commented 1 year ago

LGTM