Skip to content

Commit

Permalink
[JENKINS-28041] fix failing unittest
Browse files Browse the repository at this point in the history
  • Loading branch information
olivergondza committed Jun 9, 2015
1 parent 0e846fc commit b6bff4f
Showing 1 changed file with 14 additions and 7 deletions.
21 changes: 14 additions & 7 deletions core/src/test/java/hudson/cli/handlers/ViewOptionHandlerTest.java
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -157,10 +157,12 @@ public class ViewOptionHandlerTest {


denyAccessOn(outer); denyAccessOn(outer);


parseFailedWith(AccessDeniedException.class, "outer/nested/inner"); assertEquals(
"Access denied for: outer",
parseFailedWith(CmdLineException.class, "outer/nested/inner")
);


verify(outer).checkPermission(View.READ); verify(outer).checkPermission(View.READ);
verifyNoMoreInteractions(outer);


verifyZeroInteractions(nested); verifyZeroInteractions(nested);
verifyZeroInteractions(inner); verifyZeroInteractions(inner);
Expand All @@ -171,10 +173,12 @@ public class ViewOptionHandlerTest {


denyAccessOn(nested); denyAccessOn(nested);


parseFailedWith(AccessDeniedException.class, "outer/nested/inner"); assertEquals(
"Access denied for: nested",
parseFailedWith(CmdLineException.class, "outer/nested/inner")
);


verify(nested).checkPermission(View.READ); verify(nested).checkPermission(View.READ);
verifyNoMoreInteractions(nested);


verifyZeroInteractions(inner); verifyZeroInteractions(inner);
verifyZeroInteractions(setter); verifyZeroInteractions(setter);
Expand All @@ -184,17 +188,20 @@ public class ViewOptionHandlerTest {


denyAccessOn(inner); denyAccessOn(inner);


parseFailedWith(AccessDeniedException.class, "outer/nested/inner"); assertEquals(
"Access denied for: inner",
parseFailedWith(CmdLineException.class, "outer/nested/inner")
);


verify(inner).checkPermission(View.READ); verify(inner).checkPermission(View.READ);
verifyNoMoreInteractions(inner);


verifyZeroInteractions(setter); verifyZeroInteractions(setter);
} }


private void denyAccessOn(View view) { private void denyAccessOn(View view) {


doThrow(new AccessDeniedException(null)).when(view).checkPermission(View.READ); final AccessDeniedException ex = new AccessDeniedException("Access denied for: " + view.getViewName());
doThrow(ex).when(view).checkPermission(View.READ);
} }


private String parseFailedWith(Class<? extends Exception> type, final String... params) throws Exception { private String parseFailedWith(Class<? extends Exception> type, final String... params) throws Exception {
Expand Down

0 comments on commit b6bff4f

Please sign in to comment.