You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by "kezhenxu94 (GitHub)" <gi...@apache.org> on 2019/01/20 07:17:18 UTC

[GitHub] [incubator-dubbo-ops] kezhenxu94 opened pull request #261: add more integration test


[ Full content available at: https://github.com/apache/incubator-dubbo-ops/pull/261 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [incubator-dubbo-ops] codecov-io commented on issue #261: add more integration test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=h1) Report
> Merging [#261](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/incubator-dubbo-ops/commit/f07c9c5213f5062de4b37857dba653d2d5c8febb?src=pr&el=desc) will **increase** coverage by `2.81%`.
> The diff coverage is `50%`.

[![Impacted file tree graph](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/graphs/tree.svg?width=650&token=fYsc8RuDGB&height=150&src=pr)](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=tree)

```diff
@@              Coverage Diff              @@
##             develop     #261      +/-   ##
=============================================
+ Coverage      11.71%   14.52%   +2.81%     
- Complexity       161      205      +44     
=============================================
  Files             85       85              
  Lines           3457     3456       -1     
  Branches         547      547              
=============================================
+ Hits             405      502      +97     
+ Misses          3006     2909      -97     
+ Partials          46       45       -1
```


| [Impacted Files](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [.../org/apache/dubbo/admin/common/util/Constants.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29tbW9uL3V0aWwvQ29uc3RhbnRzLmphdmE=) | `0% <ø> (ø)` | `0 <0> (ø)` | :arrow_down: |
| [...e/dubbo/admin/controller/ManagementController.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29udHJvbGxlci9NYW5hZ2VtZW50Q29udHJvbGxlci5qYXZh) | `77.77% <100%> (+63.88%)` | `9 <0> (+7)` | :arrow_up: |
| [...ubbo/admin/service/impl/ManagementServiceImpl.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vc2VydmljZS9pbXBsL01hbmFnZW1lbnRTZXJ2aWNlSW1wbC5qYXZh) | `71.42% <40%> (+62.33%)` | `9 <0> (+7)` | :arrow_up: |
| [...n/registry/config/impl/ZookeeperConfiguration.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vcmVnaXN0cnkvY29uZmlnL2ltcGwvWm9va2VlcGVyQ29uZmlndXJhdGlvbi5qYXZh) | `76.19% <0%> (+4.76%)` | `22% <0%> (+2%)` | :arrow_up: |
| [...he/dubbo/admin/handler/CustomExceptionHandler.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vaGFuZGxlci9DdXN0b21FeGNlcHRpb25IYW5kbGVyLmphdmE=) | `29.41% <0%> (+17.64%)` | `3% <0%> (+1%)` | :arrow_up: |
| [...in/common/exception/ResourceNotFoundException.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29tbW9uL2V4Y2VwdGlvbi9SZXNvdXJjZU5vdEZvdW5kRXhjZXB0aW9uLmphdmE=) | `25% <0%> (+25%)` | `1% <0%> (+1%)` | :arrow_up: |
| [.../dubbo/admin/common/exception/SystemException.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29tbW9uL2V4Y2VwdGlvbi9TeXN0ZW1FeGNlcHRpb24uamF2YQ==) | `25% <0%> (+25%)` | `1% <0%> (+1%)` | :arrow_up: |
| [.../org/apache/dubbo/admin/common/CommonResponse.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29tbW9uL0NvbW1vblJlc3BvbnNlLmphdmE=) | `40.74% <0%> (+40.74%)` | `4% <0%> (+4%)` | :arrow_up: |
| [...va/org/apache/dubbo/admin/config/ConfigCenter.java](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree#diff-ZHViYm8tYWRtaW4tYmFja2VuZC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZHViYm8vYWRtaW4vY29uZmlnL0NvbmZpZ0NlbnRlci5qYXZh) | `92.45% <0%> (+47.16%)` | `17% <0%> (+11%)` | :arrow_up: |
| ... and [3 more](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=footer). Last update [f07c9c5...9191e7f](https://codecov.io/gh/apache/incubator-dubbo-ops/pull/261?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/incubator-dubbo-ops/pull/261 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [incubator-dubbo-ops] nzomkxia closed pull request #261: add more integration test

Posted by "nzomkxia (GitHub)" <gi...@apache.org>.
[ pull request closed by nzomkxia ]

[ Full content available at: https://github.com/apache/incubator-dubbo-ops/pull/261 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org