Skip to content

1、configuration.publishConfig returns false,without throw exception#6094

Merged
mercyblitz merged 2 commits intoapache:masterfrom
imuxi:6093-fixes-modidy_unit_test
May 6, 2020
Merged

1、configuration.publishConfig returns false,without throw exception#6094
mercyblitz merged 2 commits intoapache:masterfrom
imuxi:6093-fixes-modidy_unit_test

Conversation

@imuxi
Copy link
Copy Markdown
Contributor

@imuxi imuxi commented May 2, 2020

What is the purpose of the change

fixes [#6093 ]

Brief changelog

configuration.publishConfig默认方法不再抛出异常,默认返回false,所以单元测试需要同步修改

Verifying this change

本地已完成测试

@codecov-io
Copy link
Copy Markdown

codecov-io commented May 2, 2020

Codecov Report

Merging #6094 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #6094      +/-   ##
============================================
+ Coverage     60.79%   60.83%   +0.03%     
- Complexity      492      493       +1     
============================================
  Files           998      998              
  Lines         39852    39860       +8     
  Branches       5739     5740       +1     
============================================
+ Hits          24230    24250      +20     
+ Misses        12929    12915      -14     
- Partials       2693     2695       +2     
Impacted Files Coverage Δ Complexity Δ
...mmon/config/configcenter/DynamicConfiguration.java 95.23% <100.00%> (ø) 0.00 <0.00> (ø)
.../org/apache/dubbo/rpc/model/ServiceRepository.java 66.10% <100.00%> (+2.46%) 0.00 <0.00> (ø)
...dubbo/rpc/protocol/dubbo/CallbackServiceCodec.java 82.01% <100.00%> (+0.26%) 0.00 <0.00> (ø)
...bo/rpc/protocol/dubbo/DecodeableRpcInvocation.java 81.94% <100.00%> (+6.23%) 0.00 <0.00> (ø)
...ache/dubbo/remoting/transport/AbstractChannel.java 37.50% <0.00%> (-50.00%) 0.00% <0.00%> (ø%)
...he/dubbo/common/threadpool/ThreadlessExecutor.java 52.17% <0.00%> (-10.87%) 0.00% <0.00%> (ø%)
.../apache/dubbo/rpc/protocol/AsyncToSyncInvoker.java 62.06% <0.00%> (-10.35%) 0.00% <0.00%> (ø%)
...ubbo/rpc/protocol/dubbo/ChannelWrappedInvoker.java 43.47% <0.00%> (-4.35%) 0.00% <0.00%> (ø%)
...c/main/java/org/apache/dubbo/rpc/RpcException.java 80.00% <0.00%> (-3.34%) 0.00% <0.00%> (ø%)
...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java 86.66% <0.00%> (-1.91%) 16.00% <0.00%> (ø%)
... and 18 more

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 2d0d713...6e8313a. Read the comment docs.

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