----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65567/#review197275 -----------------------------------------------------------
security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java Lines 1293 (patched) <https://reviews.apache.org/r/65567/#comment277403> Please address unit test failures. I see below error with this patch. ``` ------------------------------------------------------------------------------- Test set: org.apache.ranger.biz.TestServiceDBStore ------------------------------------------------------------------------------- Tests run: 31, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 1.83 sec <<< FAILURE! - in org.apache.ranger.biz.TestServiceDBStore unnecessary Mockito stubbings(org.apache.ranger.biz.TestServiceDBStore) Time elapsed: 0.034 sec <<< ERROR! org.mockito.exceptions.misusing.UnnecessaryStubbingException: Unnecessary stubbings detected in test class: TestServiceDBStore Clean & maintainable test code requires zero unnecessary code. Following stubbings are unnecessary (click to navigate to relevant line of code): 1. -> at org.apache.ranger.biz.TestServiceDBStore.test21deleteService(TestServiceDBStore.java:1375) Please remove unnecessary stubbings or use 'silent' option. More info: javadoc for UnnecessaryStubbingException class. at org.mockito.internal.runners.StrictRunner.run(StrictRunner.java:49) at org.mockito.junit.MockitoJUnitRunner.run(MockitoJUnitRunner.java:161) at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:283) at org.apache.maven.surefire.junit4.JUnit4Provider.executeWithRerun(JUnit4Provider.java:173) at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153) at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:128) at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203) at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155) at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103) ``` - Velmurugan Periasamy On Feb. 12, 2018, 6:13 a.m., bhavik patel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65567/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2018, 6:13 a.m.) > > > Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay > Kulkarni, Madhan Neethiraj, Pradeep Agrawal, Ramesh Mani, Selvamohan > Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-1972 > https://issues.apache.org/jira/browse/RANGER-1972 > > > Repository: ranger > > > Description > ------- > > In Ranger Admin, need a feature to be able to group (for example by tenant > name or business unit) sets of policies with one or more labels that are > relevant to business. > Furthermore, Need to able to search policies on this additional set of > labels. This will be useful for export/import policies as well. > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java > 60daed9 > > agents-common/src/main/java/org/apache/ranger/plugin/store/AbstractPredicateUtil.java > 7583864 > agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java > fa4f767 > security-admin/db/mysql/patches/030-policy-labels-schema.sql PRE-CREATION > security-admin/db/oracle/patches/030-policy-labels-schema.sql PRE-CREATION > security-admin/db/postgres/patches/030-policy-labels-schema.sql > PRE-CREATION > security-admin/db/sqlanywhere/patches/030-policy-labels-schema.sql > PRE-CREATION > security-admin/db/sqlserver/patches/030-policy-labels-schema.sql > PRE-CREATION > > security-admin/src/main/java/org/apache/ranger/biz/RangerPolicyRetriever.java > 1b6f440 > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java > 7aee433 > security-admin/src/main/java/org/apache/ranger/common/RangerSearchUtil.java > 92b0e03 > security-admin/src/main/java/org/apache/ranger/db/RangerDaoManagerBase.java > db20a14 > security-admin/src/main/java/org/apache/ranger/db/XXPolicyLabelDao.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/db/XXPolicyLabelMapDao.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/entity/XXPolicyLabel.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/entity/XXPolicyLabelMap.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 9e4d799 > > security-admin/src/main/java/org/apache/ranger/service/RangerPolicyLabelsService.java > PRE-CREATION > > security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java > 5aec948 > > security-admin/src/main/java/org/apache/ranger/service/RangerPolicyServiceBase.java > 5d3af2f > security-admin/src/main/java/org/apache/ranger/view/VXPolicy.java c73d42b > security-admin/src/main/java/org/apache/ranger/view/VXPolicyLabel.java > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/view/VXPolicyLabelList.java > PRE-CREATION > security-admin/src/main/resources/META-INF/jpa_named_queries.xml 786b4bf > security-admin/src/main/webapp/scripts/models/RangerPolicy.js b5a274a > security-admin/src/main/webapp/scripts/modules/globalize/message/en.js > b71427c > security-admin/src/main/webapp/scripts/utils/XAUtils.js 8fa7fca > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyDetail.js > 5bf2296 > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyForm.js > 542147f > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyRO.js > a01350b > > security-admin/src/main/webapp/scripts/views/policies/RangerPolicyTableLayout.js > ce295ff > > security-admin/src/main/webapp/scripts/views/reports/PlugableServiceDiffDetail.js > 914f271 > security-admin/src/main/webapp/scripts/views/reports/UserAccessLayout.js > 90f44c1 > security-admin/src/main/webapp/styles/xa.css ee13655 > > security-admin/src/main/webapp/templates/policies/RangerPolicyDetail_tmpl.html > f8d1791 > security-admin/src/main/webapp/templates/policies/RangerPolicyRO_tmpl.html > f298fd0 > security-admin/src/main/webapp/templates/reports/UserAccessLayout_tmpl.html > 36a7a6e > security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java > f7eb0d4 > > > Diff: https://reviews.apache.org/r/65567/diff/2/ > > > Testing > ------- > > 1. While creating policy, multiple labels can be assigned also lookup is > working. > 2. On policy listing page admin is able to filter policies based on policy > labels. > 3. On report page also admin is able to filter policies based on policy > labels. > > > Thanks, > > bhavik patel > >