You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by pa...@apache.org on 2012/01/17 10:08:01 UTC

[1/2] git commit: WICKET-4340: fix testcase to actually pass with ticket fixed

Updated Branches:
  refs/heads/master 85fc48604 -> f7955440c


WICKET-4340: fix testcase to actually pass with ticket fixed


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/f7955440
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/f7955440
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/f7955440

Branch: refs/heads/master
Commit: f7955440cc8fc9d240a95c23ef23cb24c304aace
Parents: 565ea73
Author: Emond Papegaaij <pa...@apache.org>
Authored: Tue Jan 17 10:07:40 2012 +0100
Committer: Emond Papegaaij <pa...@apache.org>
Committed: Tue Jan 17 10:07:40 2012 +0100

----------------------------------------------------------------------
 .../wicket/util/tester/WicketTesterTest.java       |    8 ++++++--
 1 files changed, 6 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/f7955440/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java b/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java
index 93dd40e..c539a52 100644
--- a/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java
+++ b/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java
@@ -43,6 +43,7 @@ import org.apache.wicket.markup.html.form.Button;
 import org.apache.wicket.markup.html.form.FormComponent;
 import org.apache.wicket.markup.html.form.TextField;
 import org.apache.wicket.markup.html.link.Link;
+import org.apache.wicket.markup.html.pages.AccessDeniedPage;
 import org.apache.wicket.model.IModel;
 import org.apache.wicket.protocol.http.WebApplication;
 import org.apache.wicket.request.IRequestHandler;
@@ -67,7 +68,6 @@ import org.apache.wicket.util.tester.apps_1.ViewBook;
 import org.apache.wicket.util.tester.apps_6.LinkPage;
 import org.apache.wicket.util.tester.apps_6.ResultPage;
 import org.junit.Before;
-import org.junit.Ignore;
 import org.junit.Test;
 
 /**
@@ -1133,9 +1133,9 @@ public class WicketTesterTest extends WicketTestCase
 	 * permission anymore
 	 */
 	@Test
-	@Ignore
 	public void rerenderNotAllowed()
 	{
+		tester.setExposeExceptions(false);
 		class YesNoPageAuthorizationStrategy implements IAuthorizationStrategy
 		{
 			private boolean allowed = true;
@@ -1144,12 +1144,16 @@ public class WicketTesterTest extends WicketTestCase
 			public <T extends IRequestableComponent> boolean isInstantiationAuthorized(
 				Class<T> componentClass)
 			{
+				if (componentClass == AccessDeniedPage.class)
+					return true;
 				return allowed || !WebPage.class.isAssignableFrom(componentClass);
 			}
 
 			@Override
 			public boolean isActionAuthorized(Component component, Action action)
 			{
+				if (component instanceof AccessDeniedPage)
+					return true;
 				return allowed || !(component instanceof WebPage);
 			}
 		}