----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52503/#review152443 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On Oct. 5, 2016, 6:05 a.m., Pradeep Agrawal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52503/ > ----------------------------------------------------------- > > (Updated Oct. 5, 2016, 6:05 a.m.) > > > Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan > Neethiraj, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and > Velmurugan Periasamy. > > > Bugs: RANGER-1170 > https://issues.apache.org/jira/browse/RANGER-1170 > > > Repository: ranger > > > Description > ------- > > Update unit test cases related to changePassword and changeEmail api for > users having 'keyadmin' or 'user' role > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/biz/RangerBizUtil.java > 6d14c66 > security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java f99addf > security-admin/src/main/java/org/apache/ranger/rest/UserREST.java b2979f5 > security-admin/src/test/java/org/apache/ranger/biz/TestUserMgr.java d77b691 > security-admin/src/test/java/org/apache/ranger/rest/TestUserREST.java > a5ed33c > > Diff: https://reviews.apache.org/r/52503/diff/ > > > Testing > ------- > > **Use Case 1: Change Password API** > > 1. User having role 'admin' can change his or any other user password. > 2. User having role 'keyadmin' or 'user' can change only his password; they > should not be able to change any other user password. > > **Use Case 2: Change Email API** > > 1. User having role 'admin' can change his or any other user email. > 2. User having role 'keyadmin' or 'user' can change only his email address; > they should not be able to change any other user email address. > > > Thanks, > > Pradeep Agrawal > >