----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67835/#review205764 -----------------------------------------------------------
security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java Line 621 (original), 621 (patched) <https://reviews.apache.org/r/67835/#comment288646> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java Line 813 (original), 813 (patched) <https://reviews.apache.org/r/67835/#comment288648> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java Line 1186 (original), 1186 (patched) <https://reviews.apache.org/r/67835/#comment288649> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Line 328 (original), 328 (patched) <https://reviews.apache.org/r/67835/#comment288650> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Lines 637 (patched) <https://reviews.apache.org/r/67835/#comment288651> The vxGroupUserList = new VXGroupUserList() assignment is unnecessary, as it is reassigned in the next line security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Line 875 (original), 870 (patched) <https://reviews.apache.org/r/67835/#comment288652> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Line 1270 (original) <https://reviews.apache.org/r/67835/#comment288653> returnList is assigned in the next lines - no need to declare to be null. security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java Line 2258 (original), 2251 (patched) <https://reviews.apache.org/r/67835/#comment288655> No need to make it public, the test class is in the same package, so package protected would be fine. security-admin/src/test/java/org/apache/ranger/biz/TestUserMgr.java Lines 191 (patched) <https://reviews.apache.org/r/67835/#comment288659> You can annotate this method with @After and JUnit will call it automatically after every test method, no need to call it by hand. - Zsombor Gegesy On July 5, 2018, 8:48 p.m., Pradeep Agrawal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67835/ > ----------------------------------------------------------- > > (Updated July 5, 2018, 8:48 p.m.) > > > Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay > Kulkarni, Madhan Neethiraj, Mehul Parikh, Ramesh Mani, Selvamohan Neethiraj, > Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-2150 > https://issues.apache.org/jira/browse/RANGER-2150 > > > Repository: ranger > > > Description > ------- > > **Problem Statement:** Unit test coverage for XUserMgr and UserMgr class need > to be increased. > > Currently Unit Test coverage of XUserMgr is 33.5% and UserMgr class is 54.9% > Total No. of test cases in TestXUserMgr class are: 37 > Total No. of test cases in TestUserMgr class are: 23 > > **Proposed Solution:** > Proposed patch contains lot of additional unit test cases and modification in > existing unit test cases. > Statistics after the patch are: > Test coverage of XUserMgr is 90.2% and UserMgr class is 85.3% > Total No. of test cases in TestXUserMgr class are: 100 > Total No. of test cases in TestUserMgr class are: 41 > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java 6d94e4f > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 7f5eab7 > security-admin/src/test/java/org/apache/ranger/biz/TestUserMgr.java 10453a5 > security-admin/src/test/java/org/apache/ranger/biz/TestXUserMgr.java > 39c60a9 > > > Diff: https://reviews.apache.org/r/67835/diff/1/ > > > Testing > ------- > > Tested on my local environment by using command : > > mvn clean compile package assembly:assembly install > > [INFO] Reactor Summary: > [INFO] > [INFO] ranger ............................................. SUCCESS [ 9.399 > s] > [INFO] Jdbc SQL Connector ................................. SUCCESS [ 2.039 > s] > [INFO] Credential Support ................................. SUCCESS [ 0.792 > s] > [INFO] Audit Component .................................... SUCCESS [ 2.001 > s] > [INFO] Common library for Plugins ......................... SUCCESS [ 3.299 > s] > [INFO] Installer Support Component ........................ SUCCESS [ 0.398 > s] > [INFO] Credential Builder ................................. SUCCESS [ 1.066 > s] > [INFO] Embedded Web Server Invoker ........................ SUCCESS [ 1.243 > s] > [INFO] Key Management Service ............................. SUCCESS [ 2.776 > s] > [INFO] ranger-plugin-classloader .......................... SUCCESS [ 0.463 > s] > [INFO] HBase Security Plugin Shim ......................... SUCCESS [ 2.522 > s] > [INFO] HBase Security Plugin .............................. SUCCESS [ 1.758 > s] > [INFO] Hdfs Security Plugin ............................... SUCCESS [ 1.822 > s] > [INFO] Hive Security Plugin ............................... SUCCESS [ 7.668 > s] > [INFO] Knox Security Plugin Shim .......................... SUCCESS [ 1.452 > s] > [INFO] Knox Security Plugin ............................... SUCCESS [ 4.083 > s] > [INFO] Storm Security Plugin .............................. SUCCESS [ 1.046 > s] > [INFO] YARN Security Plugin ............................... SUCCESS [ 0.924 > s] > [INFO] Ranger Util ........................................ SUCCESS [ 7.301 > s] > [INFO] Unix Authentication Client ......................... SUCCESS [ 1.258 > s] > [INFO] Security Admin Web Application ..................... SUCCESS [01:31 > min] > [INFO] KAFKA Security Plugin .............................. SUCCESS [ 4.827 > s] > [INFO] SOLR Security Plugin ............................... SUCCESS [ 5.235 > s] > [INFO] NiFi Security Plugin ............................... SUCCESS [ 0.950 > s] > [INFO] Unix User Group Synchronizer ....................... SUCCESS [03:34 > min] > [INFO] Ldap Config Check Tool ............................. SUCCESS [ 2.540 > s] > [INFO] Unix Authentication Service ........................ SUCCESS [ 4.010 > s] > [INFO] KMS Security Plugin ................................ SUCCESS [ 5.922 > s] > [INFO] Tag Synchronizer ................................... SUCCESS [ 12.373 > s] > [INFO] Hdfs Security Plugin Shim .......................... SUCCESS [ 12.243 > s] > [INFO] Hive Security Plugin Shim .......................... SUCCESS [01:00 > min] > [INFO] YARN Security Plugin Shim .......................... SUCCESS [ 6.474 > s] > [INFO] Storm Security Plugin shim ......................... SUCCESS [ 3.564 > s] > [INFO] KAFKA Security Plugin Shim ......................... SUCCESS [ 1.029 > s] > [INFO] SOLR Security Plugin Shim .......................... SUCCESS [ 6.671 > s] > [INFO] Atlas Security Plugin Shim ......................... SUCCESS [ 2.348 > s] > [INFO] KMS Security Plugin Shim ........................... SUCCESS [ 2.024 > s] > [INFO] ranger-examples .................................... SUCCESS [ 0.422 > s] > [INFO] Ranger Examples - Conditions and ContextEnrichers .. SUCCESS [ 1.269 > s] > [INFO] Ranger Examples - SampleApp ........................ SUCCESS [ 0.380 > s] > [INFO] Ranger Examples - Ranger Plugin for SampleApp ...... SUCCESS [ 0.942 > s] > [INFO] Ranger Tools ....................................... SUCCESS [ 1.384 > s] > [INFO] Atlas Security Plugin .............................. SUCCESS [ 1.660 > s] > [INFO] Sqoop Security Plugin .............................. SUCCESS [ 1.611 > s] > [INFO] Sqoop Security Plugin Shim ......................... SUCCESS [ 0.777 > s] > [INFO] Kylin Security Plugin .............................. SUCCESS [ 2.375 > s] > [INFO] Kylin Security Plugin Shim ......................... SUCCESS [ 1.695 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 42:52 min > [INFO] Finished at: 2018-07-06T02:12:31+05:30 > [INFO] Final Memory: 415M/886M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Pradeep Agrawal > >
