-
Notifications
You must be signed in to change notification settings - Fork 404
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
Azure Bindings: Resolving IllegalArgumentException when parsing storage custom config parameters #6876
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportBase: 86.45% // Head: 86.47% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #6876 +/- ##
============================================
+ Coverage 86.45% 86.47% +0.02%
+ Complexity 16066 16060 -6
============================================
Files 1039 1038 -1
Lines 59922 59878 -44
Branches 6064 6057 -7
============================================
- Hits 51803 51779 -24
+ Misses 4938 4929 -9
+ Partials 3181 3170 -11
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
59045f2
to
70671c6
Compare
Signed-off-by: Aditi Jadhav <Aditi.Jadhav@dell.com> Signed-off-by: Aditi Jadhav <Aditi.Jadhav@dell.com>
Signed-off-by: Aditi Jadhav <Aditi.Jadhav@dell.com>
Signed-off-by: Aditi Jadhav <Aditi.Jadhav@dell.com>
Signed-off-by: Aditi Jadhav <Aditi.Jadhav@dell.com>
@@ -240,19 +240,27 @@ private Map<String, Object> getTier2Env() { | |||
return parseSystemPropertyAsMap("tier2Env"); | |||
} | |||
|
|||
private Map<String, Object> parseSystemPropertyAsMap(String systemProperty) { | |||
public static Map<String, Object> parseSystemPropertyAsMap(String systemProperty) { |
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.
If this is just for testing, this could have package visibility instead of public
, right?
} | ||
|
||
@NonNull | ||
public static Map<String, Object> parseSystemPropertyAsMap(String systemProperty, String value) { |
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.
This is completely changing the behavior of this method, right? Previously, it was returning a map in which all values were Integer
, and now it is putting some string in the value. How is this impacting the configuration of system tests? Are system tests working and configured as expected with this change?
Change log description
Resolving IllegalArgumentException when parsing config parameters in a String.
Purpose of the change
Fixes #6875
What the code does
The code splits the custom config parsed as String ("," separated) appropriately into the config parameter and it's value.
It then stores them in the form of key-value pairs and returns the resulting map.
How to verify it
All the tests should pass.