----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44354/#review121984 -----------------------------------------------------------
I think the test failure is because subject is a mock, so you need to add an expectation to the test for the `subject.getPrincipal()` call you added to generate the response entity? - Joshua Cohen On March 3, 2016, 8:16 p.m., Zameer Manji wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44354/ > ----------------------------------------------------------- > > (Updated March 3, 2016, 8:16 p.m.) > > > Review request for Aurora and Stephan Erb. > > > Bugs: AURORA-1628 > https://issues.apache.org/jira/browse/AURORA-1628 > > > Repository: aurora > > > Description > ------- > > Show principal in auth failure messaage. > > > Diffs > ----- > > > src/main/java/org/apache/aurora/scheduler/http/api/security/ShiroAuthorizingParamInterceptor.java > 3043dfacfe824551b15595a070a1a4fdb7967d3a > > Diff: https://reviews.apache.org/r/44354/diff/ > > > Testing > ------- > > > Thanks, > > Zameer Manji > >
