Repository: wicket
Updated Branches:
  refs/heads/master 09dc961ba -> 7eb4db6e5


WICKET-5665 WicketTester#assertComponentOnAjaxResponse() cannot test invisible 
components

(cherry picked from commit b25171c2ab17af6a6955d45f702daaabc396d8f1)


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

Branch: refs/heads/master
Commit: 7eb4db6e5a2d87161459b888a0589ec9770a1f91
Parents: 09dc961
Author: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
Authored: Wed Aug 6 13:58:43 2014 +0200
Committer: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
Committed: Wed Aug 6 14:02:31 2014 +0200

----------------------------------------------------------------------
 .../apache/wicket/util/tester/WicketTester.java |  2 +-
 .../wicket/util/tester/WicketTesterTest.java    | 30 ++++++++++++++++++++
 2 files changed, 31 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/7eb4db6e/wicket-core/src/main/java/org/apache/wicket/util/tester/WicketTester.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/util/tester/WicketTester.java 
b/wicket-core/src/main/java/org/apache/wicket/util/tester/WicketTester.java
index 87002ce..46b9f4a 100644
--- a/wicket-core/src/main/java/org/apache/wicket/util/tester/WicketTester.java
+++ b/wicket-core/src/main/java/org/apache/wicket/util/tester/WicketTester.java
@@ -298,7 +298,7 @@ public class WicketTester extends BaseWicketTester
         */
        public void assertComponentOnAjaxResponse(String componentPath)
        {
-               
assertComponentOnAjaxResponse(getComponentFromLastRenderedPage(componentPath));
+               
assertComponentOnAjaxResponse(getComponentFromLastRenderedPage(componentPath, 
false));
        }
 
        /**

http://git-wip-us.apache.org/repos/asf/wicket/blob/7eb4db6e/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 e8c1079..8754fc2 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
@@ -1289,4 +1289,34 @@ public class WicketTesterTest extends WicketTestCase
 
                tester.startPage(new MockPageParameterPage(new 
PageParameters()));
        }
+       
+       /**
+        * https://issues.apache.org/jira/browse/WICKET-5665
+        */
+       @Test
+       public void assertInvisibleComponentInAjaxResponse()
+       {
+               MockPageWithLinkAndLabel page = new MockPageWithLinkAndLabel();
+               final Label label = new 
Label(MockPageWithLinkAndLabel.LABEL_ID, "Some text");
+               label.setOutputMarkupPlaceholderTag(true);
+               AjaxLink link = new AjaxLink(MockPageWithLinkAndLabel.LINK_ID)
+               {
+                       @Override
+                       public void onClick(AjaxRequestTarget target)
+                       {
+                               label.setVisible(false);
+                               target.add(label);
+                       }
+               };
+               link.add(label);
+               page.add(link);
+
+               tester.startPage(page);
+
+               tester.assertRenderedPage(MockPageWithLinkAndLabel.class);
+
+               tester.clickLink("link", true);
+
+               
tester.assertComponentOnAjaxResponse(label.getPageRelativePath());
+       }
 }

Reply via email to