-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31587/
-----------------------------------------------------------
Review request for ranger.
Bugs: RANGER-278
https://issues.apache.org/jira/browse/RANGER-278
Repository: ranger
Description
-------
Validation test for services
- Junits were added for ServiceREST only for create. Update and delete are not
testable currently without non-trivial refactoring. Hence, there is no direct
test of ServiceREST functionality. Of course, validator methods called during
update and delete have their junits.
Fixes unrelaed to this bug:
- Some pom files have hardcoded version in them. Version for child pom should
be inherited from that of parent pom as it the case for rest of child poms.
- ServiceDBStore was missing Apache license boiler plate.
- LdapUserGroupBuilder file had an unused import that was causing a compile
error.
- Fixed maven build warnings due to missing findbugs version and unused/stale
eclipse m2e plugin.
Diffs
-----
pom.xml b2a955d
ranger-util/pom.xml 047008a
security-admin/pom.xml 6d313b6
security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
0cb1707
security-admin/src/main/java/org/apache/ranger/rest/RangerServiceValidator.java
PRE-CREATION
security-admin/src/main/java/org/apache/ranger/rest/RangerValidator.java
PRE-CREATION
security-admin/src/main/java/org/apache/ranger/rest/RangerValidatorFactory.java
PRE-CREATION
security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java d4851cc
security-admin/src/main/java/org/apache/ranger/rest/ValidationFailureDetails.java
PRE-CREATION
security-admin/src/main/java/org/apache/ranger/rest/ValidationFailureDetailsBuilder.java
PRE-CREATION
security-admin/src/test/java/org/apache/ranger/rest/TestRangerServiceValidator.java
PRE-CREATION
security-admin/src/test/java/org/apache/ranger/rest/TestServiceRESTForValidation.java
PRE-CREATION
security-admin/src/test/java/org/apache/ranger/rest/TestServiceValidator.java
PRE-CREATION
security-admin/src/test/java/org/apache/ranger/rest/ValidationTestUtils.java
PRE-CREATION
ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java
f2fbf02
Diff: https://reviews.apache.org/r/31587/diff/
Testing
-------
Added new junits for create/update/delete validation of services
Manual excercise of REST api to validate response and error messages(s).
Thanks,
Alok Lal
