abhiishek26 opened a new pull request, #984:
URL: https://github.com/apache/ranger/pull/984
## What changes were proposed in this pull request?
the api put plugins/services/{id} implemented in ServiceREST updateService.
do not appropriately throws error when the service name field is left blank in
the payload or mismatch between ids passed through payload and request url
.
I compared this with the put apis present in ServiceREST and other modules
there appropriate id mismatch or name absent errors are handled .
I have made changes in the implementation to properly handle those scenarios
.
## How was this patch tested?
All the unit tests were passed locally and the bulild (mvn clean install ,
mvn clean test ) successfully passed
2 unit tests have been implemented to simulate id mismatch and servicename
absent in payload scenarios
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
