You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by sv...@apache.org on 2016/04/25 11:48:53 UTC

[15/17] wicket git commit: WICKET-6137 test fixes: request listener has to exist

WICKET-6137 test fixes: request listener has to exist


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

Branch: refs/heads/master
Commit: 19e01b2a8891604249a3ac16439d6ba0d704688c
Parents: 538ca83
Author: Sven Meier <sv...@apache.org>
Authored: Sun Apr 3 21:50:30 2016 +0200
Committer: Sven Meier <sv...@apache.org>
Committed: Mon Apr 25 11:45:02 2016 +0200

----------------------------------------------------------------------
 .../core/request/mapper/CryptoMapperTest.java   | 55 +++++++++++++++++---
 1 file changed, 49 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/19e01b2a/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java b/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java
index 5a9b950..98696fd 100644
--- a/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java
+++ b/wicket-core/src/test/java/org/apache/wicket/core/request/mapper/CryptoMapperTest.java
@@ -30,6 +30,7 @@ import org.apache.wicket.core.request.handler.RequestSettingRequestHandler;
 import org.apache.wicket.markup.IMarkupFragment;
 import org.apache.wicket.markup.Markup;
 import org.apache.wicket.markup.html.WebPage;
+import org.apache.wicket.markup.html.link.Link;
 import org.apache.wicket.protocol.http.PageExpiredException;
 import org.apache.wicket.protocol.http.WebApplication;
 import org.apache.wicket.request.IRequestHandler;
@@ -77,7 +78,7 @@ public class CryptoMapperTest extends AbstractMapperTest
 	@Before
 	public void before() throws Exception
 	{
-		tester = new WicketTester();
+		tester = new WicketTester(HomePage.class);
 
 		WebApplication application = tester.getApplication();
 		application.mountPage(MOUNTED_URL, Page1.class);
@@ -161,7 +162,7 @@ public class CryptoMapperTest extends AbstractMapperTest
 	@Test
 	public void homePageForceEncryptionOfRequestListener()
 	{
-		PageAndComponentProvider provider = new PageAndComponentProvider(tester.getApplication().getHomePage(), "some:link");
+		PageAndComponentProvider provider = new PageAndComponentProvider(tester.getApplication().getHomePage(), "link");
 		IRequestHandler requestHandler = new BookmarkableListenerInterfaceRequestHandler(provider);
 		Url plainUrl = mapper.getDelegateMapper().mapHandler(requestHandler);
 		assertTrue("Plain URL for home page has segments: " + plainUrl.toString(), plainUrl.getSegments().isEmpty());
@@ -263,7 +264,7 @@ public class CryptoMapperTest extends AbstractMapperTest
 	@Test
 	public void bookmarkablePageForceEncryptionOfRequestListener()
 	{
-		PageAndComponentProvider provider = new PageAndComponentProvider(Page2.class, "some:link");
+		PageAndComponentProvider provider = new PageAndComponentProvider(Page2.class, "link");
 		IRequestHandler requestHandler = new BookmarkableListenerInterfaceRequestHandler(provider);
 		Url plainUrl = mapper.getDelegateMapper().mapHandler(requestHandler);
 		assertTrue("Plain text request listener URL for bookmarkable page does not start with: "
@@ -354,7 +355,7 @@ public class CryptoMapperTest extends AbstractMapperTest
 	@Test
 	public void mountedPageRequestListenerParameter()
 	{
-		final String componentPath = "some:path:to:link";
+		final String componentPath = "link";
 
 		PageAndComponentProvider provider = new PageAndComponentProvider(Page1.class, componentPath);
 		IRequestHandler requestHandler = new ListenerInterfaceRequestHandler(provider);
@@ -657,14 +658,46 @@ public class CryptoMapperTest extends AbstractMapperTest
 	}
 
 	/**
+	 * Home page
+	 */
+	public static class HomePage extends WebPage
+	{
+		public HomePage()
+		{
+			add(new Link<Void>("link") {
+				@Override
+				public void onClick()
+				{
+				}
+			});
+		}
+		
+		@Override
+		public IMarkupFragment getMarkup()
+		{
+			return Markup.of("<html><body wicket:id=\"link\"></body></html>");
+		}
+	}
+
+	/**
 	 * Page that is mounted
 	 */
 	public static class Page1 extends WebPage
 	{
+		public Page1()
+		{
+			add(new Link<Void>("link") {
+				@Override
+				public void onClick()
+				{
+				}
+			});
+		}
+		
 		@Override
 		public IMarkupFragment getMarkup()
 		{
-			return Markup.of("<html><body></body></html>");
+			return Markup.of("<html><body wicket:id=\"link\"></body></html>");
 		}
 	}
 
@@ -673,10 +706,20 @@ public class CryptoMapperTest extends AbstractMapperTest
 	 */
 	public static class Page2 extends WebPage
 	{
+		public Page2()
+		{
+			add(new Link<Void>("link") {
+				@Override
+				public void onClick()
+				{
+				}
+			});
+		}
+		
 		@Override
 		public IMarkupFragment getMarkup()
 		{
-			return Markup.of("<html><body></body></html>");
+			return Markup.of("<html><body wicket:id=\"link\"></body></html>");
 		}
 	}
 }