[
https://issues.apache.org/jira/browse/WICKET-4112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13122052#comment-13122052
]
Andrea Del Bene commented on WICKET-4112:
-----------------------------------------
IMHO we should consider to remove the following code
if (startComponent != null)
{
path = startComponent.getId() + ":" + path;
}
inside BaseWicketTester.getComponentFromLastRenderedPage(String path,final
boolean wantVisibleInHierarchy). This method shouòd use an absolute path to
retrieve component from page.
> WicketTester#startComponentInPage and WicketTester#assertModelValue have
> inconsistent behavior.
> -----------------------------------------------------------------------------------------------
>
> Key: WICKET-4112
> URL: https://issues.apache.org/jira/browse/WICKET-4112
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 1.5.0, 1.5.1
> Reporter: Martin Dilger
> Attachments: wicket.zip
>
>
> I try the following (in Pseudocode):
> tester = new WicketTester();
> tester.startComponentInPage(new Label("test","hello");
> tester.assertModelValue("test","hello"); --> throws
> java.lang.IllegalArgumentException: Component is not a container and so does
> not contain the path test.
> Funny as it is, to make this work, you have to do the following assertion:
> tester.assertModelValue("", "hello");
> The error lies in
> public Component getComponentFromLastRenderedPage(String path,
> final boolean wantVisibleInHierarchy)
> in Line: path = startComponent.getId() + ":" + path;
> I will provide a patch this evening.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
