----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72325/#review220304 -----------------------------------------------------------
Ship it! Ship It! - Mehul Parikh On April 13, 2020, 12:51 p.m., Dineshkumar Yadav wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72325/ > ----------------------------------------------------------- > > (Updated April 13, 2020, 12:51 p.m.) > > > Review request for ranger, Ankita Sinha, Gautam Borad, Kishor Gollapalliwar, > Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Vishal Suvagia, and Velmurugan > Periasamy. > > > Repository: ranger > > > Description > ------- > > Handling the scenario when non-existing policy send with mergeIfExist=true > > > Diffs > ----- > > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 38b49820b > security-admin/src/main/java/org/apache/ranger/rest/ServiceRESTUtil.java > d85028c95 > > > Diff: https://reviews.apache.org/r/72325/diff/4/ > > > Testing > ------- > > Test case: > > Creating non-existing policy with below param > a) mergeIfExists=true > b) serviceName > c) policyName > > Expected: New policy should be created. > > Creating policy with below param and role which is already exist > a) mergeIfExists=true > b) serviceName > c) policyName > > Expected: role should get merged. > > > Thanks, > > Dineshkumar Yadav > >
