-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
optimize: optimize Configuration
and add unit tests
#3120
optimize: optimize Configuration
and add unit tests
#3120
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3120 +/- ##
=============================================
- Coverage 50.45% 50.42% -0.03%
- Complexity 3108 3112 +4
=============================================
Files 593 594 +1
Lines 19571 19595 +24
Branches 2427 2441 +14
=============================================
+ Hits 9874 9881 +7
- Misses 8702 8715 +13
- Partials 995 999 +4
|
Configuration
provide getArray and getList methodConfiguration
provide getEnum, getArray and getList method
Configuration
provide getEnum, getArray and getList methodConfiguration
added getEnum, getArray and getList method
Configuration
added getEnum, getArray and getList methodConfiguration
added getEnum, getArray and getList method
Configuration
added getEnum, getArray and getList methodConfiguration
added getEnum, getArray and getList method
Codecov Report
@@ Coverage Diff @@
## develop #3120 +/- ##
=============================================
- Coverage 50.93% 50.89% -0.05%
- Complexity 3223 3225 +2
=============================================
Files 605 606 +1
Lines 19891 19915 +24
Branches 2483 2497 +14
=============================================
+ Hits 10132 10136 +4
- Misses 8746 8763 +17
- Partials 1013 1016 +3
|
…-Configuration # Conflicts: # config/seata-config-custom/src/test/resources/custom_for_test.properties # core/src/main/java/io/seata/core/store/db/AbstractDataSourceProvider.java # seata-spring-boot-starter/src/main/java/io/seata/spring/boot/autoconfigure/provider/SpringBootConfigurationProvider.java
Codecov Report
@@ Coverage Diff @@
## develop #3120 +/- ##
=============================================
+ Coverage 40.64% 40.69% +0.04%
- Complexity 2990 2993 +3
=============================================
Files 676 676
Lines 22582 22609 +27
Branches 2756 2809 +53
=============================================
+ Hits 9179 9201 +22
- Misses 12531 12556 +25
+ Partials 872 852 -20
|
Configuration
added getEnum, getArray and getList methodConfiguration
and add unit tests
config/seata-config-core/src/main/java/io/seata/config/AbstractConfiguration.java
Show resolved
Hide resolved
config/seata-config-custom/src/test/java/io/seata/config/ConfigurationTest.java
Show resolved
Hide resolved
config/seata-config-custom/src/test/java/io/seata/config/ConfigurationTest.java
Show resolved
Hide resolved
config/seata-config-core/src/main/java/io/seata/config/AbstractConfiguration.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Configuration
and add unit testsConfiguration
and add unit tests
optimize: opt
Configuration
and add unit tests优化:优化
Configuration
的部分代码,并添加单元测试。以后再提PR改造配置中心,参考spring-boot的环境加载器的设计。